一个2600行代码的类，干掉1100行，程序照常运行

如题，这是什么概念，至于其他的类也是一个鸟样，我不想去多改了。
很无奈，如果要去优化，还不如重新开发。你们有碰到这样“优秀”的程序。唉

评论

129 楼 getclass 2011-06-11  

痛恨写这些"异味"代码的人.这样的人在我手下,就直接开除.没商量

128 楼 NeroBest 2011-06-10  

一大堆人写一个类。。。代码行数爆炸

127 楼 william_ai 2011-06-02  

没有svn，cvs，vss之类的版本控制么

126 楼 zhongrenmeng 2011-06-02  

supertaxi 写道

我们公司一个类文件3W多行代码, 并且规定修改代码时,不能删除旧的代码,只能注释掉. 于是,一个注释直接注释一千多行...

我们公司也有很多人有这个习惯， 不过没还出现杯具到3w多行的情况

125 楼 mdsp25xhm 2011-06-02  

supertaxi 写道

我们公司一个类文件3W多行代码, 并且规定修改代码时,不能删除旧的代码,只能注释掉. 于是,一个注释直接注释一千多行...

同感

124 楼 wubaodong 2011-06-02  

sirxenofex 写道

wubaodong 写道

我写过的一个方法，将近1000行。功能很简单，逻辑单一，输出一个PDF文件。类似你直接写一个servlet输出一个HTML文件一样。1000行是因为PDF文件内容多，如果内容需要增加，我还会继续增加方法的行数。你们说这样的方法，有必要拆分吗？

非常有必要。就算你的1000行所有代码都不能重用，也要按步骤拆成很多个小函数。运行上没有任何区别，原因只有一个：给人看

如果只是从给人看的角度考虑，那我觉得不拆分跟拆分没有优劣之分

    System.out.println("我爱北京敏感词");
    System.out.println("敏感词上太阳升");
    System.out.println("伟大领袖敏感词");
    System.out.println("指引我们向前进");
    System.out.println("啦啦啦啦啦啦啦");

类似这样罗列的输出一篇文章，拆不拆分有什么区别，每一个自然段都拆分成一个小方法吗？

123 楼 from_null 2011-05-31  

zuzong 写道

赶工、加班、工资低、保险差、混乱的项目管理、蛋疼的经理、你说你还想要优雅的代码。。。。

 

122 楼 wangchong0114 2011-05-31  

其实这个感觉挺正常的
你无法保证项目组内的人的技术水平跟你一样
也无法保证项目组内的人跟你一个思路实现功能
关于重构这一块
确实是一个很大的学问
代码是由少变多再变少的一个过程

如果有人拿到你的代码再给你的代码删去一半  功能还是会实现
能说你写的代码不好么？

121 楼 sunofsummer 2011-05-31  

谨慎重构。

120 楼 crazygoal 2011-05-30  

对于维护系统来说一半都会选择重构，而不是重写。重写风险太大了，因为旧系统可以已经修正过很多BUG，直接重写的话有可能不能继承这些修复。我碰到过一个项目(C语言开发的)，里面有大量的重复代码，超过7层的if语句，超过2000行的函数等，就是看了就崩溃的那种。那时我也很想很想重写，但是考虑到现有版本经过测试人员的反复测试，客户使用情况也较好，但就是新增功能非常困难，定位和修改BUG也非常困难。最终我们对系统进行了很多轮的分析，并进行重构(功能不变 内部结构变化)。代码从38K降至18K，验收测试和后续版本基本没有BUG产生。

结论
1.重构而非重写
2.重构前作充分准备，现有系统的多轮分析，人力/时间的准备。
3.重构最好进行拆分，不要想一次搞定，迭代进行。

119 楼 jnn 2011-05-30  

szcs10138456 写道

项目就是在不断地改需求和不断地fix和Refactoring下成长的

严重同意，
我接触过的几个项目最起码都大规模的重写过两遍，同时Refactoring无数次。
采用TDD开发的最大好处就是不怕Refactoring，改完了之后跑完测试都绿了就可以收工睡觉了。

118 楼 moremoney999 2011-05-30  

有时间就拆分了，还是没时间啊，拆分了还要测试，这时候可能已经上线了或者开始下一版开发了

117 楼 nowaytj 2011-05-30  

首先不能单纯看行数论好坏
然后呢这么多行的类肯定是不应该
什么事都得有个规范，有个度
程序写成这样原因只有一个
一定是有不负责任的人混饭吃

116 楼 sirxenofex 2011-05-30  

wubaodong 写道

我写过的一个方法，将近1000行。功能很简单，逻辑单一，输出一个PDF文件。类似你直接写一个servlet输出一个HTML文件一样。1000行是因为PDF文件内容多，如果内容需要增加，我还会继续增加方法的行数。你们说这样的方法，有必要拆分吗？

非常有必要。就算你的1000行所有代码都不能重用，也要按步骤拆成很多个小函数。运行上没有任何区别，原因只有一个：给人看

115 楼 vb2005xu 2011-05-30  

你不错了 我之前遇到过 1个 方法 1700行啊 更无语 啊

114 楼 zui4yi1 2011-05-30  

流程里不要添加太多的实现代码，基本上就OK

113 楼 ansjsun 2011-05-29  

BruceXX 写道

那你们看到过一个jsp页面组成一个项目的吗？？哈哈。。。

文件管理器算不算..也可以 说成jsp木马..就一个jsp文件哦

112 楼 guaziaim9x 2011-05-29  

千万别把自己看得太牛逼，有可能捞的程序员考虑的更细致的东西

111 楼 okbaozhen 2011-05-28  

refactoring

110 楼 orcl_zhang 2011-05-28  

nakupanda 写道

到有一天发现别人臃肿的代码原来是考虑到一些自己没有考虑到的情况的时候, 不知道楼上的一些朋友会不会泪流满脸.

这是经常遇到的，有次精简代码，把8000行改成6000行，之后发现错了，又把6000行改成10000行。。