敏捷、快节奏环境中的代码评审

原文：https://medium.com/hackernoon/code-reviews-in-an-agile-fast-paced-environment-464d5e6ec860

什么是代码审查？🔗

代码审查 是对计算机源代码的系统化审查(有时也称为同行审查)。它旨在发现软件开发中被忽略的错误，提高软件的整体质量。

代码审查是一个棘手的雷区。许多开发人员(尤其是那些刚从大学毕业的)似乎对评审和被评审都有一种与生俱来的恐惧。

我听到的普遍担忧是，要么看起来过于咄咄逼人，用铺天盖地的评论冒犯了其他人，要么是影响我们行业许多人的消费瘟疫——“冒名顶替综合征”。

这两种反应都是本能的，在我看来，彼此紧密相连。这两种反应都源于一种不健康的主人翁意识——一种保护你的声誉和工作的近乎父辈的需要——是没有准备好应对高压环境下工作压力的副作用。

如果可以的话，考虑一下这个；一个在团队 A 工作的开发人员被分配了一个任务 X。他开始在 X 上工作，并被无数的中断和挫折所困扰——这在大多数开发人员的日子里很常见——这些中断和挫折表现为需要他关注的其他较小的任务。

这造成了延误，在这个时候，原定的最后期限几乎不可能实现。

那么，这个开发者是做什么的呢？他抄近路，跳过一些边缘测试用例，努力在重要的截止日期前完成，同时压制他的编程良知，这种良知正在狂热地侵蚀他内心的平静。尽管他的工作被打断，但他最终还是为自己能够交付成果而骄傲。

1.开发商已经处于守势，一种被围攻的心态已经形成，因为每一个小的干扰都增加了他对即将到来的最后期限的潜意识压力，似乎是对他按时交付项目的能力的攻击

2.这段代码显然不在最佳状态；更重要的是，开发人员在发出评审之前就知道这一点，这使他保持警惕，并增加了他的防御性

所以，现在形势非常紧张，只需要一点火星就能让整个房子倒塌。

经历几次这样的循环，你总是会开始感到筋疲力尽。即使你已经在一个相对无压力的环境中完成了世界上所有的编码工作，你也会开始害怕提到评审。你尽最大努力不引起波澜，希望同样可疑的礼貌会延伸到你身上。

这是一个没有赢家的恶性循环。为了防止陷入这种模式，我建议将以下内容纳入你的复习计划:

1.总是传达延误。不同意将仓促代码交付生产，就是不做。在编程世界中，因果报应是真实的，它会在某个时候回来踢你的屁股。

2.强迫自己期待代码审查。将其视为类似于提交的里程碑。沉迷于一个(沉默！)迷你万岁。重新连接你的大脑！

3.如果你很难将评论视为建设性的批评，这就是你有意识地疏远自己，并试图从第三人的角度处理所有评论的时候了。

4.当你在做回顾的时候，不要把它当成政治迫害。如果你没有发现守则的问题，你的晋升不会被推迟。

5.另一方面，如果你确实发现了需要解决的问题，你的态度应该是乐于助人而不是幸灾乐祸。要有建设性，要坚定；如果你真的相信一部分代码可以被重构以提高效率，比如说，简明扼要地陈述你的观点。没有必要痛击对方。一旦你养成了这种习惯，找东西来批评的冲动就不会再出现了。

不断提醒自己，代码评审的目的绝不是寻找最大数量的 bug，就像为了赶上最后期限而抄近路一样，破坏了制定时间表的目的。

在这里的 OnlineSales.ai ，我们已经看到代码评审对发布的代码质量有着深远的影响。它是一个优秀的工具，用于指导新员工，设置清晰的编码标准，帮助保持我们的代码库干净和一致，作为一个奖励，它甚至有助于促进现有模块的可重用性，而开发人员可能没有意识到这一点！

OnlineSales.ai 是零售商的用户获取、留存和货币化的操作系统。

该平台的核心是收集和挖掘数十亿个数据点，以构建产品目录智能、用户角色和零售经济动态。

这种系统智能与在线销售平台上的技术自动化相结合，该平台现在在单个平台上托管跨营销、用户体验和数据技术的互联应用。