修复代码
如果评审者说他们理解不了你代码中的某些内容 , 你首先应该把代码写清晰 。如果让代码更清晰 , 添加注释来解释清楚代码的逻辑 。如果评论似乎毫无意义 , 那么您的答复应该只是代码查看工具中的解释 。
如果评审者无法理解你的某部分代码 , 那边可能未来的阅读者也可能理解不了 。在Code Review工具中回应帮不了未来的读者 , 但是代码中的注释可以 。
自我反思
写一个变更会花费你很大的精力 , 提价Code Review时会感觉如释负重 , 而且自己也相当确定所有工作已经做完了 。所以当评审者提出改进建议时 , 你很容易认为那些都是错的 , 或者认为是评审者给你不必要的阻挠 , 再或者觉得评审者应该让你提价变更 。无论如何 , 不管你怎么想 , 花点时间回想下评审者给你的反馈有助于提升公司的代码质量 。你始终问下自己“如果评审者是对的呢?”
如果你回答不了评审者的问题 , 那可能说明评审者的评论不够清楚 。
如果你认真考虑过后依旧认为你是对的 , 放心大胆地解释清楚为什么你的方法对公司更有利 。通常 , 评审者只是提供建议 , 并且希望你能思考出更好的方法 。也许你已经知道一些评审者不知道的关于用户、代码库、或者变更 , 把这些都写下来 , 给评审者更多的上下文信息 , 通常你都可以根据某些事实和评审者达成某些共识 。
解决冲突
解决冲突的第一步 , 和你的评审者达成共识 , 如果无法达成共识 , 参阅Code Review的标准获取更多内容 。
版权声明:本文为CSDN博主「帅昕 xindoo」翻译 , 版权归作者所有 。
推荐阅读
- 科比坠机身亡真相大白 科比坠机身亡现场
- 冲泡正山堂金骏眉最佳水温是多少
- 公司电脑无法上网,是mac地址与另一台电脑的冲突了?
- 盘点国内常用CDN公共库
- 兔毛内胆派克服多少钱 派克服内胆是包胆好吗
- CSRF攻击与防御
- IP地址、子网掩码、默认网关、网络地址、广播地址都是什么意思?
- 接吻一直伸舌头的男人是什么性格 男生搂女生腰能感觉出粗细吗
- cgi、fastcgi、PHP-fpm都是什么
- 蜜獾是世界上最厉害的动物吗 什么动物能打败蜜獾