你们更偏向那种形式？

偏向

形式

种

35 replies • 2014-11-22 10:41:19 +08:00

1

21grams

Nov 20, 2014

一般来说第二种，可读性更好

2

yjsslab

Nov 20, 2014

尽量不在if里使用否定吧，也就是尽量不用unless

3

vulgur

Nov 20, 2014

我是第一种，不过不写“is not None”，就是“if something”

4

tabris17

Nov 20, 2014

1

分清主次

big_block_of_code()是程序的主干

something is None 之后是分支，次要的

应该把次要的分支写在if里

5

jimwoo

Nov 20, 2014

- -！什么东西？？看不懂！！我只看到了标题！

6

yufz

Nov 20, 2014

@jimwoo 因为lz放了个gist,但是由于某些原因就看不到了你懂的

7

mulog

Nov 20, 2014

@vulgur 这两个语义是不一样的不是任何时候都能替换的哦。。

至于LZ的问题我个人同意＠tabris17 把相对来说更“异常”的情况处理写到if里

8

rwx

Nov 20, 2014

个人来说第二种
习惯把异常或者错误情况排除之后再写主逻辑，尽可能减少if嵌套

9

onlyice

Nov 20, 2014 via Android

@jimwoo 我也看不到东西，是图片没刷出来？

10

faceair

Nov 20, 2014

1

@jimwoo
@onlyice

gist被dns污染推荐chinadns

11

Ge4Los

Nov 20, 2014

我用第二种, 优先跳出异常或错误, 然后执行主要代码;
主要代码的缩进会靠前, 更容易阅读

12

ytll21

Nov 20, 2014

说看不见的看这个
# First style
if something is not None: # or whatever condition
big_block_of_code()

# Second style
if something is None: # or whatever condition
continue # or return None or whatever
big_block_of_code()

13

jimwoo

Nov 20, 2014

@faceair 我擦~我第一次来V2EX……果断落伍了！

14

xidianlz

Nov 20, 2014

if something is not None:
为啥不用
if something:
这样不是更pythonic一点？

15

Sylv

Nov 20, 2014

@xidianlz 这两个是有区别的，例如 something = 0

16

ozking

Nov 20, 2014

第二个

17

ozking

Nov 20, 2014

相对来说少缩进一层，

18

imn1

Nov 20, 2014

貌似这两个逻辑不同

单一条件不会有太大区别，只是可读性的问题，如果多条件，可以考虑以下短路方式减少判断

19

rebornix

Nov 20, 2014

如果代码比较复杂，我觉得通过fast return来减少内嵌层级比较好。倾向第二种。

20

clino

Nov 20, 2014

@faceair 我用chinadns一样不行阿

2014-11-20 14:51:10 INFO request gist.github.com
2014-11-20 14:51:10 INFO response gist.github.com: [('203.161.230.171', 1, 1)]

21

xidianlz

Nov 20, 2014

@Sylv something = 0时候
something和something is not None不都是True么？

22

regmach

Nov 20, 2014

我喜欢第二种

23

Sylv

Nov 20, 2014 via iPhone

@xidianlz 你试试 something = 0 和 something = '' 的情况就知道了

24

yetone

Nov 20, 2014

第二种学名是防御式编程，推荐的写法。

25

mulog

Nov 20, 2014

@xidianlz
显然不是。。。
a = 0
if a:
print a
什么也不会print

26

hustlzp

Nov 20, 2014

个人第二种

27

cvrock

Nov 20, 2014

显然是要分情况的，第一种情况可能会导致if嵌套太深，第二种情况可能会导致每个if内都有一个返回的出口，需要根据具体场景决定选择哪种写法。

28

fish748

Nov 21, 2014

现在用第二种，嵌套少一层。

29

lightening

Nov 21, 2014

if some_condition
do_something
end

def do_something
big_block_of_code
end

30

dreampuf

Nov 21, 2014

TLTR: “等于”比“不等于”更符合直觉，避免嵌套优先执行异常分支。选第二种。

via. Code Complete 2nd

Summary of Techniques for Reducing Deep Nesting

The following is a list of the techniques you can use to reduce deep nesting, along with references to the sections in this book that discuss the techniques:

- Retest part of the condition (this section)

- Convert to if-then-elses (this section)

- Convert to a case statement (this section)

- Factor deeply nested code into its own routine (this section)

- Use objects and polymorphic dispatch (this section)

- Rewrite the code to use a status variable (in Section 17.3)

- Use guard clauses to exit a routine and make the nominal path through the code clearer (in Section 17.1)

- Use exceptions (Section 8.4)

- Redesign deeply nested code entirely (this section)

31

dingyaguang117

Nov 21, 2014

第二种

32

whalegia

Nov 22, 2014

这种这么细节的知识你们是在哪里看的啊……orz
我不看到这种讨论贴就根本不知道

33

jox

Nov 22, 2014

我尽量不在选择分支里进行这种判断，选择分支里的值只有两个，逻辑真或逻辑假，如果在某个对象的值不存在的时候会使用函数或者提前计算得到布尔值，然后再进行判断。类似这样的：

if isItExist(object):
-- ...
else:
-- ...

函数isItExist()的返回类型是布尔值

34

66450146

OP

Nov 22, 2014

@jox 如果是对数值进行合法性验证呢？

35

jox

Nov 22, 2014

@66450146 合法就是逻辑真，否则就是逻辑假呗

if isValid(object):
-- ....
else:
-- ....

我觉得限制选择分支里的类型为布尔类型，尽量少用0或者void来代替逻辑假，用0或者void(None)之类的代替逻辑假会导致逻辑混乱