code review机制
发布于 5 年前 作者 Rabbitzzc 6056 次浏览 来自 问答

请问一下大家有好的code review机制么？ 平常团队review的时候总是会出现偏差，比如突然讲到了业务细节，感觉效率好低，求大佬们分享和建议， 评论也会给其他需要review的coder建议🤗

22 回复

请问代码托管在什么上面呢？

@sunNode 内部仓库

@sunNode 算作gitlab吧

gitlab 里面不是可以提 PR 的啊，不过不可以设置多个 Reviewer，指给指定 Reviewer，在提交 PR 的模板里面关联 case 编号，介绍这个 PR，还有就是 code commit message规范就可以了。

同求code review 最佳实践~

我现在的团队没有 review, 但是我想说下理想中的场景:

  1. review 应该是异步的, 不需要强制把大家叫到一起, 盯着大屏幕 review
  2. review 不能光看逻辑, 代码风格, 实现是否优雅也应该是重点 review 范围
  3. 明确一点, review 不是给别人挑错, 而是一场技术交流, 差的指出来使人进步, 好的赞美一下向人学习
  4. 不一定非要熟悉该业务、模块的人 review, 每个人各有所长, 别人站在不同的视角可能有意想不到的收获 个人想法, 欢迎探讨指点 如果贵公司是这样的, 并且有前端 / node岗位的话, 欢迎联系我

@sunNode 这个应该算PR。Code Review会议还是要有的，这样大家可以一起学习学习，一起讨论。

@yinxin630 https://juejin.im/post/5d550f776fb9a06ae439ee81。这是我今天写的，也是问了很多人&查了很多资料，可以看一下。code review会议是需要有的，大家一起讨论会更加注重代码的最佳实践。还有一点就是code smell，一起讨论的时候有经验的同事会很快发现代码中的code smell。

简历发给我，我给你内推1798448128@qq.com🤝

1

目前团队 Code Review 已经跑了一年多了，效果不错，可能不算最佳实践，但是目前也没有不爽的体验。基于企业微信 & GitLab，截取部分内容供参考~ image.pngimage.pngimage.png

@guchongxi 感谢分享

@yinxin630 目前在美团住宿，团队在内推呢，不知道你感兴趣不

@guchongxi 这个应该就是PR吧

@Rabbitzzc 感谢，目前还没有这方面考虑哈

@Rabbitzzc 是的，GitHub - PR; GitLab - MR 一个东西

当然目前我们是约定优先，也可以使用 GitHub 的 Require pull request reviews before merging 或者 GitLab 的 Merge request approvals 来限制必须经过指定成员 Review 后才能合并分支

@Rabbitzzc 初期可以搞一些集体会议, 团队养成习惯后尽量减少, 不然总有种逼人的感觉, 达不到什么好的效果. 更主要还是费时间…

@yinxin630 所以review目标很重要，把握方向是review的难题

@guchongxi 不错

@Rabbitzzc 嗯嗯，我同意你的说法。但是实际开发过程中，不太可能有这样的会议，第一点是项目大了之后每个模块细致的方面不是每个成员都清楚，一般是小组 Leader 比较清晰；第二点这种 Review 会议比较耗时，如果改动点较大的话无法从 Review 会议中找出问题。所以我的建议还是从源头把控，从功能需求开始做的时候需要有 Design 会议，同时可以请组内小伙伴来听，最后从了解需求的小伙伴里面挑选 Reviewer。这样 Reviewer 可以根据的 PR 里面的描述，以及对需求的了解作出判断。我们之前的做法一般会将 PR Code 拉取到本地，结合未改动的 Code 给出建议，这样 Review Code 不会成为 Reviewer 的负担，也可以很快的给出回应。

@sunNode 感谢建议

@Rabbitzzc 分享一篇文章，讲的还可以。https://mp.weixin.qq.com/s/jL7IPDUxPtWOx1d9nLC5TA

嘿嘿，学习了

回到顶部