文中给了一个案例

class FancyLogger:
    """日志类:支持向文件、Redis 、ES 等服务输出日志"""

    _redis_max_length = 1024

    def __init__(self, output_type=OutputType.FILE):
        self.output_type = output_type
        ...

    def log(self, message):
        """打印日志"""
        if self.output_type == OutputType.FILE:
            ...
        elif self.output_type == OutputType.REDIS:
            ...
        elif self.output_type == OutputType.ES:
            ...
        else:
            raise TypeError('output type invalid')

    def pre_process(self, message):
        """预处理日志"""
        # Redis 对日志最大长度有限制,需要进行裁剪
        if self.output_type == OutputType.REDIS:
            return message[:self._redis_max_length]

FancyLogger 类在日志输出类型不同时,需要有不同的行为。因此,我们完全可以为“输出日志”行为建模一个新的类型:LogWriter ,然后把每个类型的不同逻辑封装到各自的 Writer 类中。

class FileWriter:
    def write(self, message):
        ...
        

class RedisWriter:
    max_length = 1024

    def write(self, message):
        message = self._pre_process(message)
        ...

    def _pre_process(self, message):
        """Redis 对日志最大长度有限制,需要进行裁剪"""
        return message[:self.max_length]


class EsWriter:
    def write(self, message):
        ...

基于这些不同的 Writer 类,FancyLogger 可以简化成下面这样:

class FancyLogger:
    """日志类:支持向文件、Redis 、ES 等服务输出日志"""

    def __init__(self, output_writer=None):
        self._writer = output_writer or FileWriter()
        ...

    def log(self, message):
        self._writer.write(message)

文中对这样的写法好处解释为 代码利用多态特性,完全消除了原来的条件判断语句。另外你会发现,新代码的扩展性也远比旧代码好。

但是在我看来, 你要传什么 output_writer 不还是要通过 if 来选择吗, 只是把一个地方的 if 换到了另外一个地方,

扩展性 这个模块看起来确实好了, 但是总感觉和上面一样, 这里提高了, 但是其他地方就要更多 if, TVT, 面向对象还是没有入门

举报· 722 次点击
登录 注册 站外分享
6 条回复  
shinonome 楼主 小成 4 小时前
@piglei 作者大佬能看看我的问题吗
zepc007 初学 3 小时前
emmm, 你可以理解为设计模式
yooomu 小成 3 小时前
抽离出 writer 之后,添加新的 writer 类型不再需要修改原来的类了,只需要编写一个新的 writer 实现就行了。体现了设计模式的开闭原则
zhu327 初学 3 小时前
这里如果有个工厂模式来实例化不同的 logger 的话,从上层看的话就不需要理解不同的 if 之间的差异了,可以理解为一种封装形式,隐藏不同 logger 之前的细节,对外提供统一的用户接口
ajunno 初学 3 小时前
> 你要传什么 output_writer 不还是要通过 if 来选择吗, 只是把一个地方的 if 换到了另外一个地方 这个成立吗?换到哪里了呢?我的观点是,`FancyLogger`内部解耦了类型判断——针对事物的行为建模,而不是对事物本身建模——分支语句是完全消失了,而不是转移(到外部)了。从外部使用者的角度来说,是完全一样的。
newaccount 小成 3 小时前
改之前:FancyLogger 需要知道每一个 Writer 类型,所以必须有 if 判断来根据不同的参数选择不同的具体实现 改之后:FancyLogger 与 Writer 解耦,具体的输出工作由 Writer 来完成,FancyLogger 不知道也不关心具体实现 你所指的 if 判断由 FancyLogger 转移到调用方的这个前提是不存在的,调用方直接实例化一个 Writer 实现类,或者通过工厂方法来获取配置的具体 Writer 对象
返回顶部