不要改变它

还有其他的方法来做这个，正如各种其他的答案所显示的。没有一个像原始代码那样清晰。

如果你需要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

对我来说，最好的答案是@ phill -frost，然后是@wayne-werner's。

我发现有趣的是，没有人说过一个函数将返回许多不同的数据类型，这将强制检查x本身的类型来做任何进一步的工作。

所以我会将@PhilFrost的回答与保持单一类型的想法混合在一起:

def all_conditions(x):
    yield check_size(x)
    yield check_color(x)
    yield check_tone(x)
    yield check_flavor(x)

def assessed_x(x,func=all_conditions):
    for condition in func(x):
        if condition:
            return x
    return None

注意，x被作为一个参数传递，但all_conditions也被用作检查函数的传递生成器，其中所有检查函数都得到一个要检查的x，并返回True或False。通过使用带有all_conditions作为默认值的func，您可以使用assessed_x(x)，或者您可以通过func传递进一步的个性化生成器。

这样，只要一个检查通过，您就会得到x，但它总是相同的类型。

我在过去看到过一些有趣的switch/case语句的dicts实现，这让我得出了这个答案。使用您提供的示例，您将得到以下结果。(使用complete_sentences_for_function_names非常疯狂，因此check_all_conditions被重命名为status。参见(1))

def status(k = 'a', s = {'a':'b','b':'c','c':'d','d':None}) :
  select = lambda next, test : test if test else next
  d = {'a': lambda : select(s['a'], check_size()  ),
       'b': lambda : select(s['b'], check_color() ),
       'c': lambda : select(s['c'], check_tone()  ),
       'd': lambda : select(s['d'], check_flavor())}
  while k in d : k = d[k]()
  return k

select函数消除了两次调用每个check_FUNCTION的需要，即通过添加另一个函数层，如果check_FUNCTION() else，则避免check_FUNCTION()。这对于长时间运行的函数很有用。dict中的lambdas将其值的执行延迟到while循环。

作为奖励，您可以修改执行顺序，甚至通过更改k和s跳过一些测试，例如k='c'，s={'c':'b'，'b':None}减少测试的数量并反转原始的处理顺序。

那些花时间的人可能会为在堆栈中增加一两个额外层的成本和字典查找的成本而讨价还价，但你似乎更关心代码的美观。

另一种更简单的实现方式可能是:

def status(k=check_size) :
  select = lambda next, test : test if test else next
  d = {check_size  : lambda : select(check_color,  check_size()  ),
       check_color : lambda : select(check_tone,   check_color() ),
       check_tone  : lambda : select(check_flavor, check_tone()  ),
       check_flavor: lambda : select(None,         check_flavor())}
  while k in d : k = d[k]()
  return k

我指的不是pep8，而是用一个简明的描述性词语来代替句子。当然OP可能会遵循一些编码惯例，使用一些现有的代码库，或者不关心代码库中的简洁术语。

你可以使用循环:

conditions = (check_size, check_color, check_tone, check_flavor)
for condition in conditions:
    result = condition()
    if result:
        return result

这样做还有一个额外的好处，那就是您现在可以使条件的数量可变。

你可以使用map() + filter() (Python 3版本，使用Python 2中的future_builtins版本)来获得第一个这样的匹配值:

try:
    # Python 2
    from future_builtins import map, filter
except ImportError:
    # Python 3
    pass

conditions = (check_size, check_color, check_tone, check_flavor)
return next(filter(None, map(lambda f: f(), conditions)), None)

但这是否更具可读性还有待商榷。

另一种选择是使用生成器表达式:

conditions = (check_size, check_color, check_tone, check_flavor)
checks = (condition() for condition in conditions)
return next((check for check in checks if check), None)

或者使用max:

def check_all_conditions():
    return max(check_size(), check_color(), check_tone(), check_flavor()) or None