这个疯狂的循环和 if 序列是怎么回事？有改进这段代码的想法吗？

Question

这段代码目前正在运行，但看起来很糟糕 - 并且可能在性能方面可以大大提高。

有什么建议吗？

def OnClick():
    global Selection, touch, set_elsb, set_vreg, set_els, BAR_Items
    A = viz.pick(0, viz.WORLD, all=False)
    if touch != A: return
    for i in BAR_Items:
        if not set_els: break
        elif BAR_Items[i] == A or SHAPES[i+"_SHP"] == A:
            if i in Selection:
                Selection.remove(i)
                BAR_Items[i].clearActions()
                VFrame.SetStatusText(frame, i + " has been deselected")
                viz.director( do_chart )
            else:
                Selection.append(i)

非常感谢！

Answer 1

虽然多了几行代码，但我认为它更清晰了。

def OnClick():
     if not set_els: return

     # swap this with the line above if viz.pick has side effects that should occur 
     A = viz.pick(0, viz.WORLD, all=False) 
     if touch != A: return


     keys = (key for key in BAR_Items
             if BAR_Items[key] == A or SHAPES[key+"_SHP"] == A)

     for key in keys:
         if key in Selection:
             Selection.remove(key)
             BAR_Items[key].clearActions()
             VFrame.SetStatusText(frame, key + " has been deselected")
             viz.director(do_chart)
         else:
             Selection.append(key)

整个 global 语句没有任何作用，因为您没有分配给它们中的任何一个。调用属性和设置键不需要 global 关键字。

Answer 2

我对此的正常方法是将其中一些重构为小方法。这通常使其更易于测试且更易于阅读。

Answer 3

如果在此代码执行期间 set_els 未在外部更改，则：

def OnClick():
    global Selection, touch, set_elsb, set_vreg, set_els, BAR_Items
    if set_els: return
    A = viz.pick(0, viz.WORLD, all=False)
    if touch != A: return
    for i in BAR_Items:
        if not (BAR_Items[i] == A or SHAPES[i+"_SHP"] == A): continue
        if i in Selection:
            Selection.remove(i)
            BAR_Items[i].clearActions()
            VFrame.SetStatusText(frame, i + " has been deselected")
            viz.director( do_chart )
        else:
            Selection.append(i)

无论如何，当我的坏代码检测器看到这样的代码时，它会闪烁红灯，尤其是对于如此大量的全局变量。

Answer 4

人们普遍认为，如果代码丑陋、令人困惑或难以理解，那么它一定是低效的。

许多人还认为，如果想让代码运行得更快，就必须对其进行丑化。

我见过一些丑陋、令人困惑的代码，其中一些运行得非常快，而另一些则存在巨大的性能问题。

我也看到过干净、清晰、漂亮的代码，同样可以这么说。

我的经验——速度和美是独立的。