看到的牛比代码

zylthinking

原帖由 yunhappy 于 2008-7-11 14:11 发表
感觉是被全文替换过; 可能有些功能不要了 就成了这样的

推断合理， 但我保证绝对没有替换过。
再举个例子， 还有这样的代码：

FILE* fp = fopen(...);
if(fp == 0){
    log("error open");
}
fread(..., fp);

你能说他什么好呢

[ 本帖最后由 zylthinking 于 2008-7-11 14:18 编辑 ]

yunhappy

原帖由 zylthinking 于 2008-7-11 14:17 发表


推断合理， 但我保证绝对没有替换过。
再举个例子， 还有这样的代码：

FILE* fp = fopen(...);
if(fp == 0){
    log("error open");
}
fread(..., fp);

你能说他什么好呢

得~ 重写算了

gigabyte

我喜欢这个：pagesize = pagesize

flw

很多人改代码从来不改上下文的。
见过很多这样的。

通盘考虑是好事，不过碰到烂代码的时候，往往牵扯太大太深远，因此形势有时候不允许。

一言以蔽之：在你之前最后一次修改那段代码的人，不见得就是烂人。
正如你的下一任看到这份代码的时候，不能认为你就是烂人一样。

zylthinking

汗, 近40K行........
现在宁可灌水也不想看， 别说还重写， 我都不知道它在干什么

zylthinking

原帖由 flw 于 2008-7-11 14:30 发表
很多人改代码从来不改上下文的。
见过很多这样的。

通盘考虑是好事，不过碰到烂代码的时候，往往牵扯太大太深远，因此形势有时候不允许。

一言以蔽之：在你之前最后一次修改那段代码的人，不见得就是烂人 ...

恩， 这是有道理的， 不过可以肯定， 在这个传递链中， 肯定有一个烂人

flw

原帖由 zylthinking 于 2008-7-11 14:34 发表

恩， 这是有道理的， 不过可以肯定， 在这个传递链中， 肯定有一个烂人

也许那个烂人就是领导。

aero

原帖由 zylthinking 于 2008-7-11 13:42 发表
cur_finish 是本地变量， return之前设置它没有任何意义。
已经 long pagesize=8192; 了
所以 if( pagesize > 8192 ) pagesize = 8192; 纯粹扯淡
更稀奇的居然还有 pagesize = pagesize; 这不是有毛病么

很正常，维护过程中留下的。

zylthinking

我领导不至于， 人家写文件系统什么的， 不至于混到写这些东西。

[ 本帖最后由 zylthinking 于 2008-7-14 15:25 编辑 ]

aero

原帖由 zylthinking 于 2008-7-11 13:46 发表
几行代码阿， 出这么多问题， 要是单单这一个函数也就罢了， 现在妈的3万7千多行， 差不多全是这鸟样子， 现在看得想吐。

呵呵，几十万，上百万行的代码，都这个样子，不还是维护过来了？

要站在老板的角度考虑问题。他要的是work，不是好看，用户也看不到，呵呵。只要能work，能满足客户，就是高质量的。