实习生：上次code review得罪了老员工，这次我一句话都不说了！

sssdhua

于 2018-12-31 21:48:41 发布

阅读量846

点赞数

分类专栏：程序员

本文链接：https://blog.csdn.net/sssdhua/article/details/85489261

版权

程序员专栏收录该内容

217 篇文章 0 订阅

订阅专栏

关于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的时候还是要勇于发言的，表现出积极的一面，但是具体的说话方式自己一定要好好衡量一下。