这取决于程序员。有些人实际上喜欢听到“这太糟糕了”，因为他们知道代码很臭，但不知道为什么。

其他程序员需要更多的照顾。我发现告诉他们坏事是好的;“这不是写代码的好方法”，然后是一些指导“这里，看看我们是否这样做，它更可读/更少警告/诸如此类”。建设性的批评是有帮助的;如果你不能说到做到，最好不要评论，即使你知道这是不好的。

唯一一个两种方法都失败的人是一个固执的行政助理，他用VBscript写了大量的宏，一切都是逆向的。她竟然厚颜无耻地告诉我，我对计算机编程一窍不通，我可以向她学习。

我不是我项目的首席开发人员，因此不能强加编码标准，但我发现糟糕的代码通常会迟早导致问题，当它发生时，我有一个更清晰的想法或解决方案。

通过在当时不插话，采取更自然的方法，我获得了领导更多的信任，他经常向我寻求想法，并让我参与项目的架构设计和部署策略。

进行代码审查，并从审查你的代码开始。

这将使人们在整个代码评审过程中感到轻松，因为您是通过评审自己的代码而不是他们的代码来开始这个过程的。从你的代码开始也会给他们提供如何做事的好例子。

我真的很喜欢EnderMB的回答，但我想补充一点:

培养一种鼓励讨论代码质量的环境，而不是将其视为敏感或禁忌。例如，我曾在一个开源项目(一个Python库)中工作，团队经常讨论新代码和错误修复。不仅可以说“嘿，我认为这样做更好”，而且这实际上是被鼓励的，也是我们用于维护高质量代码的过程的一部分。

我知道不是每个环境都有利于这种过程，但它确实对我们很有效。每一次代码提交并不一定是一次委员会会议，但它应该是完全可以接受的，您可以讨论有问题的或非最优的代码并寻求改进。毕竟，更好的代码对团队中的每个人都有好处，团队合作的一个主要概念是一起工作，而不是松散的个人团体。

如果你有一个松散的编码标准，能够指出这一点，或者表明你不能遵循代码，因为它不是正确的格式可能是值得的。

如果您没有编码格式，现在将是一个好时机。类似于这个问题的答案可能会有帮助:https://stackoverflow.com/questions/4121/team-coding-styles

可能在效果后有点晚了，但这就是一致认可的编码标准是件好事的地方。