【外评】代码审查反模式
💡
原文中文，约4700字，阅读约需12分钟。
📝
内容提要
代码审查存在一些反模式，如滥用权力、赎金说明等。在代码审查中，应减少往返次数，提出建设性建议，并避免滥用权力。
🎯
关键要点
- 代码审查可以提高代码质量，但也可能被滥用。
- 反模式包括千回百转之死、赎金说明、双人团队等。
- 千回百转之死指审查员反复提出问题，导致开发者失去信心。
- 赎金说明是审查员利用权力迫使开发者做额外工作。
- 双人团队中两个审查员提出不兼容要求，增加开发者负担。
- 猜谜游戏通过模糊批评让开发者难以回应。
- 优先级倒置让开发者在小问题上浪费时间，忽视重大问题。
- 姗姗来迟的设计审查在工作完成后提出设计问题，增加开发者压力。
- 自相矛盾的要求让开发者难以满足审查员的期望。
- 反模式主要是滥用权力，审查员应负责任地使用权力。
- 在同行评审中，审查员和开发者的目标应一致。
- 把关角色的审查员有权拒绝补丁，但应提供合理解释。
- 文章旨在提醒审查员避免反模式，促进有效的代码审查。
➡️
