我有一个方法，按顺序调用其他4个方法来检查特定的条件，并立即返回(不检查以下那些)每当一个返回一些真理。
def check_all_conditions():
x = check_size()
if x:
return x
x = check_color()
if x:
return x
x = check_tone()
if x:
return x
x = check_flavor()
if x:
return x
return None
这似乎是一大堆行李规定。而不是每个2行if语句，我宁愿这样做:
x and return x
但这是无效的Python。我是不是错过了一个简单、优雅的解决方案?顺便说一句，在这种情况下，这四个检查方法可能代价很高，所以我不想多次调用它们。
当前回答
不要改变它
还有其他的方法来做这个，正如各种其他的答案所显示的。没有一个像原始代码那样清晰。
其他回答
python的方法是使用reduce(有人已经提到过)或itertools(如下所示)，但在我看来，简单地使用或操作符的短路可以产生更清晰的代码
from itertools import imap, dropwhile
def check_all_conditions():
conditions = (check_size,\
check_color,\
check_tone,\
check_flavor)
results_gen = dropwhile(lambda x:not x, imap(lambda check:check(), conditions))
try:
return results_gen.next()
except StopIteration:
return None
根据Curly定律，你可以通过拆分两个关注点来提高代码的可读性:
我要检查哪些东西? 有一件事是真的吗?
分为两个功能:
def all_conditions():
yield check_size()
yield check_color()
yield check_tone()
yield check_flavor()
def check_all_conditions():
for condition in all_conditions():
if condition:
return condition
return None
这避免了:
复杂的逻辑结构 非常长的队伍 重复
.．.同时保留一个线性的，易于阅读的流。
根据您的特定情况，您可能还可以提出更好的函数名称，使其更具可读性。
不要改变它
还有其他的方法来做这个，正如各种其他的答案所显示的。没有一个像原始代码那样清晰。
这是马丁第一个例子的一个变种。为了允许短路，它还使用了“可调用对象的集合”样式。
而不是循环，你可以使用内置的任何。
conditions = (check_size, check_color, check_tone, check_flavor)
return any(condition() for condition in conditions)
注意，any返回一个布尔值，所以如果您需要检查的确切返回值，这个解决方案将不起作用。any将不会区分14，'red'， 'sharp'， 'spicy'作为返回值，它们都将作为True返回。
你是否考虑过只写if x: return x all in一行?
def check_all_conditions():
x = check_size()
if x: return x
x = check_color()
if x: return x
x = check_tone()
if x: return x
x = check_flavor()
if x: return x
return None
这并没有比你所拥有的更少的重复性，但IMNSHO它读起来更流畅。
