很多公司都有代码走读的流程,不过据我所知,这个环节更多时候是走走形式。在走读他人代码时,我们的精力更多地是放在理解对方的实现细节,顺便看看有没有内存泄漏、越界访问、变量没初始化等之类的小BUG。很少有人指出设计或风格上的缺陷,更别提大刀阔斧地修改了。
一直以来,我对此习以为常,一度怀疑这是中国软件公司里的“潜规则”(中国人比较中庸)。直至今天看到《重构——改善既有代码的设计》一书中有关“复审代码时重构”,发现我们对项目中“坏味道”的代码忍耐出奇的强。
“重构还可以帮助代码复审工作得到更具体的结果。不仅获得建议,而且其中许多建议能立刻实现。”