对于项目中风格不好的某段代码的重构,附gigix的点评
按照gigix回帖,我把原帖子第五个java代码给重构了。
我对重构理解的也许不太深入,因为周围的朋友都基本不用,而平时的工作中
也很难修改别人的代码。所以我的做法肯定有很多错误,请大家多多指教!
把gigix的精彩点评放前面。 因为目前只知道他有过大型项目的重构经验。
他说的话我认为相当有分量。
现在可以一眼就看出,程序在 if 那里修改了持久层。于是把它抽取出来,得到:
public void xxxx(参数1,参数2,参数3,参数4){ try{ var1= ... var2= ... var3=... System.out... for(...) { ..... //多行 System.out.... if.... } Map .... for(...) { var4=... if(){ if(){ ... (多行) System.out.... if(){ if(){ }esle{ } }else{ ... } } } } }catch(Exception e){ e.printStackTrace(); } .....}
1 楼 lighter 2008-12-17 人才啊,非常不错的小伙子