楼主何不做一个程序员去改进自己现在厂子的业务呢？

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

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

吹毛求疵讲两点：