Coding style 编码的好与坏实践

我已经在这家公司工作了大约8个月了，我想要一些建议，如何告诉我的同事他们写的东西是不好的做法。让我先介绍一下

我所在的公司允许客户设置问卷/调查。目前的代码已经有5年（或更早）的历史了，他们已经开发了一个支持移动设备的新平台。这个平台相对较新，但我注意到一些我称之为“坏习惯”的东西：

它们只是不区分代码和html。所有内容都是使用一个c#generaing类动态构建的，该类脱口而出html、脚本标记、内联css“style='position:relative；”等等

我的主要问题是：

我如何说服我的同事，特别是我的IT经理将这些分开

这让我很烦恼，但也许我应该放手。我们没有足够的资源来做这件事，我可能不是那个打电话的人。我只是不明白为什么制作这个新“平台”的程序员没有想到这一点

我们还有很多编码问题：

• 我们的应用程序中使用了不同版本的jquery和mootools
• 有大量重复的代码
• CSS文件被包含在彼此之后，有像“！其中的“重要”，有些被每个include覆盖。例如：“Master.css”、“另一个_file.css”都声明了完全相同的css规则

---

请帮助我说服我的IT部门，他们做的事情太杂乱无章，编写的代码无法维护，在任何地方都没有标准，听起来像是在抱怨。别误会我，我喜欢我的工作，但它真的让我很紧张。如果事情不顺利，他们会一次接一次地做一些肮脏的事情，事情会一天比一天更糟。

我想听听一些建议，如何告诉我的同事，他们写的东西是不好的做法。

。这是一种错误的方式，请以

增强功能
的形式提供任何反馈。如果你告诉他们他们错了，他们会立即进行防御。这可能更适合另一个堆栈交换。但是，您可以与IT经理会面，并解释您的担忧。也许可以演示如何改进，并解释为什么更好。团队可能没有意识到这一点，因此对其影响的解释将是好的。好吧，因为这将意味着陷入许多麻烦并花费大量资金，你可能无法让你的公司从头开始。但是，我不建议识别不良做法，而是建议识别由潜在不良做法引起的问题，并提出最佳（或更好）做法作为解决这些问题的方法。根据你的对话者，你应该调整你的演讲：对一个市场营销者来说，想想损失的时间/金钱，对一个IT演讲来说，也许是在资源消耗或质量方面，等等。。。