吐槽:维护别人的代码真蛋疼!

2013-06-06 14:29:04 +08:00
 coetzee
今天修改一个同事写的java代码,一个方法整整八百行,当时真想把《重构》甩他头上!
6267 次点击
所在节点    程序员
37 条回复
kylefeng
2013-06-06 14:37:52 +08:00
重构后:xxxAdapter, xxxFactory, xxxBuilder, xxxManager, xxxHandler, xxxFilter, xxxFacade, xxxComponent 等等,一大坨。
dreampuf
2013-06-06 15:44:12 +08:00
怎么现如今还有人天真的以为一本《重构》就能呼唤醒人性的洁癖?!

起码得九本:《GoF》,《重构》,《代码匠艺》,《代码大全》,《代码简洁之道》,《编码》,《人月》,《人件》,《修炼之道》

啪啪啪!啪啪啪!啪啪啪! 的甩在他的脸上,让他知道,什么叫做码农!
heroicYang
2013-06-06 15:46:22 +08:00
既然都接手维护了,还有什么比重构烂代码更紧急的事情呢,哈哈,祝楼主好运!
saharabear
2013-06-06 15:48:31 +08:00
见过一个牛人,过他手的代码,40%都会变得好起来。
vietor
2013-06-06 15:56:43 +08:00
重构是“纯心情”活,如果真如 @kylefeng 吐槽的那样,还不如不进行结构呢,按着原来的风格改。
重构改的是“功能和扩展性”,不是“风格”。java这种语言最大的问题就是“类”的膨胀,如果将800行代码分出4、5个类来就不合算喽。
Golevka
2013-06-06 16:16:17 +08:00
@dreampuf 虽但厉
vvii
2013-06-06 16:19:13 +08:00
@saharabear 60%的隐喻。
acpp
2013-06-06 17:36:39 +08:00
反之,别人维护你的也是一样。
akira
2013-06-06 19:21:15 +08:00
今天修改一个同事写的java代码,一个方法能实现的东西,整了八个类出来,当时真想把《GoF》,《重构》,《代码匠艺》,《代码大全》,《代码简洁之道》,《编码》,《人月》,《人件》,《修炼之道》甩他头上!

别人看你代码会这样吐槽不。。。
brando
2013-06-06 19:27:50 +08:00
不要抱怨,默默的改進...加油!
tyzc
2013-06-06 20:32:17 +08:00
@saharabear 必须对业务觉得了解才能修改,要不然就Bug了!!
banxi1988
2013-06-06 21:09:40 +08:00
@kylefeng
@akira
重构最多的是extract method。
method name as a direct comments。
哪里像你们说的就一定要extract 这么多类来,
而且如果像你们说的搞这么多类的话,肯定是设计模式患者了。
800多行的代码,要么是逻辑真的太复杂了。要不就是。太懒了。
一直写,写啊写。吃完饭回来,写啊写。
一不小心800多行了。下班了。......,然后.....
wzxjohn
2013-06-06 21:20:42 +08:00
呵呵
liwei
2013-06-06 21:29:31 +08:00
彼此彼此
judasnow
2013-06-06 21:46:58 +08:00
他的变量名全是用的拼音缩写而且没有文档吗(如 XX XQ CSRQ XGID XJJ ..)?
没有的话 他对你已经很仁慈了。。。
saharabear
2013-06-06 23:15:02 +08:00
@tyzc 靠单元测试。
saharabear
2013-06-06 23:15:22 +08:00
@vvii 我很佩服能改10%以上的人。
kk71
2013-06-07 08:55:21 +08:00
pythoner飘过……java的类膨胀太可怕了
aurorawu
2013-06-07 09:00:54 +08:00
跟楼主握个爪,这两天我也是修改之前项目别人写的Java代码要疯掉了
xiluo
2013-06-07 09:04:03 +08:00
不要为事情的开始而兴奋,不要为事情的结束而懊恼,不要为生活的开始而疯狂,不要为旅程的结束而伤感,一切随风而来,随风而去,自然自由自在,如云如海如山。

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/71460

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX