CodeReview会让你下不来台么?

故事的背景是这样滴:某同事改动以前是我写的一部分代码,除却功能上和代码位置的改动外,经常无意识的将原本的代码格式改成自己习惯的代码格式。Leader review的时候cc上了我,让我review一下,我简单看了下功能,并没有什么问题,但是做为一个对代码风格要求统一的程序猿来说,同一个文件出现多个风格,这不是我的风格。所以就有如下的comments:

CodeReview会让你下不来台么?_第1张图片
recorde.jpg

作为一个team,同一个app中的代码风格保持统一,为什么要统一风格?

  • 方便阅读
  • 方便新人学习
  • 方便查找问题

毕竟代码是给人阅读的。作为一个程序猿来说,至少之前公司(外企)接触到的程序员来说,这都不用说,每个人都会自觉遵守。但是现在呢?CodeReview只能说是个形式,根本无法提问题,你也不敢提问题,谁提问题就好像是打了人脸一样,让人下不来台。你不信?好,让我们来看看下面的对话:

CodeReview会让你下不来台么?_第2张图片
Paste_Image.png

面对这种情况,只能无语了。很简单一个review的问题,就是代码保持好原来的风格而已,作为一个工作了很多年的程序猿,难到这点都不了解?实在让我无法理解。要么你全部代码都改成你自己的风格,要么保持原来风格不变。

不知道大家对这个问题如何看呢?

你可能感兴趣的:(CodeReview会让你下不来台么?)