通过调试的重构代码(设计是对的,但是实现过程却有诸多的问题)
#!/usr/bin/env python class Processor: def __init__(self, processor): self.processor = processor def isAllowSend(self, message, setChargeFlag): pass class WhildListProcessor(Processor): def isAllowSend(self, message, setChargeFlag): print "WhildListProcessor.isAllowSend is Running" if message.sender in (1,2): print "WhildListProcessor.isAllowSend return True" return True return self.processor.isAllowSend(message, setChargeFlag) class DateProcessor(Processor): def isAllowSend(self, message, setChargeFlag): print "DateProcessor.isAllowSend is Running" if message.date > "20130101": print "DateProcessor.isAllowSend return False" return False return self.processor.isAllowSend(message, setChargeFlag) class DefaultProcessor(Processor): def isAllowSend(self, message, setChargeFlag): print "DefaultProcessor.isAllowSend is Running" setChargeFlag(message) print "DefaultProcessor.isAllowSend return True" return True def getProcessor(): return WhildListProcessor(DateProcessor(DefaultProcessor(None))) class Message: def __init__(self, sender): self.sender = sender self.isSend = False self.isCharge = False self.date = "20110201" def setChargeFlag(message): print "Set Charge Flag" message.isCharge = True def setSendFlag(message): print "Set Send Flag" message.isSend = True def run(): message = Message(10) processor = getProcessor() if (processor.isAllowSend(message, setChargeFlag)): setSendFlag(message) run()
这是前一篇博客里的代码,通过实际调试之后的代码,下列的体会:
1, 设计是对的
2, 我在实现设计的过程里却有bug产生
2,1 WhildListProcessor 等类的isAllowSend方法没有return
2,2 DefaultProcessor没有继承我的Processor类
2,3 既然增加了DateProcessor类并检查了Message.date属性,却没在Message类里增加date属性
2,4 出现一个拼写错误 DefaultProcessor 写成 DefaltProcessor
实际上我的多数bug都是这么产生的,是我的性格太粗疏了吗? :( 另外,各位看官,这是一个设计模式,您看出来了吗
【推荐】编程新体验,更懂你的AI,立即体验豆包MarsCode编程助手
【推荐】凌霞软件回馈社区,博客园 & 1Panel & Halo 联合会员上线
【推荐】抖音旗下AI助手豆包,你的智能百科全书,全免费不限次数
【推荐】轻量又高性能的 SSH 工具 IShell:AI 加持,快人一步