![]() | 1 kylefeng 2013-06-06 14:37:52 +08:00 重构后:xxxAdapter, xxxFactory, xxxBuilder, xxxManager, xxxHandler, xxxFilter, xxxFacade, xxxComponent 等等,一大坨。 |
![]() | 2 dreampuf 2013-06-06 15:44:12 +08:00 ![]() 怎么现如今还有人天真的以为一本《重构》就能呼唤醒人性的洁癖?! 起码得九本:《GoF》,《重构》,《代码匠艺》,《代码大全》,《代码简洁之道》,《编码》,《人月》,《人件》,《修炼之道》 啪啪啪!啪啪啪!啪啪啪! 的甩在他的脸上,让他知道,什么叫做码农! |
![]() | 3 heroicYang 2013-06-06 15:46:22 +08:00 既然都接手维护了,还有什么比重构烂代码更紧急的事情呢,哈哈,祝楼主好运! |
![]() | 4 saharabear 2013-06-06 15:48:31 +08:00 见过一个牛人,过他手的代码,40%都会变得好起来。 |
![]() | 5 vietor 2013-06-06 15:56:43 +08:00 重构是“纯心情”活,如果真如 @kylefeng 吐槽的那样,还不如不进行结构呢,按着原来的风格改。 重构改的是“功能和扩展性”,不是“风格”。java这种语言最大的问题就是“类”的膨胀,如果将800行代码分出4、5个类来就不合算喽。 |
7 vvii 2013-06-06 16:19:13 +08:00 @saharabear 60%的隐喻。 |
![]() | 8 acpp 2013-06-06 17:36:39 +08:00 反之,别人维护你的也是一样。 |
![]() | 9 akira 2013-06-06 19:21:15 +08:00 今天修改一个同事写的java代码,一个方法能实现的东西,整了八个类出来,当时真想把《GoF》,《重构》,《代码匠艺》,《代码大全》,《代码简洁之道》,《编码》,《人月》,《人件》,《修炼之道》甩他头上! 别人看你代码会这样吐槽不。。。 |
![]() | 10 brando 2013-06-06 19:27:50 +08:00 不要抱怨,默默的改...加油! |
![]() | 11 tyzc 2013-06-06 20:32:17 +08:00 @saharabear 必须对业务觉得了解才能修改,要不然就Bug了!! |
![]() | 12 banxi1988 2013-06-06 21:09:40 +08:00 |
![]() | 13 wzxjohn 2013-06-06 21:20:42 +08:00 呵呵 |
14 liwei 2013-06-06 21:29:31 +08:00 彼此彼此 |
15 judasnow 2013-06-06 21:46:58 +08:00 他的变量名全是用的拼音缩写而且没有文档吗(如 XX XQ CSRQ XGID XJJ ..)? 没有的话 他对你已经很仁慈了。。。 |
![]() | 16 saharabear 2013-06-06 23:15:02 +08:00 @tyzc 靠单元测试。 |
![]() | 17 saharabear 2013-06-06 23:15:22 +08:00 @vvii 我很佩服能改10%以上的人。 |
![]() | 18 kk71 2013-06-07 08:55:21 +08:00 pythoner飘过……java的类膨胀太可怕了 |
![]() | 19 aurorawu 2013-06-07 09:00:54 +08:00 跟楼主握个爪,这两天我也是修改之前项目别人写的Java代码要疯掉了 |
![]() | 20 xiluo 2013-06-07 09:04:03 +08:00 via Android ![]() 不要为事情的开始而兴奋,不要为事情的结束而懊恼,不要为生活的开始而疯狂,不要为旅程的结束而伤感,一切随风而来,随风而去,自然自由自在,如云如海如山。 |
![]() | 21 RisingV 2013-06-07 09:41:33 +08:00 写点简洁点的代码,真的需要看那么多书吗?不妨自己仔细思考,提取抽象,照样能写得很漂亮。 要知道设计模式不是本来就存在的,也是别人的经验。 按照个人经验来讲,可以概括一下自己的抽象方式,然后和书上一比对,你发现自己重新发明了这些设计模式,你明了这些设计模式真正的适用场景,而不是死搬硬套,除了成就感还得到了很好的修炼。 |
![]() | 22 fishsjoy 2013-06-07 10:26:44 +08:00 |
![]() | 23 ijse 2013-06-07 10:41:13 +08:00 呵呵 ,, 呵呵 。。 |
![]() | 24 zhangdawei 2013-06-07 12:33:28 +08:00 800行就扛不住了? 我在维护的代码,一个function 3000多行,最多11层if/else/switch。 |
25 sdslnmd 2013-06-07 13:19:56 +08:00 维护2K行的方法的微笑走过 |
![]() | 26 likexian 2013-06-07 13:31:30 +08:00 我写了一个函数,3万多行,你们看着办 |
![]() | 28 luikore 2013-06-07 14:25:31 +08:00 换个语言就40行的事没人纠结长度了... |
![]() | 30 achellis 2013-06-07 23:19:36 +08:00 String jsOnData=" { // 800行 }"; XD |
![]() | 36 samuel 2013-06-09 10:51:50 +08:00 via Android 别人的代码永远没有自己的代码看着舒心啊,试着将维护别人代码看作是一种修行,你会更有收获的 |
![]() | 37 stevenhu888 2013-06-09 10:56:38 +08:00 哥,先不要考虑太多,先保证系统正常运行。。别改崩了。 |