Python多源Message类的类方法映射实现问题与规范问询

一、修复类方法无法访问映射字典的问题 #

核心问题是你把sources定义成了实例属性（self.sources），但类方法属于类本身，不依赖实例，自然无法访问实例属性。只需把sources改成类属性即可解决：

修正后的基础实现 #

class MessageBase:
    # 定义为类属性，类方法可直接通过cls访问
    _sources = {
        'outlook': 'from_outlook',
        'sendinblue': 'from_sendinblue_email'
    }

    @classmethod
    def from_source(cls, source, raw_message):
        # 通过类属性获取对应方法名
        method_name = cls._sources.get(source)
        if not method_name:
            raise ValueError(f"不支持的消息来源: {source}")
        # 获取并调用对应的类方法
        parse_method = getattr(cls, method_name)
        return parse_method(raw_message)

    @classmethod
    def from_outlook(cls, raw_outlook_message):
        # Outlook消息解析逻辑示例
        msg = cls()
        msg.subject = raw_outlook_message.get("subject")
        msg.sender = raw_outlook_message.get("sender")
        return msg

    @classmethod
    def from_sendinblue_email(cls, raw_sib_message):
        # SendinBlue消息解析逻辑示例
        msg = cls()
        msg.subject = raw_sib_message.get("subject")
        msg.sender = raw_sib_message.get("from_email")
        return msg

二、关于Python编码规范的说明 #

你的实现思路完全符合Python规范，属于轻量工厂模式的典型应用，适合多来源实例创建的场景。补充几个优化细节，让代码更贴合PEP8规范：

• 用下划线前缀命名内部属性（如_sources），明确标识这是类的内部实现，不建议外部直接修改。
• 类方法命名采用from_xxx格式，语义清晰，符合Python"可读性优先"的设计原则。
• 加入异常处理（如来源不支持时抛出ValueError），让接口更健壮，避免静默失败。

进阶优化：用装饰器自动注册解析方法 #

如果后续需要频繁新增消息来源，可以用装饰器自动维护映射字典，避免手动修改_sources，更符合开闭原则：

class MessageBase:
    _sources = {}

    @classmethod
    def register_source(cls, source_name):
        """装饰器：注册消息来源对应的解析方法"""
        def decorator(parse_method):
            cls._sources[source_name] = parse_method
            return parse_method
        return decorator

    @classmethod
    def from_source(cls, source, raw_message):
        parse_method = cls._sources.get(source)
        if not parse_method:
            raise ValueError(f"不支持的消息来源: {source}")
        return parse_method(raw_message)

    @register_source('outlook')
    @classmethod
    def from_outlook(cls, raw_outlook_message):
        msg = cls()
        msg.subject = raw_outlook_message.get("subject")
        return msg

    @register_source('sendinblue')
    @classmethod
    def from_sendinblue_email(cls, raw_sib_message):
        msg = cls()
        msg.subject = raw_sib_message.get("subject")
        return msg

内容的提问来源于stack exchange，提问作者Jeroen Vermunt