这两天，在微博上表达了一下Code Review的重要性。因为翻看了阿里内部的Review Board上的记录，从上面发现Code Review做得好的是一些比较偏技术的团队，而偏业务的技术团队基本上没有看到Code Review的记录。当然，这并不能说没有记录他们没有做Code Review，于是，我问了一下以前在业务团队做过的同事有没有Code Review，他告诉我不但没有Code Review，而且他认为Code Review没用，因为：
　　1）工期压得太紧，时间连coding都不够，以上线为目的，
　　2）需求老变，代码的生命周期太短。所以，写好的代码没有任何意义，烂烂吧，反正与绩效无关。
　　我心里非常不认同这样的观点，我觉得我是程序员，我是工程师，像医生一样，不是把病人医好好了，还要对病人的长期健康负责。对于常见病，要很快 地医好病人很简单，下猛药，大量使用抗生素，好得飞快。但大家都知道，这明显是“饮鸩止渴”、“竭泽而渔”的做法。医生需要有责任心和医德，我也觉得程序 员工程师也要有相应的责任心和相应的修养。东西交给我我必需要负责，我觉得这种负责和修养不是”做出来“了事了，而是要到“做漂亮”这个级别，这是 “山寨”和“工业”的差别。而只以“做出来”为目的标准，我只能以为，这样的做法只不过是“按部班”的堆砌代码罢了，和劳动密集型的“装配生产线”和 “砌砖头”没有什么差别，在这种环境里呆着还不如离开。
　　老实说，因为去年我在业务团队的时候，我的团队也没有做Code Review，原因是多样的。其中一个重要原因是，我刚来阿里，所以，需要做的是在适应阿里的文化，任何公司都有自己的风格和特点，任何公司的做法都有他 的理由和成因，对于我这样的一个初来者，首要的是要适应和观察，不要对团队做太多的改动，跟从、理解和信任是融入的关键。（注：在建北京团队和不要专职的 测试人员上我都受到了一些来自上面和下面的阻力），所以跟着团队走没有玩Code Review。干了一年后，觉得我妥协了很多我以前所坚持的东西，觉得自己的标准在降低，想一想后背拔凉拔凉的，所以我决定坚持，而且还要坚持高标准。
　　对于Code Review很重要的这个观点，在微博上抛出来后，被一些阿里的工程师，架构师/专家，甚至架构师批评，我在和他们回复和讨论的过程中，居然发现有个 把“因为对方用户的设置”我无法回复了（我被拉黑了）。这些批评我的阿里工程师/架构师的观点总结一下如下：（还有一些直接是冷讽和骂人了，微博中我 直接删除了）（顺便说一下，阿里内还是有很多团队坚持做Code Review的）
　　1）到业务团队体会一下，倒逼工期的项目有多少？订好交付日期后再要求提前1个月的有多少？现在是做到已经不容易，更不谈做得漂亮！。
　　2）Code Review是一种教条，意义不大，有测试，只要不出错，可以了。
　　3）目标都是改进质量，有限的投入总希望能有大的产出，不同沉湎改进质量的方式不一样，业务应用开发忙的跟狗一样，而且业务逻辑变化快，通用性差，codereviw的成本要比底层高。
　　4）现在的主要矛盾是倒排出来的工期和不靠谱的程序员之间的矛盾，我认为cr不是解决这个问题的银弹。不从实际情况出发光打正义的嘴炮实在太过于自慰了 。
　　我们可以看到，上面观点其实和Code Review没有太多关系，其实是在抱怨另外的问题。这些观点其实是技 术团队和业务团队的矛盾，但不知道为什么强加给了我的“Code Review很重要”的这个观点，然后这些观点反过来冲击“Code Reivew”，并说“Code Review无用”。这种讨论问题的方式在很常见，你说A，我说B，本来A、B是两件事，但是要混为一谈，然后似是而非的用B来证明你的A观点是错的。 （也许，这些工程师/架构师心存怨气，需要一个发泄的通道）
　　我觉得，很多时候，人思考问题思考不清楚，很大一部分原因是因为把很多问题混为一谈，连我自己有些时候都会这样。引以为戒。
　　即然被混为一谈，那我来拆分一下，也是下面这三个问题：
　　Code Review有没有用的问题。
　　Code Review做不起来的问题。
　　业务变化快，速度快的问题，技术疲于跟命。
　　Code Review
　　你Google一下Code Reivew这个关键词，你会发现Code Review的好处基本上是不存在争议的，有很多很多的文章和博文都在说Code Review的重要性，怎么做会更好，而且很多公司在面试过程中会加入“Code Review”的问题。打开Wikipedia的词条你会看到这样的描述——
　　Code Review的好处我觉得不用多说了，主要是让你的代码可以更好的组织起来，有更易读，有更高的维护性，同时可以达到知识共享，找到bug只是其中的副产品。这个东西已经不新鲜了，你上网可以找到很多文章，我不多说了。像你写程序要判断错误一样，Code Review也是基本的常识性的东西。
　　我从2002年开始浸泡在严格的Code Review中，我的个人成长和Code Review有很大的关系，如果我的成长过程中没有经历过Code Review这个事，我完全不敢想像。
　　我个人认为代码有这几种级别：1）可编译，2）可运行，3）可测试，4）可读，5）可维护，6）可重用。通过自动化测试的代码只能达到第3）级，而通过Code Review的代码少会在第4）级甚至更高。关于Code Review，你可以参看本站的《Code Review中的几个提示》
　　可见，Code Review直接关系到了你的工程能力！
　　Code Review 的问题
　　有下面几个情况会让你的Code Review没有效果。
　　首当其冲的是——“人员能力不足”，我经历过这样的情况，Code Review的过程中，大家大眼瞪小眼，没有什么好的想法，不知道什么是好的代码，什么是不好的代码。导致Code Review大多数都在代码风格上。，我告诉你，代码风格这种事，是每个程序员自查的事情，不应该浪费大家的时间。对此，我有两个建议：1）你团队的 人招错了，该换血了。2）让你团队的人花时候阅读一下《代码大全》这本书（当然，还要读很多基础知识的书）。
　　次当其冲的是——“结果更重要”，也是说，做出来更重要，做漂亮不重要。因为我的KPI和年终奖based on how many works I’ve done！而不是How perfect they are ! 这让我想到那些天天在用Spring MVC 做CRUD网页的工程师，我承认，他们很熟练。大量的重复劳动。其实，仔细想一下好多东西是可以框架化，模板化，或是自动生成的。所以，为了堆出这么多网 页停地去堆，做的东西是很多，但是没有任何成长。急功近利，也许，你做得多，拿到了不错的年终奖，但是你失去的也多，失去了成为一个卓越工程师的机会。 你本来可以让你的月薪在1-2年后翻1-2倍的，但一年后你只拿到了为数不多的年终奖。
　　然后是——“人员的态度问题”，一方面是懒，不想精益求精，只要干完活交差了事。对此，你更要大力开展 Code Review了，让这种人写出来的代码曝光在更多人面前，让他为质量不好的代码蒙羞。另一方面，有人会觉得那是别人的模块，我不懂，也没时间 去懂，不懂他的业务怎么做Code Review? 我只想说，如果你的团队里这样的“各个自扫门前雪”的事越多，那么这个团队也越没主动性，没有主动性也越不可能是个好团队，做的东西也不可能好。而对 于个人来说，也越不可能有成长。
　　接下来是——“需求变化的问题”，有人认识，需求变得快，代码的生存周期比较短，不需要好的代码，反正过两天这 些代码会被废弃了。如果是一次性的东西，的确质量不需要太高，反正用了扔。但是，我觉得多多少少要Review一下这个一次性的烂代码不会影响那些长 期在用的代码吧，如果你的项目全部都是临时代码，那么你团队是不是也是一个临时团队？关于如果应对需求变化，你可以看看本站的《需求变化与IoC》《Unix的设计思想来应对多变的需求》的文章 ，从这些文章中，我相信你可以看到对于需求变化的代码质量需要的更高。
　　后是——“时间不够问题”，如果是业务逼得紧，让你疲于奔命，那么这不是Code Review好不好问题，这是需求管理和项目管理的问题以及别的非技术的问题。下面我会说。
　　不管怎么样，上述Code Review的问题不应该成为“Code Review无意义”的理由或借口，这好像“因噎废食”一样。干什么事都会有困难和问题的，有的人这样退缩了，但有的人看得到利大于弊，还是去坚持， 人与人的不同正在这个地方。这是为什么运动会受伤，但还是会人去运动，而有人因为怕受伤退缩了一样。
　　被业务逼得太紧
　　被业务逼得太紧，需求乱变，这其实和Code Review没有多大关系了。对此，我想先讲一个我的故事。
　　我去年在阿里的聚石塔，刚去的时候，聚石塔正在做一个很大的重构——对架构的大调整。因此压了很多业务需求，等这个项目花了2-3个月做完了后，一 下子涌入了很多30-50个需求，搞得团队疲于奔命。在累了两周后，我仔细分析了一下这些需求，发现很多需求是在重复做阿里云已经做过的东西，还有一些需 求是因为聚石塔这个平台不规范没有标准所产生的问题。于是，我做了这么三件事：
　　1）重新定义聚石塔这个产品主要目标和范围，确定哪些该做，哪些不该做。
　　2）为聚石塔制定标准 ，让阿里云的API都长得基本一样，并制订云资源的接入标准。
　　3）推动重构阿里云的Portal系统，不再实现队里云已经做过的东西，与阿里云紧密结合。
　　这些事情推动起来并不容易，聚石塔的业务方一开始也不理解，我和产品一起做业务方的工作，而阿里云也被我逼得很惨（在这里一并感谢，尤其阿里云的同 学，老实说，和阿里云跨团队合作中是我这么多年来感觉好的一次，相当赞）。通过这个事，聚石塔需求一下有质的下降了。搞得还有几个工程师来和我说，你 这么搞，聚石塔没事可干了。姑且不说工程师对聚石塔的理解是怎么样的。 我只想说，我大量地减少了需求，尽大可能联合了该联合的人，而不是自己闭门造车，并让产品的目标和方向更明确了。做了这些事情后，大家不但不用加班，而 且还有时间充电去学技术，并为聚石塔思考未来的方向和发展。去年公司996的时候，我的团队还在965（搞得跟异教徒似的），而且还有很多时间去专研新的 东西。
　　说这个故事，我不是为了得瑟，而是因为有些人在微博上抨击我是一个道貌岸然的只会谈概念讲道理的装逼犯。所以，我告诉大家我在聚石塔是怎么做的，我 公开写在这里，你也可以向相关的同学去求证我说的是不是真的。也向你证明，我可能是个装逼犯，但绝不是只会谈概念讲道的装逼犯。
　　被业务方逼得紧不要抱怨，你没有时间被逼得像牲口一样工作，这个时候，你需要的是暂停一下想一想，为什么会像牲口一样？而这正是让你变得聪明的机会。