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

[jpmorn]

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


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

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

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

[ming2281]

黑点是啥，没有看懂

[jpmorn]

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

[function007]

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

[awanabe]

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

[wuchizhitu1988]

为了黑而黑

[zztczcx]

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

[hei1000]

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

[SpicyCat]

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

[jatsz]

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

[MarioLuisGarcia]

吹毛求疵讲两点：

[banricho]

如果是 JS 的话

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

感觉差别不大。。

[MarioLuisGarcia]

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

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

[aisk]

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

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

[mulog]

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

[EPr2hh6LADQWqRVH]

这哥不会用 Dict 么

[canesten]

喜感十足

[yimity]

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

[jpmorn]

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

[thinker3]

这跟 url route 很类似

[neilwong]

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

[caomaocao]

要用 dict 来表示 switch 呀~ python 里

[sherlocktheplant]

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

[icedx]

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

[icedx]

Switch

[felinx]

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

[gamexg]

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

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

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

[hepochen]

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

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


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