今天看到一家公司的源代码，我疯了！

to2030

回复 125# pmerofc


    见过太多正确但一塌糊涂的代码, 稍加改动就引入无数bug, 不得不推倒重写, 这种事见多了, 就会明白我的话了.
所谓正确, 不过是通过测试. 这个正确, 是暂时的, 需求变更一来前面的正确性就没有什么意义了, 而且它也是不完备的, 因为测试用例永远只能覆盖极小部分的现实. 我的想法是, 准确性应该是不存在的.
而可读性则可感受, 可理解, 可度量, 可回溯, 可靠, 而且和代码质量尤其是可维护性强相关.

to2030

正确性有点类似因果关系, 其实是人脑臆造出来的.

to2030

回复 128# pmerofc


    如果问题是正确性和**性之间只能取一样的话, 那我也和你一样, 会选择正确性. 但是经常碰到这种选择, 只能说明那是个王八蛋公司, 王八蛋项目. 赶紧走人是正经.
就象网上经常有人质问别人, 在生存和尊严(或者别的)只能选一样的情况下, 你选哪样. 可是, 一个经常逼人们做这种二选一的社会, 是个何等糟糕的社会, 根本不应该再存在下去. 赶紧想办法改变它是正经.

SuperZ

回复  pmerofc

    见过太多正确但一塌糊涂的代码, 稍加改动就引入无数bug, 不得不推倒重写, 这种事见 ...
to2030 发表于 2010-11-17 22:20

你说的情况99％不是编码引起的，而是设计引起的！
只要你坚持一些小小规则：
一个函数超过2个屏幕就拆分；
在函数内部不使用全局变量；
函数的参数和返回值有文档说明。

那么这个函数不论怎么乱写，也没什么大问题，因为它能影响的范围很小，也很好改动。

newmax123

函数对于指针参数判空没有意义，除非接口定义了空指针的语义。
JohnBull 发表于 2010-11-17 20:50

    那么是否 所有的空指针传进来我们都应该执行 让系统core呢？ 从系统的强壮性上来看， 因为任何地方的任何一个疏忽，就导致整个系统崩溃？  比如航母上一个螺母脱离了，结果整个航母就爆炸？ kernerl 有一个bug 就该让它无法运行？ 让bug 暴露出来？

flw

那么是否 所有的空指针传进来我们都应该执行 让系统core呢？ 从系统的强壮性上来看， 因为任何地 ...
newmax123 发表于 2010-11-18 09:53

调用者负责。

说白了就是各干各事，你一造螺钉的关心人家航母干嘛，这是该你操心的吗？
你只要按照图纸做好螺钉就行了，既不能因为它是玩具上用的从而掉以轻心，也不能因为它是航母上用的所以诚惶诚恐。

machine

多余的代码而已，不必大惊小怪。常有的事。