锁定老帖子 主题:17486行的java 类文件
精华帖 (1) :: 良好帖 (0) :: 新手帖 (0) :: 隐藏帖 (1)
|
|
---|---|
作者 | 正文 |
发表时间:2011-08-31
最后修改:2011-08-31
真的很长!这样长的代码一定要加上注释!!
重构不是一句话得事! 要考虑的很多,出了BUG怎么办,时间怎么办! |
|
返回顶楼 | |
发表时间:2011-08-31
最后修改:2011-08-31
quxiaoyong 写道 为什么参数不是List<?> list,而是ArrayList list???
你用JDK1.3 compile试试.. 此外为什么有许多硬编码, 因为早期JDK许多东西都没, 后人在其基础上扩展功能就需要自己写了.. |
|
返回顶楼 | |
发表时间:2011-08-31
private String getBooleanHas(ArrayList list,String barcode){ String has="N"; if(list!=null&&list.size()>0){ for(int i=0;i<list.size();i++){ Object[]obj=(Object[])list.get(i); if(obj[0].toString().trim().equals(barcode.trim())){ has="Y"; break; } } } return has; } return ""+CollectionUtils.exists(list,new Predicate(){ public evaluate(java.lang.Object object) { return ((Object[])object)[0].trim().equal(barcode.trim()); //这里有个魔术数字需要重新设计接口 } }); |
|
返回顶楼 | |
发表时间:2011-08-31
哥们,你是做用友二开的吧,是说咋看着这么眼熟呢。
|
|
返回顶楼 | |
发表时间:2011-08-31
optimism_best 写道 哥们,你是做用友二开的吧,是说咋看着这么眼熟呢。
不是吧,用友的类就这水平? 说实话“时间长,难重构,有风险”可不是借口,唯一的借口是懒惰。 这类难道你们打算一直维护下去么?如果是,那么对后来人来说,绝对是个悲剧。 我接手的项目,无论代码有多长,业务多么重要,如果类设计得很烂,逻辑混乱,代码没有规范注释,命名不规范,我都会当机立断:重构,如果领导不让重构,那好,您找找别人来负责这堆烂番茄吧,我要回家种土豆了。 |
|
返回顶楼 | |
发表时间:2011-08-31
george_space 写道 optimism_best 写道 哥们,你是做用友二开的吧,是说咋看着这么眼熟呢。
不是吧,用友的类就这水平? 说实话“时间长,难重构,有风险”可不是借口,唯一的借口是懒惰。 这类难道你们打算一直维护下去么?如果是,那么对后来人来说,绝对是个悲剧。 我接手的项目,无论代码有多长,业务多么重要,如果类设计得很烂,逻辑混乱,代码没有规范注释,命名不规范,我都会当机立断:重构,如果领导不让重构,那好,您找找别人来负责这堆烂番茄吧,我要回家种土豆了。 07年以前因为这种事,回家种过几次土豆,后来得知,那些烂番茄最后不是被重构了,而是直接被扔到历史的垃圾堆里了,直接重做了。 |
|
返回顶楼 | |
发表时间:2011-08-31
这么长的代码看着头疼,IDE工具也很受伤
|
|
返回顶楼 | |
发表时间:2011-09-01
optimism_best 犀利
用友二开 |
|
返回顶楼 | |
发表时间:2011-09-01
我有找到一个更犀利的2w过行的
从2001年一直在扩展写到现在 http://demojava.iteye.com/blog/1160562 知道一个方法知道能写多长吗? 这个我4过一个方法只能写65535个字节 详见 http://demojava.iteye.com/blog/1160562 |
|
返回顶楼 | |
发表时间:2011-09-01
quxiaoyong 写道 为什么参数不是List<?> list,而是ArrayList list???
2001年的JDK,不支持List<?> list,这是JDK5.0以后的事了. |
|
返回顶楼 | |