“如何做一个代码审查？”或“如何告诉同事她的HTML吮吸？”

我刚刚开始为一家拥有一名设计师和一位程序员的小型网络公司工作.我是两个世界的经验之间的人.

我现在看到的问题是,设计师似乎正在设定标准,即使她的做法有时也是错误的(例如无效的html：使用内联元素封装块级元素,或者在没有明显需要的情况下使用表格进行布局) .我知道她的意思很好,但我也知道她所执行的标准要么过时,要么没有效果.有时,很难和她沟通,因为她似乎不愿意接受别人的想法.

我认为做同行代码审查可能有帮助.有一次,我建议我们可能有代码审查,但是我们中只有3个人,程序员和设计师并不总是共享相同的代码,所以做代码审查真的有意义吗？

我该怎么办？

我如何影响我的同事更加开放的建议和/或学习一些最佳实践？

更新：感谢大家的一切建议.我发现“寻求帮助”使我的同事变得更好,更愉快地工作,更加开放沟通.

解决方法

还要确切地解释为什么它吸吮.

坏：你的HTML吮吸

更好：您的HTML吮吸,因为您在HTML元素上使用了大量的内联样式

甚至更好：您的HTML吸引人,因为您使用了大量的内联样式,这使得更难以一次对整个网站进行审美更改,因为必须编辑每个单独的标签,而不是单个CSS规则,其针对每个人标签.此外,通过移动到单个CSS文件可能会减少页面加载时间,因为客户端可以单独缓存页面加载时间.

而且,很明显,避免使用“你的html吮吸”这样的煽动性语言也是一个好主意.我只是用“你的HTML吮吸”作为占位符来批评任何一种.