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

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

评论

89 楼 cxshun 2011-05-27  

这个杯具啊，不过才2000多行，我们公司的一般都上4K行的，看着那个蛋疼啊，if else一大堆。

88 楼 tang_kun_cool 2011-05-27  

andy_ghg 写道

同楼主，我以前曾经吹过牛：“你们这个系统，我能砍掉五分之二的前端代码和五分之一的后台代码”。当然我是有根据的也不是乱吹牛。

我遇到的极品项目，让我整整蛋疼了一年，到现在都心有余悸，现在那个项目找我帮忙，我头可断，血可流，代码我坚决不动。

现在的程序就是没有设计好，乱写，完全脱离设计模式，耦合度太高，完全不能脱离出来，这样在删除或修改某段代码的时候，就会有问题！问题归根就是代码编写的水平。

87 楼 phoenixup 2011-05-27  

xzj127 写道

lgsun592 写道

半人马 写道

supertaxi 写道

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

“规定”这个的人，一定不知道这世上还有VCS这种东西存在。

做日本某大型保险公司的项目的时候，客户也是那么要求的，只加不删，你想的太简单了
PS:他们的确不用cvs那烂东西，用svn

  我非常的迷惑  我用过CVS 和SVN ....那个VCS 还真是第一次听说...LS LS  真没打错？

VCS=VSS+CVS+SVN的简写~~~不过现在还是觉得GIT更爽快。。。

86 楼 dajian 2011-05-27  

caoyangx 写道

常见到其他国内框架代码数量优化如下
for(..){
   abc;
}
变成for(..)abc;
if(..){
   abc;
}
变成if(..)abc;
还真别说，能减少一半以上行数的代码。有意义吗？

我经历过的二个公司，这种优化要被批死~~

85 楼 wj539h 2011-05-27  

真不多，我们这儿通常是类里的一个方法2600多行

84 楼 ariestiger 2011-05-27  

老实说， 除开各种开源类库的源码， 这咱上万行的代码还真没看到， 不过我之前一个同事做的事， 一个类有一千多行是有的， 而且也就那么几个方法， 也就是说一个方法大概在两三百行， 奶奶的， 看得我， 那叫一个郁闷。 基本上我觉得， 类文件有多大有多少行倒不是太大的问题， 主要是那个方法啊， 不能太长了， 超过两百行基本上就会造成其他的人阅读上的郁闷了。

83 楼 openFox 2011-05-27  

bugu1986 写道

gch_h 写道

不了解真相，不方便评论
不过我自己写的代码通常都是删掉50%依然还能运行正常业务。其实业务代码并不多，绝大部分（超过三分之二）的代码都是数据校验或者出于保证程序的稳定性、安全性、效率及其他原因写出来的。这些代码都可以删掉。不过这样一来你就得假设你的用户都是小心谨慎的业务，并且没有人蓄意或者无意破坏到你的系统。结果这样的系统也能验收通过的话，只能说明你们公关能力强而不是技术多好。

其实代码不是写的短就表示你牛B

短不是这个意思，是类和方法短。

LS LS说的对，所以都说对于开发人员来说，用户和测试人员是邪恶的。

82 楼 openFox 2011-05-27  

lerous 写道

doylecnn 写道

xzj127 写道

lgsun592 写道

半人马 写道

supertaxi 写道

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

“规定”这个的人，一定不知道这世上还有VCS这种东西存在。

做日本某大型保险公司的项目的时候，客户也是那么要求的，只加不删，你想的太简单了
PS:他们的确不用cvs那烂东西，用svn

  我非常的迷惑  我用过CVS 和SVN ....那个VCS 还真是第一次听说...LS LS  真没打错？

VCS就是CVS,SVN,GIT,HG等这类东西的名字呀

缩写的话，那VCS应该是VSS,CVS,SVN~

了解了，用过svn

81 楼 openFox 2011-05-27  

半人马 写道

supertaxi 写道

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

“规定”这个的人，一定不知道这世上还有VCS这种东西存在。

额，CVS吧

80 楼 sunney2010 2011-05-27  

任何傻瓜都能写出计算机可以理解的代码。好的程序员能写出人能读懂的代码!

79 楼 beyondsphinx 2011-05-27  

supertaxi 写道

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

哈哈，我们有个做报表的8千多行，我老觉得太大了，没想到你们来个3W的 折服了。

78 楼 alyouge 2011-05-27  

supertaxi 写道

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

+1

77 楼 qianhd 2011-05-26  

kulinglei 写道

以前接手一个同事的模块开发，我刚开始随便点了一个类，就被吓了一跳，后来这个模块还是那个人开发了。就因为我看到一个for循环1500行。

烂代码的多少 跟项目参与人数成正比
一些知名的外企同样如此

76 楼 kulinglei 2011-05-26  

以前接手一个同事的模块开发，我刚开始随便点了一个类，就被吓了一跳，后来这个模块还是那个人开发了。就因为我看到一个for循环1500行。

75 楼 heyongyin 2011-05-26  

哥去年负责以前的一个模块，前台一个类13000行左右的代码，大概10多个类文件。哥疯了。
今年哥重写了。发现代码之复杂，抽象出了30多个类文件，每一个就几百行了。

真的是：很多问题不是程序员的问题，是设计上，或者架构上有问题。

74 楼 blackbat 2011-05-26  

一个方法几千行 看的你想吐血 慢慢重构吧...

73 楼 bugu1986 2011-05-26  

fengzw1211 写道

szcs10138456 写道

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

  

表示很赞同。重要的是后面 

72 楼 bugu1986 2011-05-26  

gch_h 写道

不了解真相，不方便评论
不过我自己写的代码通常都是删掉50%依然还能运行正常业务。其实业务代码并不多，绝大部分（超过三分之二）的代码都是数据校验或者出于保证程序的稳定性、安全性、效率及其他原因写出来的。这些代码都可以删掉。不过这样一来你就得假设你的用户都是小心谨慎的业务，并且没有人蓄意或者无意破坏到你的系统。结果这样的系统也能验收通过的话，只能说明你们公关能力强而不是技术多好。

其实代码不是写的短就表示你牛B

短不是这个意思，是类和方法短。

71 楼 bugu1986 2011-05-26  

xzj127 写道

lgsun592 写道

半人马 写道

supertaxi 写道

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

“规定”这个的人，一定不知道这世上还有VCS这种东西存在。

做日本某大型保险公司的项目的时候，客户也是那么要求的，只加不删，你想的太简单了
PS:他们的确不用cvs那烂东西，用svn

  我非常的迷惑  我用过CVS 和SVN ....那个VCS 还真是第一次听说...LS LS  真没打错？

VersionsControlSystem .......

70 楼 Asdpboy 2011-05-26  

只追求怎么快吧项目做完，拿钱，不重视技术、代码质量全是浮云，review code怎么能少呢？