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

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

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

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

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

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

您可能想要关注坏代码的影响，而不是您对它的风格是好是坏的主观看法。

耐心再怎么强调都不为过。我见过这种完全事与愿违的事情，主要是因为有人希望现在就发生变化。相当多的环境需要进化的好处，而不是革命。今天强行改变，可能会给所有人带来一个非常不愉快的环境。

接受是关键。你的方法需要考虑到你所处的环境。

听起来你所处的环境有很多“个性”。所以…我不建议使用一套编码标准。你会发现你想把这个“有趣”的项目变成一个高度结构化的工作项目(哦，太好了，接下来是什么……功能文件?)相反，正如其他人所说，你必须在一定程度上处理它。

Stay patient and work toward educating others in your direction. Start with the edges (points where your code interacts with others) and when interacting with their code try to take it as an opportunity to discuss the interface they've created and ask them if it would be okay with them if it was changed (by you or them). And fully explain why you want the change ("it will help deal with changing subsystem attributes better" or whatever). Don't nit-pick and try to change everything you see as being wrong. Once you interact with others on the edge, they should start to see how it would benefit them at the core of their code (and if you get enough momentum, go deeper and truly start to discuss modern techniques and the benefits of coding standards). If they still don't see it... maybe you'll need to deal with that within yourself (especially on a "fun" project).

耐心。进化，而不是革命。

祝你好运。

坦白地说，我相信当某人的代码更容易修改、调试、导航、理解、配置、测试和发布时，他的代码就会更好。

也就是说，我认为不可能告诉某人他/她的代码不好，而不先让他/她解释它是做什么的，或者任何人应该如何增强它(比如，创建新功能或调试它)。

只有到那时，他们的大脑才会崩溃，任何人都能看到:

全局变量值的变化几乎总是不可追踪的 庞大的函数很难阅读和理解 模式使您的代码更容易增强(只要您遵守它们的规则) (等)

也许一段结对编程就能达到目的。 至于执行编码标准——这是有帮助的，但它们离真正定义什么是好代码还很远。

根据我的经验，曾经有一段时间，我们想要将一个windows应用程序更改为一个web应用程序并进行优化，因为它更容易更新和维护。但由于我的朋友是windows应用程序的主要贡献者，他不允许更改，然后剩下的就是历史了。

寓意:为了代码优化和更好的维护，在任何编程环境中，重视组织的目标都比重视个人的目标更重要。