你如何处理一个持续写出低质量代码的团队成员？

Question

Accepted Answer

来自团队成员的低质量代码首先是一个教练问题，而不是问责问题。我的方法很冷静、具体和及时：诊断原因、私下给出善意的具体反馈、设定明确的标准、支持他们改进，只有在没有改变时才上报。目标是成长，而不是留下书面记录。

## 我如何处理

- **区分技能差距和粗心大意。** 这些需要相反的应对方式。技能差距需要教学和例子；粗心大意需要就标准和注意力进行坦诚对话。在决定是哪一种之前，先观察模式。
- **尽早、私下并具体地给予反馈。** 不要让它积累成一场大的对抗。"这个函数很难测试，因为它做了三件事"是有效的；"你的代码很草率"不是。指向代码，而不是指向人。
- **设定明确、共享的标准。** 很多"低质量"实际上是*未说明的期望*。代码审查规范、linters 和 formatters，以及参考示例使"优秀"变得具体，并从等式中消除个人判断。
- **配对和指导。** 一起在真实变更上工作比十条审查评论教会更多，并表明你投入于他们的成功，而不仅仅是批评。
- **后续跟进。** 当你看到改进时予以承认；人们会重复受到关注的行为。如果一个模式再次出现，直接指出并重新审视计划。
- **只有在没有改进时才上报。** 如果持续的、得到支持的教练没有产生效果，请告知他们的经理。到那时，你已经有了具体情况和尝试帮助的记录，这使对话变得公平。

## 为什么这很重要

处理不当的代码审查、公开批评、含糊的投诉、静悄悄地重写人们的工作，对信任和士气的伤害远大于代码对系统的伤害。如果处理得当，同样的情况会培养一位工程师、提高团队的质量标准，并表明这里的反馈是关于帮助人们进步，而不是抓住他们的把柄。