大家所在的公司都做 CodeReview 么？

[peterswan]

我现在所经历的公司，还没有做 CodeReview 的，也许公司太小的原因，但是我觉得 CodeReview 是非常有效的方式，能提高代码可控性和质量，还能促进技术交流。不知道你们经历的公司有 CodeReview 么，有的话可以分享一下感受，和如何进行 CodeReview 比较好。

[lxerxa]

团队小的时候还做，后来慢慢放弃了。。。

[wuhanchu]

有 review 都是好公司啊

[hasbug]

@peterswan 因为项目已经不受项目经理控制，一个老员工总把自己当回事，什么都他说了算，项目经理心累，直接不管了

[peterswan]

@liyuhang 欢迎提 pr

[shijingshijing]

@coolhubery 如果养成习惯了就真不错，大家共同进步，code review 还有一些计划类的 paperwork 其实从长远看都是必不可少的。如果为了赶工期某一个人开始写 smelly code，而且不加制止就 merge 进来，最终导致的是整个 project 散发臭味。就跟加班一样。

ps：比较好奇你们的覆盖率达到 100%，是指需求覆盖率还是语句覆盖率？如果是严格按照 MCDC 做语句覆盖，每个条件都 cover 到，那 100%就厉害了。

[peterswan]

@jiangjz 同感，有的代码看着让人头皮发麻，经过了 n 个人的修改，修改成本很高，只能重构。

[UIXX]

有 CodeReview 要两点：
1、领导强制力
2、员工自觉性
团队要有能人，员工要会进取、肯学习。其他都是扯淡。

[coolhubery]

@shijingshijing 语句 100%覆盖。除了极少部分遗留代码无法实现 100%的情况外，新增代码必须语句 100%，否则 Gerrit CheckBot 静态检查都过不了。

[coolhubery]

@fanqianger 西安...

[peterswan]

@micean 就我的经历来看，靠自觉解决不了工程中出现 smelly code，只要有这种代码，就很有可能会让项目代码维护成本增高，而且如果维护的人写的代码继续糟糕下去，会使的项目走向慢性死亡，这些时间还不如拿出来放到开发的时候去 Review，其实是节省的整体项目的时间，管理人员更应该有这个意识。

[shijingshijing]

@coolhubery 那这个成本很高了，我只在某些 safty critical 的软件上看到过类似甚至更严的方法，比如 pacemaker 产品不仅对覆盖率有要求，而且对实时性有非常高的要求，必须在指定时间内完成指定任务。

你们测试是自己做还是外包给三哥？包括 test plan，test case，test report 都是怎么做的？

[Rico]

有。且代码评审作为升等的一个参数。

[thinkmore]

每次提交代码都要 code review.

好处：学习别人写的好的代码，让别人指出自己写得不合理的地方或者造成歧义的地方。

坏处：codereview 有时候时间太长，无法快速合并代码影响开发效率。

[coolhubery]

@shijingshijing SAP HANA 内存数据库执行引擎。
因为是全新的执行引擎去替代老的引擎，所以大部分的端到端测试，性能测试，压力测试等都可以重用。自己的 change 提交的话是需要自己写 unit test 的，一些本引擎特有的特性需要自己额外写 E2E 测试。

[xoxo419]

人少 活多 事赶

[wujiyu115]

阶段性 review,一个开发周期结束,上线之前 review 一下

[woofee]

@NCry 结对编程

[micean]

@peterswan 本来也是半路接手的项目，项目本身的设计就有问题，从去年开始就跟上头不止一次提过要重构整个设计，以免项目越来越膨胀的时候彻底改不动了。然而一点卵用都没有，1 版本还在测试，2 版本的需求就过来了，3 版本的需求就开始规划了，设计思想基本没有，有点无力回天的感觉了

[peterswan]

@micean 对于这种项目，上头的人如果只想着眼前开发的速度，不考虑可维护性，如果你本身也没这么大的权利去实施重构和审查，我觉得这个项目已经走向凉凉的道路了。如果能够实施，复盘重构是必须的了，重构加入代码审查，人力充足的话老版本的迭代和重构版本同步进行，员工心里写代码也会开心提高效率。

[mingyun]

那么问题来了，有什么好的 code review 工具