重构没有什么出处,是我平时经常使用而总结出来的。欢迎您发表任何改进意见或建议。我相信一定还有其他比较好的重构可以解决类似的问题。
我曾无数次面对的一个代码坏味道就是,使用异常来控制程序流程。您可能会看到类似的代码:
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 setterpublic Boolean start(Object food) { if (Motor.IsInUse) return false;Motor.Cook(food); return true;}}
时间: 2024-10-25 05:09:59