当我的团队负责人编写意大利面代码时，我应该建议进行代码审查吗？

Question

当我inheret从我的一些同事的代码在团队（谁在例如度假去了），我简直遭受修改自己的代码，并修复它，我需要很长的时间做这件事：

• 错误代码结构
• 很多写得不好功能
• 误导性变量的名称，如xxxObj，我觉得后面包含一个列表！
• 同样的事情，但在不同的状态
• 的可能的代码路径
• 和当然，很少或无意见不可预知号码太多的变数通过他们的丛林代码来带领我。

这里是我的真正的问题，甚至我的团队领导者的代码（之前她成为一个团队领导，现在她几乎从不代码）使我遭受同样，当我继承了它出于同样的原因。 ..

现在我很害怕，如果我建议代码审查中实现，她将是一个做这件事，我不认为这是非常有用的..

因为，例如，当我被聘用 - 她在那里 - 我们一起编码，我发现她将x分配给y（复制值），其中o如果他们是对象，她不知道参考和值类型之间的区别。当然，我很好地向她解释，但是......你明白我的观点。

所以，在这种情况下，它是一个好主意，建议代码审查，以提高代码质量和减少维护时间？

如果不能，我能做些什么来帮助提高代码的质量？

PS：我认为这是值得一提的是，一旦我工作的一个代码，完全取决于我的同事的代码，这是很难理解和处理一个的输出，我从字面上告诉她本人亲自“可能会增强代码结构，所以我可以更容易处理我的代码？我发现很难理解。”她说，你是来告诉我的代码结构？然后她抱怨我领队谁发送了一封电子邮件给整个团队（针对我）说一些听起来像这样“大家好，只是你的工作任务，不多不少，如果有问题请咨询我“。

Answer 1

它始终是最好的一个项目，以保持清洁，无重复的，可读的代码。你应该在任何环境中鼓励这种最佳实践。

只要确保尊重地解决问题并且不用指尖。如果她对此有疑问，则可能需要与高级经理交谈。

这个问题应该是最好的项目，而不是某人的自我。

Answer 2

我想你在问什么：“我如何让团队提高代码质量？”

对这个问题有很多很好的答案。如何在不踩人脚的情况下做到这一点是另一个问题。

在提出解决方案之前，我先从团队中解决问题开始。像你说的。凌乱的代码会浪费你大量的时间。这是不可维护的，很难理解。让人们理解这个问题将使他们更加开放的解决方案。 你甚至可以使用代码示例来解释问题。

一旦你完成了，团队可以与解决方案一起来。你把他们转向正确的方向。

我建议为团队定义代码准则。指导方针将由整个团队决定。这些也可以帮助你做代码评论。因为您可以在添加代码评论评论时参考他们。

当建议将代码审查作为一种解决方案时，请不要关注您希望让开发人员编写更好的代码的事实。尽管这是您想要进行代码审查的原因之一。而是集中于其他许多被认为不那么敌对的原因。 像：知识分享，团队合作，每个人都在一起工作，...可能有一些其他好的理由有一个代码审查。

如果您认为团队不够成熟，无法编写干净的代码。你可以建议围绕干净的代码书做一个阅读组。例如在午餐休息期间。这对于更简洁的代码来说是一个很好的起点。

因此，在短期：

是的，你应该提出解决方案，以提高代码。我宁愿在我的团队中有一个敢于发言的开发者，而不是那些即使我们走错了路也会一直跟随的开发者。

• 让人理解的问题
• 与解决方案拿出作为一个团队一起。
• 实施这些解决方案
• 在实施的解决方案反馈
• 提高