如何提高代码质量（管理篇）：代码复查

现在东西只能经理通过自己的经验和对下面开发人员的了解去把握了，不好把握，也不能一棒子打死

罗嗦了一大堆，没啥收获！

听了多位的讨论，我却觉得学习了不少。尤其LZ的思想令我影响最深，LZ讲得非常中肯，非常实际。  记得好像有一本书叫什么代码重构的，应该不错...

我听说亚信里面，固定时间（可能是每个月吧）会展开代码审核会，每个成员要向所有同组的成员，把自己的代码展示出来供大家一起审核，项目经理基本都为每个新加入亚信的同事（里面可能有几年经验，新加入的新同事）指出编码不规范的地方。如果编码非常不规范，还要惩罚这个新同事。

我觉得整个项目组的编码规范都向同一个方向是一件好事，问题是如何能做到...

楼主，冒失忽略了些问题:

1.开发水平和工作经验不一定成正比，更多地经验是关注于业务范畴，而不是从维护性和扩张性

2.良好的编码习惯是慢慢养成的，必须自己注重培养

3.业务逻辑问题可以由测试用例检验

4.Code review的过程是难以展开的，不是技术问题，而是人事。再说很少有项目经理会把代码质量放在项目首位

5.Code review不应该着重于重大问题，而是细小的环节。在维护中，记录下值得商榷的问题，有限度地修改。很多可能会在Code review之后，留下新的问题，带来不必要的成本。

对于程序员，良好的编码习惯是慢慢养成的；
对于管理者，良好的编码习惯是必须通过指导和督促使其养成的。

在这篇文章中，最为核心的内容是最后的“最佳实践”。认为文章过于冗长的朋友，可以跳过直接看最后一段。

管理者很多不写代码，架构师才干这事情。

在论坛里看了很多对代码复查的讨论，感慨很多。大家对代码复查褒贬不一，总体来说，都认为必要，但难于实践，其中一个核心就是组织形式。组织得不好就会流于形式，甚至遭人厌。

我认为，代码复查要组织得好，要解决3个要点：谁来复查，何时复查与怎样复查。

1、谁来复查。当然是那个对被查者有权威和管理能力的人。让有能力的人担任组长去查，既赋予了管理权利，又从一定程度保证了权威性。

2、何时复查。过于频繁的复查代价太大，影响项目进度，但完成项目再查，很多问题生米已经煮成熟饭了，再修改代价会更大。按我的经验，项目刚开始，特别是新成员加入，要查得频点儿；步入正轨以后则按阶段复查。

3、怎样复查。简单的问题可以通过工具复查。人工的复查更多地应当放到设计思路上，即设计是否合理。

想法是好的，不过实施起来难度太大，尤其是小公司，一忙起来领导也是开发者，而且还是主力，想要抽出时间去做代码复查，几乎不可能，很多只要功能不出错，基本就算完了。

同意楼上的说法,如果人多事少 又有产品化需求 可以考虑严格的代码审查  但是对新手进行代码审查是很有必要的