Python 的 if 如果判断语句过于冗长，有书写技巧不？

[pppguest3962]

如:

if (a or b) and (c.get('d') == 1 or e.get('f') == 1) and (not g or not h) and (type(i) == list or type(j) == list):  

如果 a,b,c,d...j （而且还不止）的正式变量名，均在 8 个字符以上，这种条件判断阅读和书写上一行都超屏了， if 如果下面嵌套 if ，也不好看，if 就不能逐个子条件折一下行么？

[pppguest3962]

如果 if 嵌套，那么真正的执行内容缩进到那一节，也超屏了

[lasuar]

```
and_set1 = (a or b) and (c.get('d') == 1 or e.get('f') == 1)
and_set2 = (not g or not h) and (type(i) == list or type(j) == list)

if and_set1 and and_set2:
```

[pppguest3962]

@lasuar 妙，多弄几个新变量。

[Pilippa]

长度不是问题，
条件应该可读清晰才是重要的
以下纯粹举例

is_qualified = a or b
is_allowed = c.get('d') == 1 or e.get('f') == 1
is_valid_type = isinstance(i, list) or isinstance(j, list)

if is_qualified and is_allowed and is_valid_type:
# do something here...

[pppguest3962]

还能多几行码量增加了工作量，妙

[shawnbluce]

@pppguest3962 #3 不过这种写法要注意一个问题。如果写成原来那种很长一串的话，不论是 and 还是 or 都有可能提前遇到判定短路从而跳过后面的条件，例如 a or b or c or d or e 的情况下如果 a 直接就 True 了那么后面的判断就不需要了，性能上要比多弄几个变量更好一些

[shawnbluce]

@shawnbluce #6 不是说前面这种写法就不好，只是单纯提出一种看法

[cclin]

if (
(a or b)
and (c.get("d") == 1 or e.get("f") == 1)
and (not g or not h)
and (type(i) == list or type(j) == list)
):
pass

使用 black 格式化一下就好

[gadfly3173]

不如把判断结果抽成几个变量，这也太长了

[ClericPy]

忘了哪本书提到过, if 里面逻辑太复杂, 把相关逻辑抽取成一个完整逻辑的函数, 保证可读性. 当然如果计算非常简单提取成变量也没问题, 函数是为了逻辑短路避免无用计算

[imycc]

折行呗，大概长这样，也不用什么括号

```
if cond_a \
and cond_b \
and cond_c:
pass
```

[imycc]

缩进被删了有点麻。if 折行之后下一行是两个缩进的距离，为了 if 里面一个缩进的语句区分开来

[rwecho]

pep8 会给加换行的

[imn1]

conditions =[
(a or b),
(c.get('d') == 1 or e.get('f') == 1),
(not g or not h),
(type(i) == list or type(j) == list)
]
if all(conditions):

all 和 any 是支持短路的

或者
写成闭包函数（也无需传参），每行短路判断 --> if not xxx: return False

[chevalier]

这种 if 很显然，你的程序设计有问题，逻辑划分和封装得不合理

这一个 if 的圈复杂度已经是 13 ，先不说队友得死多少脑细胞才能看明白和维护，光你的单测得写多少用例才能覆盖？

[lasuar]

@imn1 这种更妙！

[cmdOptionKana]

@chevalier 确实，这么复杂的 if 语句，逻辑没划分清楚的可能性极大。

[karloku]

如果优化不了条件的话, 可以设计成能提前进行 return/continue/break 的执行方式

当然最好是想办法优化一下判断条件, 这么长的判断式可读性很差

[ilylx2008]

Python: 时间花在排版上

[msg7086]

@shawnbluce #6
是不是可以把变量改写成 lambda 延迟执行，这样就支持短路了吧。