需要重构此功能的帮助

Question

我目前正在完成大学作业。我们在第一学期一直在使用John Zelle的graphics.py模块。

任务是构造两种不同的图案，然后将它们放置在特定设计中，根据窗口是否为500 x 500, 700 x 700 or 900 x 900像素自动调整。

我已经完成了任务，但是，我的代码的一大部分效率很低，很长，因此这并不理想。

下面是长且效率低下的代码：

def DrawPattern(width,size,win,colour): 

    if size == 5: 
     for x in range(0,width,100): 
      drawCircleExpanse(win,x,400,colour) 
     for j in range(100,400,100): 
      drawCircleExpanse(win,j,300,colour) 
     drawCircleExpanse(win,200,200,colour) 
     for j in range(100,400,100): 
      drawCircleExpanse(win,j,100,colour) 
     for x in range(0,width,100): 
      drawCircleExpanse(win,x,0,colour) 
    if size == 7: 
     for x in range(0,width,100): 
      drawCircleExpanse(win,x,width-100,colour) 
     for j in range(100,width-100,100): 
      drawCircleExpanse(win,j,width-200,colour) 
     for i in range(200,width-200,100): 
      drawCircleExpanse(win,i,width-300,colour)   
     drawCircleExpanse(win,300,300,colour) 
     for j in range(100,width-100,100): 
      drawCircleExpanse(win,j,100,colour) 
     for x in range(0,width,100): 
      drawCircleExpanse(win,x,0,colour) 
     for i in range(200,width-200,100): 
      drawCircleExpanse(win,i,width-500,colour) 
    if size == 9: 
     for x in range(0,width,100): 
      drawCircleExpanse(win,x,width-100,colour) 
     for j in range(100,width-100,100): 
      drawCircleExpanse(win,j,width-200,colour) 
     for i in range(200,width-200,100): 
      drawCircleExpanse(win,i,width-300,colour) 
     for y in range(300,width-300,100): 
      drawCircleExpanse(win,y,width-400,colour) 
     drawCircleExpanse(win,400,400,colour) 
     for j in range(100,width-100,100): 
      drawCircleExpanse(win,j,100,colour) 
     for x in range(0,width,100): 
      drawCircleExpanse(win,x,0,colour) 
     for i in range(200,width-200,100): 
      drawCircleExpanse(win,i,width-700,colour) 
     for y in range(300,width-300,100): 
      drawCircleExpanse(win,y,width-600,colour) 

的size变量取用户的输入;例如5, 7 or 9。这转换为500 x 500, 700 x 700 or 900 x 900。与colour类似，它只会将颜色更改为用户选择的颜色之一。

此代码，将产生以下的图案，与所述尺寸5其转换为500 x 500：

的drawCircleExpanse功能，它构造图案本身，包含以下代码：

def drawCircleExpanse(win,x,y,colour): 
    rad = 50 
    for c in range(10): 
     circle = Circle(Point(50+x,(50+(c * 10)/2)+y), rad) 
     circle.setOutline(colour) 
     circle.draw(win) 
     rad -= 5 

如果有人能够帮助缩短我的大量代码块，我将不胜感激。

Answer 1

我同意，这可能是对代码审查更适合，但有两个想法，可以帮助你：

• 对于size每个值，行数（循环）之前和中心后-element drawCircleExpanse是floor(size/2)。既然你可以计算行数，你可以使用另一个循环来嵌套行循环。在中心元素draw...调用之前和之后使用嵌套循环意味着您可以完全摆脱if陈述。
• 为了使代码更容易阅读，我实际上将100的因子移动到drawCircleExpanse，以便drawCircleExpanse的坐标位于虚拟网格上而不是屏幕像素上。如果需要，本地化网格到像素的转换还可以让您更灵活地在以后更改该转换。

Answer 2

你为什么不让计算机为你做数学？对于size == 5，这是在重构第一遍：

x_list = range(0, 500, 100) + \ 
     range(100, 400, 100) + \ 
     [200] + \ 
     range(100, 400, 100) + \ 
     range(0, 500, 100) 

y_list = [400] * 5 + \ 
     [300] * 3 + \ 
     [200] + \ 
     [100] * 3 + \ 
     [0] * 5 

def drawCircleExpanse(win, x, y, colour): 
    print "%3d, %3d" % (x, y) 

win = None 
colour = None 
for x, y in zip(x_list, y_list): 
    drawCircleExpanse(win, x, y, colour) 

输出

0, 400 
100, 400 
200, 400 
300, 400 
400, 400 
100, 300 
200, 300 
300, 300 
200, 200 
100, 100 
200, 100 
300, 100 
    0, 0 
100, 0 
200, 0 
300, 0 
400, 0 

您应该能够通过使用size创建x_list和y_list进一步重构这个。

Answer 3

如果有人能帮助缩短我的 大量代码块，我将不胜感激。

这是足够短：

from graphics import * 

RADIUS = 50 
NUMBER_CIRCLES = 10 
DECREMENT = RADIUS // NUMBER_CIRCLES 

def drawCircleExpanse(win, x, y, colour): 
    for radius in range(RADIUS, 0, -DECREMENT): 
     circle = Circle(Point(RADIUS + x, (2 * RADIUS - radius) + y), radius) 
     circle.setOutline(colour) 
     circle.draw(win) 

def DrawPattern(width, size, win, colour): 
    for y in range(size, 0, -2): # 5 3 1 
     for x in range(width // 2 - y * RADIUS, width // 2 + y * RADIUS, RADIUS * 2): 
      drawCircleExpanse(win, x, width // 2 - y * RADIUS, colour) 
      if y > 1: 
       drawCircleExpanse(win, x, width // 2 + (y - 2) * RADIUS, colour) 

win = GraphWin("Optimize Drawing", 700, 700) 

DrawPattern(700, 7, win, "red") 

win.getMouse() 

win.close() 

我想你使它远远超过必要更加复杂。以上也应该适用于'3'（和300）以及可能超过5的其他值。7 & 9.我留给你的轻微窗口边界（铬）宽度有一点问题。