简洁优雅 pythonic 的代码风格和更容易阅读的代码风格你更倾向哪一种?

fast, slow, pre = head, head, None
while fast and fast.next:
    fast = fast.next.next
    pre, slow.next, slow = slow, pre, slow.next
slow = slow.next if fast else slow
while slow and slow.val == pre.val:
    pre, slow = pre.next, slow.next
return not pre

这是我的烂代码

fast = head
slow = head
pre = None

while fast and fast.next:
    fast = fast.next.next
    temp = slow
    slow = slow.next
    temp.next = pre
    pre = temp

if fast:
    slow = slow.next

while slow and slow.val == pre.val:
    pre = pre.next
    slow = slow.next

if not pre:
    return True
else:
    return False

核心逻辑完全一致, 但是在给一起刷题的一个大三实习生看时, 对方表示 dalao 的代码比较难懂, 反而觉得我的烂代码比较适合在工作中使用, 因为一眼就能看明白不会浪费时间.

感觉有点迷茫在工作中应该使用什么样的代码风格了...

slow

Fast

代码

30 条回复 • 2020-10-09 19:27:23 +08:00

linw1995

2020-09-27 19:09:20 +08:00

对于这两段代码来说，肯定是更容易阅读的好。现在回答标题问题，两种风格并不冲突…

chinazz

2020-09-27 19:18:21 +08:00

第一种可读性也不差吧。感觉第一种读起来会更快

Hstar

2020-09-27 19:18:45 +08:00

多写写，写多了就变成第一种了

Trim21

2020-09-27 19:20:31 +08:00 via Android

前三行和最后一行感觉大佬的写法好，其他的感觉你的写法好

loliordie

2020-09-27 19:26:35 +08:00

@chinazz 实际上这个题目第四行, 由于指针移动的关系, 想根据赋值的优先级看清楚指针以及节点指向关系并不是很容易的事情, 需要先在脑内依次求出右侧三个值, 再依次赋值给三个变量. 这种做法虽然避免了中间变量, 但是增加了大脑的阅读负担...

tcfenix

2020-09-27 19:33:56 +08:00

代码上用骚操作提升性能或者降低空间复杂度这个的确听过,
但是想问下,城里人都是觉得降低代码量能更牛逼么?代码量越低人越厉害么?还是代码跑的更快?

lc 本身对性能有一定的需求,在满足的前提下代码就要让人容易看懂是最有帮助的

前一种写法限定了只写 python 的人才习惯看
后一种写法则是对 python 基础一般或者基本上只是熟悉其他语言的读者也友好

可能写前者代码的大佬不需要把代码给其他普通人看吧....

mcfog

2020-09-27 19:48:38 +08:00

第一种需要改进的：加空行

第二种需要改进的：
多赋值应该还是适当用的
`if .. return True else return False` 这个谁这样写我头都给他打烂

linnchord

2020-09-27 19:50:59 +08:00

对比一下，流程完全一样，纯粹是语法熟悉度问题。

你 python 熟练了，语法翻译脑力消耗低于直白手打体力消耗以后自然就变成了上面的写法，没什么神秘的。

reus

2020-09-27 19:52:22 +08:00 via Android

@loliordie 你应该右左右左右左地读，而不是右右右左左左

imn1

2020-09-27 20:09:21 +08:00

自己写喜欢第一种，读人家的希望第二种
哈哈，我就俗人一个

然后
两个月后，自己写的也没记起当时的思路是什么 😛

结束代码块的判断，我一般只写 if，不写 else，反正 False 的情况就执行下一句也是结束，没必要再写 else
例如，continue/break/return/exit 这些
如果 return 一定是 True/False 二选一，我也是第一种写法
或者说，我会在满足需求的前提下，尽快 return 结束，甚至在循环里面就 return，而不是先 break 跳出去再 return

dustinth

2020-09-27 20:24:35 +08:00

第一种代码不难懂. 唯一不好的地方是这一行:
pre, slow.next, slow = slow, pre, slow.next
因为这一行做了几件事情, 一行下来对读者心智负担比较重.

aleung

2020-09-27 21:59:36 +08:00 via Android

第一种写法好。唯一问题是没有加空行。

littlewing

2020-09-27 22:09:23 +08:00

记住一点：代码是给人看的

lsj8924

2020-09-27 23:03:11 +08:00

第一种看起来多快呀，你一句分三行，眼球得多 paser 多少次。

lithbitren

2020-09-27 23:06:55 +08:00

个人愿意读第一种代码，只要效率没问题就行。

用临时变量来实现交换对于 python 这种解释型语言来说是有效率差别的，直接元组交换视对象类型会快 20%-200%。

如果不涉及交换，多行赋值会比单行赋值要快 20%-50%，单行赋值涉及到元组解构，如果没有交换的话应该是慢一点的，有交换的是解释器生成临时变量帮你交换，比自己声明的临时变量要快点。

相同功能的 if 块运算以及 if-else 三元运算以及短路，从时间上几乎没有差别，怎么清晰怎么来，不过短路可以做多不止三元，多元运算的时候一般是短路比较快，但缺点就是一些布尔性质为 False 的变量不太好短路。

利用变量的布尔性质来返回肯定比写 if 块要性能好点

freakxx

2020-09-27 23:21:34 +08:00

标题是有倾向性的

pythonic 的代码风格和非 pythonic 的代码风格你更倾向哪一种？
不容易阅读的代码风格和更容易阅读的代码风格你更倾向哪一种?

就 python 而言，这种赋值和解包是很舒服的，没必要用 tmp 来做一个转换

至于三元运算，这个比较短也是可以的，
按第二种写法，如果要说的，补多个 else，补足情况也是好的做法

最后的判断
if True return True 这种“容易阅读”是完全没有必要的。

freakxx

2020-09-27 23:25:12 +08:00

假如你从 python 角度看的话，

第一种代码只要你肯给他加多几个空行那么是很漂亮的，
而且用法是很简练的。