作为程序员，你该如何评价别人的代码？

引子

遇到一个陌生人，只看到他沾满泥土的鞋子。

夏日地铁里，闻到了周围人的汗臭。

朋友聚会，似乎有些人眼屎没擦干净。

初次相亲见面，就一直在说对方不好看。

你看到别人的种种不足，当面指出来，为什么没有人感谢你p>

不光没有人感谢，似乎还有人恶言相向，某些人还因此要打你。

你做错了吗p>

————《帐前卒的寓言》

初识，最好多点赞，并建议如何编码。如果对方是新入职的人员， 最好一次性的指出程序要修改的地方。一是他有时间修改；二是新人需要了解公司里的编码规范；三是提升他的编码水平。

在review的过程中不要否定新技术，也不要轻易肯定新技术，适用才是评判依据。为什么使用这些新技术的写起来麻烦，还是旧的无法满足需求，还是只为尝鲜调研团队，我们鼓励使用各种新的技术。不管是挖坑还是填坑，需要通过他们的使用情况，了解多种新技术的功能特性、成熟度、性能优劣和适用条件。对于功能交付团队，不管是上线还是开发软件给客户使用，能满足需求的同时，一定要为了稳定不折腾。否则后期就会被bug所累，疲于奔命。

表达相同意思的一段文字可以有不同的写法。工作要有乐趣，不要评论千篇一律、枯燥乏味。下面我要说几种评论的风格:

• 吐槽。针对熟识的同事或者针对技术水平很高的人。幽默风趣，又让人容易接受。

• 委婉建议。针对有激情的人，不应该抹杀别人的技术追求。多以疑问句为主，让其自己反思。

• 批评。针对多次犯同一错误的新人。他们犯的错误一般都是非常容易改正，例如，多写常量定义、使用linux换行符、打印不用System.out等等。有时用感叹句让批评的语气再严厉一点，否则下次新人还会犯同样的错误。

• 直白陈述。可能大多数评论采用的是这种句式。另外针对编码“得过且过”的老码农，你可能需要多写点：除了点明问题所在， 还应该建议他怎样去修改代码。这样可以减少他的返工次数，消减他的抵触情绪。

• 讥讽。除非是你的好基友，否则不要用，不要用，不要用，说三遍，不要树敌。

当面交流