通过调试的重构代码(设计是对的,但是实现过程却有诸多的问题)

#!/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都是这么产生的,是我的性格太粗疏了吗? :( 另外,各位看官,这是一个设计模式,您看出来了吗

转载于:https://www.cnblogs.com/code-style/p/3502105.html

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值