通过一个简单的案例,来谈谈代码的重构

上伪代码:

func a(){
    ...
    order = ***;
    payOrder = new v1{orderNo=order.orderNo, amt=order.amt, remark='结算资金下发'}; //通过order得到v1;
    pay(payOrder)    ;
    ...
}

func b(){
    ...
    order = ***;
    payOrder = new v1{orderNo=order.orderNo, amt=order.amt, remark='结算资金下发'}; //通过order得到v1;
    pay(payOrder)    ;
    ...
}

func pay(payOrder){
    ...
}

 

有同学一眼就看出了问题——重复代码。这同学给的重构方案如下。

func a(){
    ...
    order = ***;
    foo(order);
    ...
}

func b(){
    ...
    order = ***;
    foo(order);
    ...
}

func foo(order){
    payOrder = new v1{orderNo=order.orderNo, amt=order.amt, remark='结算资金下发'}; //通过order得到v1;
    pay(payOrder)    ;
}

func pay(payOrder){
    ...
}

 

 

 

如果是你,当如何进行重构?

 

 

 

 

没错,这段程序存在的问题是“存在重复代码”。我们在重构时,应以简单、清晰、易读、已维护为宗旨。能充分理解这一点,也并不容易,需要功力。

就像上面同学的重构方案那样,引入新的方法 foo,毋庸置疑达到了“消除重复”的目的,不过,这在一定程度上,增加了程序的复杂性。

许多同学,包括你,必然明白,下面是正确姿势。

本案是一个非常简单的程序重构案例。实际企业应用开发过程中,我们遇到的场景远比这个要复杂。以此为例,希望大家加强对代码重构的理解,从复杂的代码分析出条理,发现存在的问题,并进行更有效的重构。

func a(){
    ...
    order = ***;
    pay(order);
    ...
}

func b(){
    ...
    order = ***;
    pay(order);
    ...
}

func pay(order){
    payOrder = new v1{orderNo=order.orderNo, amt=order.amt, remark='结算资金下发'}; //通过order得到v1;
    pay(payOrder)    ;
    ...
}

 

热门相关:农家小福女   暧昧的话   我家老公超宠哒   整你的人生   九重神格