故事的背景是这样滴:某同事改动以前是我写的一部分代码,除却功能上和代码位置的改动外,经常无意识的将原本的代码格式改成自己习惯的代码格式。Leader review的时候cc上了我,让我review一下,我简单看了下功能,并没有什么问题,但是做为一个对代码风格要求统一的程序猿来说,同一个文件出现多个风格,这不是我的风格。所以就有如下的comments:
作为一个team,同一个app中的代码风格保持统一,为什么要统一风格?
- 方便阅读
- 方便新人学习
- 方便查找问题
毕竟代码是给人阅读的。作为一个程序猿来说,至少之前公司(外企)接触到的程序员来说,这都不用说,每个人都会自觉遵守。但是现在呢?CodeReview只能说是个形式,根本无法提问题,你也不敢提问题,谁提问题就好像是打了人脸一样,让人下不来台。你不信?好,让我们来看看下面的对话:
面对这种情况,只能无语了。很简单一个review的问题,就是代码保持好原来的风格而已,作为一个工作了很多年的程序猿,难到这点都不了解?实在让我无法理解。要么你全部代码都改成你自己的风格,要么保持原来风格不变。
不知道大家对这个问题如何看呢?