Open arlyxiao opened 2 years ago
凡事皆有轻重缓急,Code Review 同样符合 2/8 法则。 这里根据重要性依次列出
不同水平的开发者在面对 code review 时表现出现的情况也不太一样。
初级选手,会很在意后面两点,甚至也只能在意到后面两点,特别是自身开发习惯,比如觉得 if else 看起来不顺眼,可以换成 switch case。
有经验的架构师或者成熟一点的开发者,会从更高的维度来看代码逻辑,会更关心功能的正确性还有整体的架构,业务层上方的代码会随着需求的变动而变化,今天写的代码,可能一个星期后就因为业务修改而重写了。而底层的设计则有可能很久都不会有变化。
很重点的需要说明的一点是,code review 跟公司管理层创造的技术风格会有很大关联。
凡事皆有轻重缓急,Code Review 同样符合 2/8 法则。 这里根据重要性依次列出
不同水平的开发者在面对 code review 时表现出现的情况也不太一样。
初级选手,会很在意后面两点,甚至也只能在意到后面两点,特别是自身开发习惯,比如觉得 if else 看起来不顺眼,可以换成 switch case。
有经验的架构师或者成熟一点的开发者,会从更高的维度来看代码逻辑,会更关心功能的正确性还有整体的架构,业务层上方的代码会随着需求的变动而变化,今天写的代码,可能一个星期后就因为业务修改而重写了。而底层的设计则有可能很久都不会有变化。
很重点的需要说明的一点是,code review 跟公司管理层创造的技术风格会有很大关联。