研发管理 codereview

angern

为什么技术人员包括 leader 都要做 code review
谚语曰: ‘Talk Is Cheap, Show Me The Code’。知易行难，知行合一难。嘴里要讲出来总是轻松，把别人讲过的话记住，组织一下语言，再讲出来，很容易。绝知此事要躬行。设计理念你可能道听途说了一些，以为自己掌握了，但是你会做么？有能力去思考、改进自己当前的实践方式和实践中的代码细节么？不客气地说，很多人仅仅是知道并且认同了某个设计理念，进而产生了一种虚假的安心感—自己的技术并不差。但是，他根本没有去实践这些设计理念，甚至根本实践不了这些设计理念，从结果来说，他懂不懂这些道理/理念，有什么差别？变成了自欺欺人。

代码，是设计理念落地的地方，是技术的呈现和根本。同学们可以在 review 过程中做到落地沟通，不再是空对空的讨论，可以在实际问题中产生思考的碰撞，互相学习，大家都掌握团队里积累出来最好的实践方式！当然，如果 leader 没时间写代码，仅仅是 review 代码，指出其他同学某些实践方式不好，要给出好的实践的意见，即使没亲手写代码，也是对最佳实践要有很多思考。
/ t4 ?! x% \: }. u* w
为什么同学们要在 review 中思考和总结最佳实践
5 `: d8 n1 U8 }3 i' \1 d我这里先给一个我自己的总结：所谓架构师，就是掌握大量设计理念和原则、落地到各种语言及附带工具链（生态）下的实践方法、垂直行业模型理解，定制系统模型设计和工程实践规范细则。进而控制 30+万行代码项目的开发便利性、可维护性、可测试性、运营质量。

* l* l( D  X8 G! F9 T3 I3 {厉害的技术人，主要可以分为下面几个方向：

% v* W! q+ s% N, A( _掌握很多技巧，以及发现技巧一系列思路，比如很多编程大赛，比的就是这个。但是，这个对工程，用处好像并不是很大。
  V( Y( U3 P0 N' y
- \0 e" M* m3 j4 Z, A- w) e领域奠基
3 K! s: E4 t# l2 f: b- Y0 }
9 k/ p* x% ^; x- V) j0 Y比如约翰*卡马克，他创造出了现代计算机图形高效渲染的方法论。不论如果没有他，后面会不会有人发明，他就是第一个发明了。1999 年，卡马克登上了美国时代杂志评选出来的科技领域 50 大影响力人物榜单，并且名列第 10 位。但是，类似的殿堂级位置，没有几个，不够大家分，没我们的事儿。
' }  O3 s& r; e' {
, e; X+ R+ V  b2 K理论研究
8 W7 L7 I$ @  Q: F$ E% J1 ~1 u- U( s
2 e' u7 s2 v; B( o6 s! @. h八十年代李开复博士坚持采用隐含马尔可夫模型的框架，成功地开发了世界上第一个大词汇量连续语音识别系统 Sphinx。我辈工程师，好像擅长这个的很少。
3 T% D1 ]! R& g7 B7 u. [' G9 z* [, P
产品成功

9 E: K" m! @; B2 M  {小龙哥是标杆。
# N: }9 s" G: h* |0 F: b& A& k. Z
最佳实践

这个是大家都可以做到，按照上面架构师的定义。在这条路上走得好，就能为任何公司组建技术团队，组织建设高质量的系统。
0 a; y4 x# e' g
从上面的讨论中，可以看出，我们普通工程师的进化之路，就是不断打磨最佳实践方法论、落地细节。

) X6 S# \# P0 K$ K. `/ a- O代码变坏的根源
- Y! U' u$ s2 t在讨论什么代码是好代码之前，我们先讨论什么是不好的。计算机是人造的学科，我们自己制造了很多问题，进而去思考解法。


, e4 r( ?( y5 w$ j) T! J7 j

jspij1

所谓架构师，就是掌握大量设计理念和原则、落地到各种语言及附带工具链（生态）下的实践方法、垂直行业模型理解，定制系统模型设计和工程实践规范细则

nuiga

技术人要掌握很多技巧，以及发现技巧一系列思路

瞪郜望源_21

说的很是浅显易懂，确实是这样，眼高手低就是说的没有知行合一，蟹蟹分享