如果条件分支里的表达式过于复杂,出现了太多的 not/and/or,那么这段代码的可读性就会大打折扣,比如下面这段代码:
# 如果活动还在开放,并且活动剩余名额大于 10,为所有性别为女性,或者级别大于 3
# 的活跃用户发放 10000 个金币
if activity.is_active and activity.remaining > 10 and \
user.is_active and (user.sex == 'female' or user.level > 3):
user.add_coins(10000)
return
对于这样的代码,我们可以考虑将具体的分支逻辑封装成函数或者方法,来达到简化代码的目的:
if activity.allow_new_user() and user.match_activity_condition():
user.add_coins(10000)
return
事实上,将代码改写后,之前的注释文字其实也可以去掉了。因为后面这段代码已经达到了自说明的目的。至于具体的 什么样的用户满足活动条件? 这种问题,就应由具体的 match_activity_condition() 方法来回答了。
Hint: 恰当的封装不光直接改善了代码的可读性,事实上,如果上面的活动判断逻辑在代码中出现了不止一次的话,封装更是必须的。不然重复代码会极大的破坏这段逻辑的可维护性。
1
j2gg0s 2019-01-20 23:26:25 +08:00
According to PEP8, long lines should be placed in parentheses. When using parentheses, the lines can be broken up without using backslashes.
|
2
ericls 2019-01-21 05:02:51 +08:00
如果没有需要重用,避免过早优化
这种问题只要整个 team 统一用一个 formatter 和配置 就好, 比如 black |
3
wdv2ly 2019-01-21 07:32:33 +08:00 via Android
编程基础第一节就没必要科普了吧?
|
4
nekoneko 2019-01-21 09:06:09 +08:00
二楼说得对
|
5
xpresslink 2019-01-21 16:27:12 +08:00
谢谢楼主不辞辛劳从火星赶来告诉大家大清忘了。
|