📈
eng-practices-cn
  • CONTRIBUTEGUIDE
  • README
  • review
    • 紧急情况
    • 代码评审开发者指南
    • developer
      • 写一个好的CL描述
      • 如何处理评审者的评论
      • CL开发者指南 —— 更好的通过代码评审
      • 简短化的CL
    • reviewer
      • 怎样写代码评论
      • 如何做代码评审?
      • 代码评审应该关注什么?
      • 代码评审步骤
      • 处理代码评审中的反驳
      • 代码评审的速度
      • 代码评审标准
Powered by GitBook
On this page
  • 不要掺杂个人情感
  • 修复代码
  • 自我反思
  • 解决冲突

Was this helpful?

  1. review
  2. developer

如何处理评审者的评论

当你提交了一个变更做代码评审时,很可能你都会收到评审者在变更中的评论。这里有一些处理这些评论的建议。

不要掺杂个人情感

代码评审的目标是维护代码库和产品的质量。如果评审者批评了你的代码,可以理解为他们在帮你、帮整个代码库、甚至是帮整个公司,而不是攻击你或者是质疑你的能力。

有时候评审者会情绪低落,然后在评论中说出一些令人沮丧的话,虽然评审者这样做不对,但作为开发者你应当有心理准备。问问你自己,“评审者试图与我交流的建设性意见是什么?” 然后照他们说的那些去做。

永远不要回应充满怒气的评论,代码评审工具中违反职业礼仪的情况永远存在。如果你真的忍无可忍,建议先离开电脑一会儿,或者干一些其他的事,等心情平复下来再回复。

通常,如果评审者没有以礼貌的方式提供反馈,请亲自向他们解释。如果你无法亲自或者通过视频和他们联系,就向他们发一份私人邮件。 友好地告诉他们你不喜欢的事情以及你希望他们做些什么。 如果他们也以非建设性的方式对此私密讨论做出回应,或者没有起到预期的作用,请酌情上报给您的经理。如果他们已经以不礼貌的方式回应,或者没有取得预期的效果,视情况汇报给你的经理。

修复代码

如果评审者说他们理解不了你代码中的某些内容,你首先应该把代码写清晰。如果让代码更清晰,添加注释来解释清楚代码的逻辑。 如果评论似乎毫无意义,那么您的答复应该只是代码查看工具中的解释。

如果评审者无法理解你的某部分代码,那边可能未来的阅读者也可能理解不了。在代码评审工具中回应帮不了未来的读者,但是代码中的注释可以。

自我反思

写一个变更会花费你很大的精力,提交代码评审时会感觉如释负重,而且自己也相当确定所有工作已经做完了。所以当评审者提出改进建议时,你很容易认为那些都是错的,或者认为是评审者给你不必要的阻挠,再或者觉得评审者应该让你提交变更。 无论如何,不管你怎么想,花点时间回想下评审者给你的反馈有助于提升公司的代码质量。你始终问下自己“如果评审者是对的呢?”

如果你回答不了评审者的问题,那可能说明评审者的评论不够清楚。

如果你认真考虑过后依旧认为你是对的,放心大胆地解释清楚为什么你的方法对公司更有利。通常,评审者只是提供建议,并且希望你能思考出更好的方法。也许你已经知道一些评审者不知道的关于用户、代码库、或者变更,把这些都写下来,给评审者更多的上下文信息,通常你都可以根据某些事实和评审者达成某些共识。

解决冲突

Previous写一个好的CL描述NextCL开发者指南 —— 更好的通过代码评审

Last updated 2 years ago

Was this helpful?

解决冲突的第一步,和你的评审者达成共识,如果无法达成共识,参阅获取更多内容。

代码评审的标准