重构没有什么出处,是我平时经常使用而总结出来的。欢迎您发表任何改进意见或建议。我相信一定还有其他比较好的重构可以解决类似的问题。
我曾无数次面对的一个代码坏味道就是,使用异常来控制程序流程。您可能会看到类似的代码:
public class Microwave {
private MicrowaveMotor Motor;//getter setter
public Boolean start(Object food) {
Boolean foodCooked = false;
try {
Motor.Cook(food);
foodCooked = true;
} catch (InUseException e) {
foodcooked = false;
}
return foodCooked;
}
}
异常应该仅仅完成自己的本职工作:处理异常行为。大多数情况你都可以将这些代码用恰当的条件判断替换,并进行恰当的处理。下面的代码可以称之为契约式设计,因为我们在执行具体工作之前明确了Motor类的状态,而不是通过异常来进行处理。
public class Microwave {
private MicrowaveMotor Motor;//getter setter
public Boolean start(Object food) {
if (Motor.IsInUse) return false;
Motor.Cook(food);
return true;
}
}