码农:给同事代码提评论被他默默删除,并警告别再评了,好奇葩!
对于相当一部分公司,都有自己带代码review系统,在这样的系统上,程序员可以查看同事发布的代码,可以学习他人的优秀的编程思想,如果发现他人的代码有不足的地方,或者有一些bug,可以在相应的地方进行标记评论,这样的平台算是很不错了,可以促进同事们之间相互学习,相互促进改正问题,从某种程度上来说,可以避免掉一些错误的发生,然而,近期,一名程序员网友吐槽他遭遇了一个奇葩的事情。
据这名程序员网友说,最近他的工作不算太忙,有空了他就在平台上看看其他人的代码,学习一下其他人的思想,如果发现问题了,就顺便提个评论帮助对方改进,上次提了几个comments(评论)然后发现被人默默从PR里面移除了,他也没有多想,今天他又在代码里提了一个评论上去了,结果,刚提交上去,那人隔着几个座位直接对他说不要加了,代码马上要提交了。于是这名程序员网友觉得这人有点奇葩,这名程序员网友说:“其他人都挺好。审查代码是互相学习,不是挺好的吗?我很感谢给我提建议的人。”,针对这名程序员网友遇到这样的事情及他个人的观点,让我们一起看看其他网友们都是怎么认为的吧!
Amazon员工:这个就看情况了,看你的comment有没有道理,还有你写comment的语气,以及你是不是一次都写完(那种每次写一点的,别人改完,再写一点,没完没了的,特别讨厌)
上世是朵花:这名网友说的可以体会,凡事也不能听一面之词,按里说给人提建议应该是受欢迎的,毕竟对对方有帮助,如果在具体操作层面做得有问题可能也不会受欢迎。
美团员工:你这个同事心态有问题
上世是朵花:如果提出的建议确实是有价值的,一味拒绝就说明是心态问题。
网友三:给建议是活雷锋啊,
上世是朵花:没错,正常情况下是这样,如果具体操作层面没啥问题的话。
网友四:我更希望提好的建议,如果仅仅是观念不同而不是对不正确的纠正或更优解那感觉不太好
上世是朵花:同感,这名网友的80%的观点我是非常同意的!
网友五:审核代码挺好的机会互相学习
上世是朵花:嗯,没错,这种机制是很好的,前提是需要有人主导一个规则,别让小伙伴们给玩坏,玩的变味了就糟糕了。
网友六:要是老是找茬说我大括号不要另起一行,我也不加你
上世是朵花:呵呵,有道理,所以我说要看具体操作层面嘛。
网友七:你们遇到过测试审查开发的代码吗?不是领导要求的,只是测试测得时候,看着代码测
上世是朵花:那说明这名测试同学是开发出身呀,这种测试方法并不是很建议,容易陷入代码逻辑,无法梳理整体思路。
网友八:除非发现错误,不然不要提意见。风格规范之类的问题应该由编码规格检查器来搞定,人没法对软件发火不是。
上世是朵花:没错,在这件事情上,大家的观点还是比较类似。
针对这样一个事情,看了之后,给人的感觉是他的同事不对,人家好心提建议为啥要拒绝呢?不过仔细想想,抛开具体的操作层面的事情来判断谁对谁错,也未免有点果断了,这要看具体提的是什么问题了,有时提的问题让对方感觉没有任何价值,甚至说还不如原来的那样,那就会让对方感觉是在浪费人家的时间,不是在帮忙,而是在帮倒忙,所以,我们评论代码也要慎重,说出去的话,就是泼出去的水,无法收回的,如果提一个好的评论,让人看了之后眼睛一亮,感觉十分有价值,我想对方会是十分敬佩你的,如果提一些不屑的问题,反而会增加对方的反感,不是么?具体这个度怎么把握与权衡就需要按实际情况去自己琢磨与领悟了,因此,针对这样的案例,我不会抛开具体事实而去评论谁对还是谁错,这样是不科学的,建议这名楼主网友可以仔细分析一下自己的问题和具体环境氛围等因素,想必一定能从中得出一个答案!
以上所有图片均来之互联网
大家好,我是“上世是朵花”。如果你有什么好的看法或者观点可以在评论区展现你的才华,互动交流,如果想进一步了解我,那就关注我吧!