上伪代码：
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) ; ... }
本案只是一个非常简单的程序重构案例。实际企业应用开发过程中，我们遇到的场景远比这个要复杂。以此为例，希望大家加强对代码重构的理解，从复杂的代码分析出条理，发现存在的问题，并进行更有效的重构。