Python 里你们会把 if 写成单行吗(`if cond:doit()`)?

推荐学习书目

› Learn Python the Hard Way

Python Sites

› PyPI - Python Package Index

› http://diveintopython.org/toc/index.html

› Pocoo

值得关注的项目

› PyPy

› Celery

› Jinja2

› Read the Docs

› gevent

› pyenv

› virtualenv

› Stackless Python

› Beautiful Soup

› 结巴中文分词

› Green Unicorn

› Sentry

› Shovel

› Pyflakes

› pytest

Python 编程

› pep8 Checker

Styles

› PEP 8

› Google Python Style Guide

› Code Style from The Hitchhiker's Guide

This topic created in 2616 days ago, the information mentioned may be changed or developed.

有时候判断一个条件,决定是不是执行一个方法. 今天看了一下自己写的代码里,出现了不少这种类似的判断,写的时候没多想,review 的时候比较纠结.觉得不好看

if dirty:
    clean()

网上看到很多人喜欢写这样的代码,不幸的是,我使用 autopep8 会自动换行,写成这样也白扯

if dirty:clean()

这样效果看起来也一样,但是这就是赋值语句拿来做流程控制好像不太合适,可读性也有一些损失.

dirty and clean() 

clean() if dirty else None

写完这几行字我决定不改这一部分了

26 replies • 2019-03-20 13:49:07 +08:00

nfroot

Mar 18, 2019 via Android

写同一行看的时候不是很喜欢

cxyfreedom

Mar 18, 2019 via iPhone

这个我遵循 pylint 提示换行

Leigg

Mar 18, 2019 via iPhone

省略这一行代码换来的是可读性的缺失，并不值当。

Leigg

Mar 18, 2019 via iPhone

if else 的简洁我一般只在逻辑很简单时这样干

ynkkdev

Mar 18, 2019

我自己基本不这么写。。。读过的一些 python 项目源码里也没有见大佬们这么写。
主要是真的不好看吧

whileFalse

Mar 18, 2019

clean() if dirty else None

我怎么觉得这句话中 clean()总是会被调用？

Leigg

Mar 18, 2019 via iPhone

@whileFalse 并不会

Leigg

Mar 18, 2019 via iPhone

py3 可以写的更优雅一点，利用省略号这种文法。
func( ) if xx else ...

shn7798

Mar 18, 2019

比较喜欢第一种写法，判断条件和逻辑分支识别度很高。
第二种写法一眼看去需要花点时间去区分，看起来比较累。
第三种写法是 Perl 时代的风格，write only，当然破解之法就是写诗，哈哈。

Kirscheis

Mar 18, 2019

dirty and clean() 也太危险了。。万一有一天他们决定从右边开始短路怎么办。。😂

yim7

Mar 18, 2019

第一种很好，逻辑清晰，第二种是被 c 洗脑的哈皮写的，后面两个都是没事找事，代码可读性最重要

Qzier

Mar 19, 2019 via iPad

不会，因为不符合 PEP8

jerrylogin

Mar 19, 2019

可以啊。。。。pylint 不会报错啊。。。。不是和 list comprehension 很像吗

binux

Mar 19, 2019

我一般只在两个场合这么写：

1. if condition: return

2. 多个
if condition_a: do_a()
if condition_b: do_b()
if condition_c: do_c()

但是我很多年不写 python 了。

luckyc

Mar 19, 2019

vscode->右键->格式化文档

Ginson

Mar 19, 2019

一般只会在简单的条件赋值上这样写，比如 x = 1 if y > 0 else 2

MilkShake

Mar 19, 2019

一般不这么干。。自己都看不下去。

HelloAmadeus

Mar 19, 2019 via iPhone

标准库里有很多这样一行的代码

Vegetable

Mar 19, 2019

@whileFalse
@Leigg
我测试了一下
```
clean() if dirty else ...
```
这种写法会执行 clean(),@whileFalse 说的对

Vegetable

Mar 19, 2019

@whileFalse
@Leigg

...不是是我看错条件了,不会

xpresslink

Mar 19, 2019

哪有空儿想这个，写代码时候能少按一下就少按一下。
在 Pycharm 里面，都是提交代码前，习惯性的 Ctrl+A 然后再 Ctrl+Alt+L 一下，所以楼主说的这事儿根本不存在的。

sazima

Mar 19, 2019

不符合 PEP8

karllynn

Mar 19, 2019

有啥不好看的，你要写 go，到处都是

```
if err != nil {
}
```

不好看你也得忍着，不然根本编译不过

D0n9

Mar 19, 2019

@Kirscheis 啊哈哈哈哈哈

qf0129

Mar 19, 2019

自从知道有领导把代码行数作为绩效后，我再也不这样写了

Loner233

Mar 20, 2019 via Android

一行。写 Scala 常用，反正也就写着玩，不用别人读。反正隔天自己也看不懂