徽与章网站建设宗旨,杨凯做网站,网站建设服务哪家好 价格多少钱,门户网站建设管理工作在代码的编写中有一个很重要的环节#xff0c;经常会被忽视#xff0c;那就是 Code Review ,据说在 Facebook、Google 这种互联网大公司#xff0c;要求每一个提交都必须通过审查#xff0c;对于每个工程师来说 Code Review 是一项十分重要的工作#xff0c;甚至比写代码本… 在代码的编写中有一个很重要的环节经常会被忽视那就是 Code Review ,据说在 Facebook、Google 这种互联网大公司要求每一个提交都必须通过审查对于每个工程师来说 Code Review 是一项十分重要的工作甚至比写代码本身更重要。这里所说的 Code Review 是指人工的方式进行代码的检查通常会给我们带来下面的一些好处编码风格可以保持一致目前团队中虽然有编码规范的指引但在代码抽查时还是会看到很多「个性」的代码将明显问题扼杀在摇篮里有时候存在设计上的一些错误在后期要调整起来非常麻烦改动大容易引发新的问题还需要修复历史数据等新人能够快速融入团队知道团队的编码风格能学习到一些优秀代码的写法也能知道哪些是禁区不能触碰团队成员之间能够互相学习构建良好的团队氛围。不做 Code Review 也能完成功能的实现只不过慢慢会带来下面的问题从每天写功能慢慢变成每天写 Bug代码的坏味道越来越浓代码变得难以维护修改一行代码测试没有覆盖到往往就会带来很严重的后果可能过一段时间就需要进行大规模的重构新人的技能得不到快速提升。其实我们都知道 Code Review 的重要性敏捷开发中的结对编程就包含了 Code Review 但为什么却难以执行呢我认为有下面一些原因项目急时间紧完成功能都需要加班加点哪还有时间做 Code Review对 Code Review 的认知不足不够重视没有相关的流程和制度进行约束很难坚持执行下去。我们团队的代码采用私有 GitLab 服务器自然也使用了 GitLab 中的 MR 模式不清楚 MR 是什么的同学可以看看我之前写的《在团队中使用 GtiLab 中的 Merge Request 工作模式》。曾经有一个美好的设想就是利用 Merge Request 让每个人都能参与进来在 GitLab 中进行代码的讨论但非常遗憾最终没能执行起来。Code Review 的工具和方式方法非常多我们如果能挑一两种方式落地执行下去就是非常好的一个开始。上面说到 Merge Request 在团队中没有推行起来但我个人还是在经常使用我是代码合并的管理员之一当合并代码时我会重点关注两个方面1、核心代码的改动当前功能的提交是否有必要修改到这些地方理由是什么这些代码的改动有没有可能引发一些严重问题2、MR 是谁提交的如果是资深开发人员提交的代码Review 的粒度会比较粗如果是新人提交的代码则会重点关注包括规范以及逻辑的合理性。除此之外我们领导推荐的一种做法目前在团队中一直在执行就是写代码前先写空方法。将任何的需求转化成代码中间的思考过程是复杂的需要考虑很多东西性能、扩展、是否优雅等反倒是最终的编码实现相对是简单的。而写空方法的过程就是思考的过程涉及到了类的创建、抽象、组合方法的职责事先没有思考清楚是写不出来的。快速出一版空方法后再进行沟通和讨论找出其中有遗漏和有问题的点进行修改最终的版本在大方向上基本是没什么问题的。对于 Code Review 我自己也还在不断地探索和实践找到适合团队的方法执行下去然后再持续进行改进和完善。
