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 3
怎么现如今还有人天真的以为一本《重构》就能呼唤醒人性的洁癖?!
起码得九本:《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 1
不要为事情的开始而兴奋,不要为事情的结束而懊恼,不要为生活的开始而疯狂,不要为旅程的结束而伤感,一切随风而来,随风而去,自然自由自在,如云如海如山。
|
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
哥,先不要考虑太多,先保证系统正常运行。。别改崩了。
|