代码风格和可读性[was:Re:检查集合的布尔值]
在2008年9月13日星期六下午4:11,Fredrik Lundh
成为我在那句话混乱中抱怨的第一件事...
"冗长的混乱".
当您做自己认为最好的事情时,总是令人沮丧
你读了.
无论如何:我在这里学习,当然,其中的一部分是听
那些去过那里比你更长的人.
因此,感谢您的真诚评估,Fredrik :-).
也许我现在正在看错误的方向.从
folderinuse客户端的视图,他们将做:
条件= folderinuse(core)
条件.true_for(文件夹)
这太冗长了吗?这不是一个争论的声明,我真的
问你的选择.
true_for内部的表达确实很复杂,也许我可以
简化它;但是,我深信不疑
instance.forbid_to_close(文件夹)
上面有一些好处;我的意思是,一旦我阅读了这种代码,我
可以希望理解它而无需查看什么forbid_to_close
做.
- -
marco bizzarri http://notenotturne.blogspot.com/ http://iliveinpisa.blogspot.com/(_@_ _)
Marco Bizzarri写道:
好吧," true_for"的确是难以理解的,但是我不确定这是我在那个冗长的烂摊子中抱怨的第一件事...
"冗长的混乱".
当您做自己认为最好的事情时,总是令人沮丧
你读了.
无论如何:我在这里学习,当然,其中的一部分是听
那些去过那里比你更长的人.
因此,感谢您的真诚评估,Fredrik :-).
也许我现在正在看错误的方向.从
folderinuse客户端的视图,他们将做:
条件= folderinuse(core)
条件.true_for(文件夹)
这太冗长了吗?这不是一个争论的声明,我真的
问你的选择.
true_for内部的表达确实很复杂,也许我可以
简化它;但是,我深信不疑
instance.forbid_to_close(文件夹)
上面有一些好处;我的意思是,一旦我阅读了这种代码,我
可以希望理解它而无需查看什么forbid_to_close
做.
恕我直言
为了清楚起见,重视并将其重写为常规循环.有时最简单的方法
是最简单的阅读.
class folderinuse:
def __call __(self,档案库):
结果= false
例如,在self.core.active_outonogen_registration_instances():
如果是instance.forbid_to_close(ArchiveFolder):
结果= true
休息
返回结果
然后可以与:
如果folderinuse(存档文件):
...
-larry