文无第一，武无第二，大城小胖说的 python 这段代码怎么优化

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

我今天白天说得就是这个... 所以, 年轻人千万别轻易的拜大神。那些满口软件工程、重构、敏捷开发、 OOP 、 TDD 、高质量、可复用……的人啊，写起代码来说不定有多魔幻呢…

/@老赵:无论如何也应该把 cmd 提取出来，然后每个 if 分支执行后就跳过，现在先不论代码漂不漂亮，每个 if 条件都判断一遍是几个意思，至少也 elif 一下呢？ //@欧长坤:[doge]谁能讨论点儿正经的这玩意儿到底应该怎么写?

//@GeniusVczh:我仔细地看了一眼这个代码，发现很有规律啊，正确的做法是写个独立程序 parse 这个文件，然后 codegen 出这一段函数来，除此以外没有任何更好的办法。

.
轮子哥说的这个是啥，有没有库之类的

代码

拜大神

elif

Python

27 replies • 2016-01-28 15:45:29 +08:00

ming2281

Jan 28, 2016 via Android

黑点是啥，没有看懂

jpmorn

Jan 28, 2016

@ming2281 没搞懂，抱着学习的态度来的，他们说能优化，那就优化的方法呢

function007

Jan 28, 2016

不知道 python 里有没有 switch ，找到之后可以 break 出去的
话说照首字母分个类，第一步判断首字母的话，说不定能省不少工作量？

awanabe

Jan 28, 2016

py2 是没有 switch 的...只有 if 了..所以其实没啥大问题

wuchizhitu1988

Jan 28, 2016

为了黑而黑

zztczcx

Jan 28, 2016

反射，一行代码解决那么多 if

hei1000

Jan 28, 2016

虽然不是太懂 Python ，但是每次检测都执行一遍 dataItem.get("cmd")是几个意思？

SpicyCat

Jan 28, 2016

能不能发个 gist
另外，像这种一长串 if 语句到底有什么坏处？除了看起来不高大上以外？

jatsz

Jan 28, 2016

觉得代码也没有太难看，业务复杂度，你用代码也解决不了。
非要说让代码变好看一点，而然实际上用户不大，那就是讲 else 的部分改成表驱动，方法的 dispatch 改成查表。

MarioLuisGarcia

Jan 28, 2016

吹毛求疵讲两点：

banricho

Jan 28, 2016 via Android

如果是 JS 的话

缓存 cmd
obj[key]代替多个 if （ py 好像管这叫字典？）

感觉差别不大。。

MarioLuisGarcia

Jan 28, 2016

1. 这个代码里在最后的 else 分支里有多少次 if ，就会调用多少次 dataItem.get("cmd")
除了性能消耗外，还有潜在隐患。
我认为作者在这块现在 else 分支下的第一行使用 cmd_result = dataItem.get("cmd"), 之后再用 cmd_result 和字符串比较是更好的

2. 文中的比较都是相等性比较，且比较的对象字符串都不同，所以只会有一个满足条件。所以在这儿除了第一个 if 外，之后的都应该改成 elif 。这样在 elif 的某一个条件满足后，后面的 elif 语句就会停止执行了。全部使用 if 的话，每一次都要判断所有的 if 语句。

aisk

Jan 28, 2016

如果单纯因为性能考虑的话，一般都是用一个 dict 保存 key 和响应处理的函数，这样就不需要执行这么多次判断了。

不过从可读性上讲，这坨代码根本就不该写成这样。

mulog

Jan 28, 2016

别这样说不定人家代码行数计入 KPI

EPr2hh6LADQWqRVH

Jan 28, 2016

这哥不会用 Dict 么

canesten

Jan 28, 2016 via Android

喜感十足

yimity

Jan 28, 2016

我也觉得 Dict 可以。我其实不会 Python

jpmorn

Jan 28, 2016

看起来需要来个实际点的例子学学啊

thinker3

Jan 28, 2016

这跟 url route 很类似

neilwong

Jan 28, 2016 via iPhone

昨天和大城小胖因为这个微博讨论到凌晨三点…

caomaocao

Jan 28, 2016

要用 dict 来表示 switch 呀~ python 里

sherlocktheplant

Jan 28, 2016

直接设计类的时候 cmd 和函数名一一对应就好了啊参数用** kwargs 传就好了

icedx

Jan 28, 2016 via Android

有低级错误另外不要轻易用 duct 代替 swirch

icedx

Jan 28, 2016 via Android

Switch

felinx

Jan 28, 2016

根据 cmd 的字符和名字规则用 getattr 自动获取函数名，代码可以简练的多。且看命名习惯，这不是一个熟练的 Python 工程师写的，一点不 Pythonic 。

gamexg

Jan 28, 2016 via Android

data 这个变量名被多次重复使用没人提吗？之后修改逻辑时很有可能掉坑里。

其他的看起来都是小问题。

反射搞不定，参数不一致。

hepochen

Jan 28, 2016

像这种规则性非常明确的，也不需要用到额外的 dict 来建立映射。

替换这进百行的重复代码，基本思路大抵如此：
handler = getattr(self, data_item.get('cmd', '')+'_handler', None)
if handler:
handler(*[data_item.get(key) for key in inspect.getargspec(handler).args])

写出脏代码并不可耻，为脏代码辩解的倒真算是可耻的。