我有一个方法，按顺序调用其他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。我是不是错过了一个简单、优雅的解决方案?顺便说一句，在这种情况下，这四个检查方法可能代价很高，所以我不想多次调用它们。
当前回答
或者使用max:
def check_all_conditions():
return max(check_size(), check_color(), check_tone(), check_flavor()) or None
其他回答
不要改变它
还有其他的方法来做这个，正如各种其他的答案所显示的。没有一个像原始代码那样清晰。
实际上与timgeb的答案相同，但你可以使用括号来更好地格式化:
def check_all_the_things():
return (
one()
or two()
or five()
or three()
or None
)
如果你需要Python 3.8，你可以使用“赋值表达式”的新特性来减少If -else链的重复:
def check_all_conditions():
if (x := check_size()): return x
if (x := check_color()): return x
if (x := check_tone()): return x
if (x := check_flavor()): return x
return None
我很惊讶没有人提到内置的任何是为了这个目的:
def check_all_conditions():
return any([
check_size(),
check_color(),
check_tone(),
check_flavor()
])
注意，尽管这个实现可能是最清晰的，但它计算所有的检查，即使第一个检查为True。
如果你真的需要在第一次检查失败时停止，考虑使用reduce来将一个列表转换为一个简单的值:
def check_all_conditions():
checks = [check_size, check_color, check_tone, check_flavor]
return reduce(lambda a, f: a or f(), checks, False)
reduce(function, iterable[， initializer]):应用2的函数 参数从左到右累加到iterable的项， 从而将可迭代对象减少为单个值。左边的参数x， 累积值和正确的参数y是否更新 值。如果存在可选初始化式，则为 在计算中置于可迭代对象的项之前
在你的情况下:
lambda a, f: a或f()是检查累加器a或当前检查f()是否为True的函数。注意，如果a为True, f()将不会被求值。 检查包含检查函数(来自lambda的f项) False是初始值，否则不会发生检查，结果总是True
Any和reduce是函数式编程的基本工具。我强烈建议你训练这些以及地图，这是很棒的!
根据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
这避免了:
复杂的逻辑结构 非常长的队伍 重复
.．.同时保留一个线性的，易于阅读的流。
根据您的特定情况，您可能还可以提出更好的函数名称，使其更具可读性。
