什么是codereview？字面意思是审查您的代码，Talkis便宜，showme代码是它的意思。越来越多的公司要求研发团队在代码开发过程中进行CODEREVIEW（CR）。在确保代码质量的同时，它促进了团队成员之间的沟通，并提交了代码级别。然而，近年来，CR文化的促进只开始了。像亚马逊和谷歌这样的外国公司需要在代码合并到主分支机构时完成CR。国内互联网比国外延迟，加上一个略微复杂的汉语特征（我对自己的其他人写的坏代码不太尴尬......）。早期Cr更多的形式。现在随着互联网流量的迅速增加，必须保证代码质量，以确保业务的高稳定性和高可用性，因此CR开始成为企业发展的必备现象。简单地说，Cr就像吃早餐。如果你不吃它，你的身体似乎似乎并不糟糕。因此，由于忙碌的工作或睡觉后，每个人都能忽视早餐或只是吃饭和吃东西来应对。然而，由于身体在高负荷和高压下逐渐变得有点不同，因此每个人都开始意识到“身体是革命的首都”。必须吃早餐，必须始终如一地食用。这对身体最有利。更进一步，以力量更好地做到。但为什么你吃早餐，当你吃它时，你是如何吃它的，你吃的东西也很优雅。就像当Cr会这样做时，它将如何做到，应该如何注重，也很细腻。

为什么？ - 你为什么需要审查？ CR是代码标准化的保证，带来知识传播和团队建设。有些人可能认为代码审查是找到错误，并且不需要代码审查。事实上，这不是这种情况。从编码人的角度来看，他们每天都忙于编码，交货时间即将推出。为了快速交付，要求减少，未写入单元测试用例，并且在编码时的性能和安全性的角度下不考虑更好的实施。但是，虽然代码按时提交，但它不是一种高质量的代码。在运营中可能存在问题，并且人们难以接管。如果有CR，认为您的代码将由您的同事和领导审查，您还会降低要求吗？我必须加班完成以质量和数量的代码写作。此外，在CR的过程中，有高级老年人解释您的代码设计思想和算法。这绝对比降低你的头脑更快，思考进步。 CR.判断的原因就像和家人和朋友一起吃早餐。当你孤独时，你会醒来醒来或匆忙吃东西。如果您的家人或朋友与您同住，您会考虑您的家庭的健康和您的家人对您的担忧。 ，你还不会早起吃早餐甚至喝早餐？什么时候？ - 审查是什么？每个代码合并（PullRequest / Merge Request）是最好的时间。 PullRequest意味着您无权提交您写入特定仓库或分支的代码。您可以请授权人员将您从源仓库的源分支中提交的代码合并到目标仓库的目标分支中。每个要求更改应通过提交合并请求尽快合并到主分支机构中，因此可以尽早找到代码写作中的问题。我们现在所倡导的持续整合也是相同的想法。不要等到所有要求的开发和合并。一次提交大量代码也会为审稿人带来巨大的负担。修改一次并提交一次。当然，这份提交也是一个质量提交，至少在提交之前，我已经完全审查并通过了单位测试。 Cr评论的时间就像做早餐一样。你必须尝试用正确的原料和煮熟，甜蜜和咸的东西吃它，然后让别人一起吃饭。如何？ - 怎么判断？选择合适的工具，与合适的开发过程合作，选择正确的形式非常重要。对于工具，许多代码托管工具，如Github，Gitlab，Alibaba云云和腾讯工作人员Bee都有自己的CR工具，而开发团队可以根据自己的情况选择。对于开发过程，当前流行的Gitflow，Trunk开发模型和Fork开发模型所有支持PullRequest / MergeRequest当代码合并到主分支时。

出于适当的表格，包括在线审查，离线审查和特殊处理，为轻量级CR（如小型功能模块的开发，小于500行代码），您可以直接邀请代码托管工具人员中的同一组高级人士审查代码并与反馈结合进行更改;为了开发大型功能模块或架构的变化，团队成员可以组织用于离线评论。开发人员谈论自己的设计逻辑，审稿人提供意见，代码审核行按行;对于一些紧急情况，例如需要迫切地推出的线路上的紧急错误，但没有人在那里，可以在此时进行紧急合并，但之后仍然需要添加CR。 CR.评估方法就像吃早餐。如果你是一个人，它可以更简单，牛奶面包可以补充必要的蛋白质;如果是一个家庭在一起，那么它肯定会更富裕。馒头，粥，大豆牛奶，油炸面团棒和泡菜都会有一点，所有谷物都会补充;如果是紧急火车或飞行，你可以跳过吃它并等到你上火车或飞机来弥补它。什么？ - 审查是什么？ CR评论什么？在CR中，我们审查了标准化，一致性，编码样式，代码的安全问题，代码冗余和代码的功能性能设计。关于标准化，在Java中，我们将检查后台线程是否具有对主线程的同步访问，无论是功能和变量命名是否准确，是否组件分层是合理的，无论是合理提取的常用逻辑是否合理，是文件组织是否是合理的逻辑合理，无论函数评论是清晰全面的，代码的可读性都很好，无论是更优雅的写作方式，程序设计是否满足单个原理，开放和封闭的原理。为了完整性，我们检查代码是否完全实现了设计文档中提出的功能要求，无论是否已创建所需的数据库，以及它是否包含正确的初始化数据。对于正确性，我们检查是否正确定义并使用了所有变量，无论是否使用未定义的变量，是否存在明显或潜在的逻辑错误，无论它们都意外落入无限循环，是否已经避免了无限递归。对于稳健性，我们检查代码是否与阵列折叠或内存溢出的例外处理。为了重新使用，我们检查组件是可重复使用的以及是否有重复代码。为了可伸缩性，我们检查功能组件是否易于展开以及是否可以重用代码。为了安全，我们检查是否执行身份验证，授权，输入数据验证，避免安全威胁，如SQL注入和跨站点脚本（XSS），加密敏感数据（密码，信用卡信息等）以及是否导入依赖项是安全的，将成熟，公共组件和工具函数的变化会影响其他业务吗？可以看出CR不是一个简单的事情。一个好的代码和良好的工程师必须被锻炼。 CR的评论内容就像早餐一样。我们将注意哪些成分在营养方面是否均衡，无论烹饪是合适的，无论是含量是否足够，无论是成分都是安全的，无论是清洁，无论是清洁的，无论是清洁的，卫生，无论价格合适， 等等。不要说经营迭代太多，需求太多，发射时间紧张。没有时间做Cr。不要为您的丑陋代码制作华丽的借口。如果您没有时间做好准备，您将有很多时间来处理错误和投诉。 。就像不要说你没有时间吃早餐，工作太忙或困倦，你现在的拯救时间将被未来的各种疾病令人作呕。因此，如果您的团队没有完成CR，并且CR并不好好，您必须推动您的领导者找到根本原因，并实施CR。对于质量代码，一切都值得。就像那样，如果你还没有享用美味的早餐，那么从现在吃早餐时，就会为健康的身体值得一切值得。