谁需要改进自己的代码

[wuzhi1234]

背景：

AB 的代码能力是 50，C 的代码能力是 100 AB 的代码互相都可以看懂，并且可以互相修理对方的 bug C 可以看懂 AB 的代码，并且可以互相修理他俩的 bug AB 很难看懂 C 的部分代码，并且无法修理对应的 bug

问题： 谁需要改进自己的代码

[ghiei9101]

C 需要改进注释和文档

[sea516]

有 bug 不能给 100

[wuzhi1234]

@ghiei9101 注释经过投诉后增加挺多的，但是私有方法跳来跳去，一个方法动辄上百行，真的很难看下去

[hsuan]

AB

[ytll21]

一个方法动辄上百行，真的很难看下去 -> 这个能给 100 ？

[nino]

代码写的看不懂恐怕给不了 100 分啊

[whypool]

那 C 的 100 就有点假了
100 的人也是经历过 50 的阶段的

[wuzhi1234]

@ytll21 我是随便给个分数，满分 1000

[eaglexiang]

代码能力的体现之一就是易读性与易维护性，为什么 100 写出来的代码反而不如 50 的代码易懂？
AB 互相能理解对方的代码？两个代码能力很差的人，别说对方的代码，过两个月连自己的代码都理解不了。

[msg7086]

不知道这打分怎么来的。
写出来的代码要短小精悍，易读，不容易读的部分要加注，有条件的话要做单元测试和特性测试。
要是 1000 分打出 50 和 100，那这几个人全都得开了。

私有方法上百行，内部没注释，其实是小事。如果你特牛逼，上百行代码不出错，无 Bug （至少是无显著 Bug ），完美测试覆盖，内部就当做黑匣子来做都行。如果不牛逼，那就老老实实按照要求写代码。

[wuzhi1234]

@msg7086 你说的很在理，上百行的本事是你的代码别人不用维护不用管

[Richardhtw]

代码能力强不是应该写出简明易懂高效便于维护的代码吗？？？
感觉前提不成立啊。

[SuperMild]

一个重点：C 的代码，难懂的部分，是不是对系统有很大作用（比如性能提高很多，并且该性能是系统原本的短板之一）。

如果有简明易懂的方法来实现 C 的难懂部分，并且性能啥的没什么损失，那 C 这是能力不行啊，或者明明能力很行，却故意弄些难维护的代码出来，这就不是代码改不改进的问题，是心态有问题。

[fatttt]

@wuzhi1234 一个方法超过 100 行就要思考一下了

[SuperMild]

还有就是，AB 代码所谓的易懂，是不是没考虑到以后系统变得复杂的问题，直接把全部逻辑写在一个函数里，前期是更好懂，但后面可能重构的工作量就大了。C 是不是用了更容易扩展的 pattern 呢。

[moult]

C 代码难懂的部分，是不是用到一些高级高效算法，亦或是符合一定的规范，还是就纯粹的抖机灵？
最好同一个需求，让 ABC 都写出一段，贴上来，大家评判一下。

举个例子，实现用户资金变动功能。
AB 写了个函数，在里面做更新余额等一系列操作，所有的操作都在一个文件里面，通俗易懂。
但是 C 却构造了一个事件机制，更新余额、记录账目流水、发送通知等操作，通过侦听这个事件来实现。
例子可能不是很恰当，大概意思是 C 灵活使用事件钩子机制，自然就比 AB 来的规范，当然 AB 就应该学习 C 了。

[qiumaoyuan]

“ AB 很难看懂 C 的部分代码，并且无法修理对应的 bug ”，说明 C 的代码能力 100 的评分不正确。

结论是需要重新评分，并且大家都要改进自己的代码。

[Lucups]

论 Code Review 的重要性。

别人看不懂。。。除非 AB 真小白，不然就是 C 有大问题。
我之前维护的一个 PHP 项目，所有的数字(0~9)值不定义成常量也就算了，还写成 ASCII 码（都用 chr(48)~chr(57) 来表示），生怕别人能看懂他的代码。

[CoderGeek]

说实话 写的有效率也该注释 抖机灵啥的没意思 C 如果不是高效之类的还不如 B

[CEBBCAT]

啥啥啥？