大部分公司应该都不会 code review 吧？

[polyang]

以我所在的公司为例，平时项目周期已经很紧张了，根本没时间 code review 。

之前有个需求，从开发到联调再到 showcase，只给你一周的时间，这么短的时间，根本不会考虑什么 code review，只要能把功能实现就不错了，等到做完这个需求，还没等你缓过来，下一个需求就立马来了。

[cwliang]

code review 很有必要，可以防止一些 anti-pattern 、性能冲击、维护性，虽然没时间看具体业务逻辑

[jorneyr]

理想很丰满，现实很骨感

[polyang]

@cwliang 我也觉得有必要，但首先是得给够时间，没时间的话，根本没办法做。

[leekafai]

sr
self review

[runliuv]

有个锤子

[LemonK]

@timedivision lint 和风格是两码事。比如有个前同事：他用他自己发明的标准划分为同类的变量，要求别人必须统一前后缀命名；多种语法都 ok 的逻辑，要求必须按他喜欢的那种写法；别人写 if 的地方他要求改成三目，别人写三目的地方他又要理论一下 if 更好。如果之前在这种代码孔乙己的身边工作，强调一下代码风格不重要还是有必要的。

[8888888888]

@LemonK 风格不是一个人制定的，不管怎么说，代码这种二进制的东西，实现某个功能逻辑有很多种方法，但一定是有一个最优解的，尽管说简单的 if 判断扯不上什么最优解，但对于项目本身来说，统一的风格一定是利于维护的

[LemonK]

@timedivision “写法一定有一个最优解”这句前提完全不认同。听起来就好像天下同一题目的作文只有一种标准写法一样。除了“一定不要这样写”的 Bad smell 共识之外，自由发挥的空间依然非常大。除非你只写 JAVA 。

[8888888888]

@LemonK 不要断章取义啊，我说的是实现某个功能逻辑的方法一定有一个最优解，另外你拿作文和代码相比，这两者根本没有可比性

[mlbjay]

项目组有个老同事，技术很强，但是 CR 后 总我“建议”我改一些“可有可无”的东西，我也不敢反对，改咯。

[saulshao]

@mlbjay 这其实就是代码 review 的价值。你可以尝试从中吸收对你有用的东西。