有时我会把if中的长条件分解成几行。最明显的方法是：
if (cond1 == 'val1' and cond2 == 'val2' and
cond3 == 'val3' and cond4 == 'val4'):
do_something
视觉上不是很吸引人，因为动作与环境融为一体。然而，这是使用4个空格的正确Python缩进的自然方式。
目前我正在使用：
if ( cond1 == 'val1' and cond2 == 'val2' and
cond3 == 'val3' and cond4 == 'val4'):
do_something
但这不是很漂亮
你能推荐另一种方法吗？
当前回答
加上@krawyoti所说的。。。长时间的环境气味难闻，因为它们难以阅读和理解。使用函数或变量可以使代码更清晰。在Python中，我更喜欢使用垂直空格，括起括号，并将逻辑运算符放在每一行的开头，这样表达式看起来不像“浮动”。
conditions_met = (
cond1 == 'val1'
and cond2 == 'val2'
and cond3 == 'val3'
and cond4 == 'val4'
)
if conditions_met:
do_something
如果需要对条件进行多次求值，例如在while循环中，那么最好使用局部函数。
其他回答
我知道这个线程很旧，但我有一些Python 2.7代码，PyCharm（4.5）仍然抱怨这个例子：
if foo is not None:
if (cond1 == 'val1' and cond2 == 'val2' and
cond3 == 'val3' and cond4 == 'val4'):
# some comment about do_something
do_something
即使有PEP8警告“视觉缩进的行与下一个逻辑行缩进相同”，实际代码也完全正常？这不是“过度缩进？”
……有时我希望Python能咬紧牙关，只带上大括号。我想知道这些年来有多少错误是由于错误的缩进而被意外引入的。。。
这没有多大改善，但。。。
allCondsAreOK = (cond1 == 'val1' and cond2 == 'val2' and
cond3 == 'val3' and cond4 == 'val4')
if allCondsAreOK:
do_something
我建议将和关键字移到第二行，并用两个空格而不是四个空格缩进包含条件的所有行：
if (cond1 == 'val1' and cond2 == 'val2'
and cond3 == 'val3' and cond4 == 'val4'):
do_something
这正是我在代码中解决这个问题的方法。将关键字作为行中的第一个单词可以使条件更加可读，并且减少空格的数量可以进一步区分条件和操作。
您不需要在第二个条件行上使用4个空格。可能使用：
if (cond1 == 'val1' and cond2 == 'val2' and
cond3 == 'val3' and cond4 == 'val4'):
do_something
此外，不要忘记空格比您想象的更灵活：
if (
cond1 == 'val1' and cond2 == 'val2' and
cond3 == 'val3' and cond4 == 'val4'
):
do_something
if (cond1 == 'val1' and cond2 == 'val2' and
cond3 == 'val3' and cond4 == 'val4'):
do_something
不过这两个都相当难看。
也许会失去括号（尽管《风格指南》不鼓励这样做）？
if cond1 == 'val1' and cond2 == 'val2' and \
cond3 == 'val3' and cond4 == 'val4':
do_something
这至少给了你一些区别。
甚至：
if cond1 == 'val1' and cond2 == 'val2' and \
cond3 == 'val3' and \
cond4 == 'val4':
do_something
我想我更喜欢：
if cond1 == 'val1' and \
cond2 == 'val2' and \
cond3 == 'val3' and \
cond4 == 'val4':
do_something
这是《风格指南》，（自2010年以来）建议使用括号。
当我有一个非常大的if条件时，我更喜欢这种风格：
if (
expr1
and (expr2 or expr3)
and hasattr(thingy1, '__eq__')
or status=="HappyTimes"
):
do_stuff()
else:
do_other_stuff()
