本文共 1776 字，大约阅读时间需要 5 分钟。

关于code review一事，想必好多程序员都不会陌生，当然一些公司可能没有这一项事情，但是也会有人经常提起这个词，对于不同公司，code review的形式也是不一样的，有的比较随意，几个人围在一起一块看一遍就算一次code review了，有的公司可能有专门的线上平台，大家可以通过平台code review,并且还可以在代码上标注自己的建议等，也有的公司可能是专门有会议室，投影仪，大家坐在一起code review,总之，对于不同公司，code review的重视程度和形式也不尽相同，近期一名程序员网友给我们分享了他在code review中遇到的情况。

据这名程序员网友说，他是在公司的一名实习生，上次code review的时候把同事（老员工）得罪了，具体是什么细节他并没有过多的说，然后他说这次又要进行code review了，他准备打算这次code review一句话都不说。害怕会祸从口出，不过，我倒是觉得这就有点极端了，应该好好想想自己怎么提建议，怎么友好的表达，只要是一种善意的表达，我想大多数人还是愿意接受的吧，针对他说的这种情况，让我们一起看看其他网友们都是怎么认为的吧！ 网友一：code review只要讲事实，没什么可怕的，另外老员工要尊重不等于有错不能说，也不等于想到什么就脱口而出。程序员，最大的限制就是与人交往的能力。

上世是朵花：感觉这很可能是说话方面的问题，比如带有个人的偏见与态度等，如果只是说代码的事情，比较好的建议，我想大多数人也是乐意去接受的。

网友二：情商问题 你诚恳的提出问题 老员工未必不能接受 最常见的就是新员工总想让别人知道自己多牛逼 而且楼主明显就是小孩子呀 心态不稳定

上世是朵花：有可能是这种情况吧，作为新员工可以保持适当的低调，以谦虚的姿态示人，好多刚参加工作的，才开始都是有一点傲气，刚到一个新工作环境，都会把这种傲气有意或者无意的体现出来，可谓是未见实力，先露锋芒！

网友三：怕什么，就是要打脸

上世是朵花：也可以是友好的交流啊，为什么非要打脸呢。

网友四：说话姿势的问题

上世是朵花：感觉很有可能是这样的。

网友五：学聪明了

上世是朵花：选择一句话都不说，也并不是什么明智的选择，最好还是想想应该怎么去说比较好。

网友六：和码农沟通有技巧 要处处示弱有必要时一针见血

上世是朵花：感觉这个技巧并不适合在这里了，码农的群体还算好吧，没有你想的这么复杂！

网友七：方式方法的问题 年轻人不要太着急出名

上世是朵花：很有可能是与说话方式有关

网友八：我也被坑过，也是刚进公司时领导安排的

上世是朵花：用坑这个词视乎不太恰当吧，刚工作，好多事情都需要自己慢慢学。

其实，这个"code review"的中心是code,大家通过共同review的形式去发现代码中所存在的缺陷与问题，也是共同进步的一种方式，可是往往有相当一部分人会曲解这个概念，认为是在挑毛病，如果真的指出其中的不足就是对代码主人的一种不尊重，把纯粹的技术交流转化人与人之间的感情问题了，这样的看法并不科学，当然在code review的时候还是要讲究自己说话的方式，如果说话方式有问题，真的是会得罪人的，比如举个例子，在code review时，你这样说：“这代码好乱啊”，“这代码思路不是很清晰”，“这代码很垃圾”（这个应该不会有人说，如果能这么说的话情商该有多低呀），“这代码效率不高”，总之类似这样不具体说，只是笼统概括的就很有可能得罪人了，如果换种方式，像下面这么说：“第10-35行可以提出来单独封装一个方法，方便其他地方调用，能起到很好的复用效果”，“第100行这个递归的方法的结束终止条件可以再加一个条件，这样能提前终止，减少几次没必要的调用”，如果是这种方式去说，我想没有人会听了不高兴吧，即使你的观点不算正确，但起码能明显看出你这个是对事不对人，也不会使其他人不高兴，因此，关于这件事情，我的个人建议，在code review的时候还是要勇于发言的，表现出积极的一面，但是具体的说话方式自己一定要好好衡量一下。

以上所有图片均来之互联网

大家好，我是“上世是朵花”。如果你有什么好的看法或者观点可以在评论区展现你的才华，互动交流，如果想进一步了解我，那就关注我吧！ (微信公众号：sssdhua )

转载地址：http://qbivi.baihongyu.com/