如何优化Reddit非官方API的结构设计？

原结构的核心问题 #

1. 继承关系不合理：RedditClient → RedditSession → HTTPClient的继承链违背了"is-a"原则——RedditClient不是一种RedditSession，RedditSession也不是一种HTTPClient，这种强耦合会导致后续扩展困难，比如更换HTTP库时所有子类都要修改。
2. Session传递混乱：组合Profile时手动传递session，和继承得到session的方式混用，既冗余又容易出错，新增子模块（比如Message）还要继续层层传递session，维护成本高。
3. 职责边界模糊：RedditClient既负责账号操作（改密码、邮箱），又初始化业务模块，职责过于混杂，不符合单一职责原则。

优化方案：用「依赖注入+组合」替代「继承+零散session传递」 #

核心思路是：让HTTPClient成为唯一的请求入口，统一管理session，所有业务模块（认证、个人资料、消息等）都依赖HTTPClient而非直接操作session，入口类RedditClient负责组装所有模块。

1. 重构HTTPClient：统一管理请求与session #

import requests
from requests import Session

class HTTPClient:
    BASE_URL = "https://reddit.com/api"  # 中心化配置API前缀

    def __init__(self, session: Session = None):
        self.session = session or Session()

    def request(self, method: str, endpoint: str, **kwargs):
        full_url = f"{self.BASE_URL}{endpoint}"
        resp = self.session.request(method, full_url, **kwargs)
        if resp.ok:
            return resp
        # 自定义异常，替代模糊的Exception
        raise RedditRequestError(f"请求失败 {resp.status_code}: {resp.text}")

# 自定义异常类，方便调用方精准捕获
class RedditRequestError(Exception):
    pass

2. 拆分认证逻辑：独立成RedditAuth服务 #

把登录、认证逻辑从继承链中抽出来，作为依赖HTTPClient的独立服务：

class RedditAuth:
    def __init__(self, http_client: HTTPClient, username: str, password: str):
        self.http_client = http_client
        self.username = username
        self.password = password
        self._login()

    def _login(self):
        # 实现Reddit登录逻辑，session由HTTPClient自动维护
        login_data = {
            "username": self.username,
            "password": self.password
        }
        self.http_client.request("POST", "/login", data=login_data)
        # 处理认证后的token/cookie等，直接存在http_client.session中即可

3. 业务模块分层：Profile、Message等依赖HTTPClient #

业务模块之间可以嵌套组合，但统一依赖HTTPClient，无需手动传递session：

class MessageService:
    def __init__(self, http_client: HTTPClient):
        self.http_client = http_client

    def delete(self, message_id: str):
        self.http_client.request("DELETE", f"/message/{message_id}")

class ProfileService:
    def __init__(self, http_client: HTTPClient):
        self.http_client = http_client
        # 直接嵌套MessageService，复用同一个HTTPClient
        self.message = MessageService(http_client)

    def get_profile_info(self):
        return self.http_client.request("GET", "/profile/me").json()

4. 入口类RedditClient：组装所有服务 #

RedditClient作为对外入口，负责初始化所有依赖，提供统一的调用入口：

class RedditClient:
    def __init__(self, username: str, password: str, session: Session = None):
        self.http_client = HTTPClient(session)
        self.auth = RedditAuth(self.http_client, username, password)
        # 可以用懒加载优化，第一次访问时才初始化ProfileService
        self._profile = None

    @property
    def profile(self):
        if not self._profile:
            self._profile = ProfileService(self.http_client)
        return self._profile

    # 账号管理方法，直接复用HTTPClient
    def get_account_info(self):
        return self.http_client.request("GET", "/account/me").json()

    def change_password(self, new_password: str):
        self.http_client.request(
            "POST", 
            "/account/change-password", 
            data={"new_password": new_password}
        )

    def change_email(self, new_email: str):
        self.http_client.request(
            "POST", 
            "/account/change-email", 
            data={"new_email": new_email}
        )

调用示例 #

现在你可以轻松实现想要的链式调用：

client = RedditClient("your_username", "your_password")
# 删除指定消息
client.profile.message.delete("msg_12345")
# 获取个人资料
profile_info = client.profile.get_profile_info()

额外优化建议 #

• 配置分离：把API前缀、超时时间、请求头等配置抽成单独的配置类或字典，方便统一修改。
• 缓存策略：对频繁调用的接口（比如个人资料）添加缓存，减少重复请求。
• 异步支持：如果需要高并发，可以基于aiohttp实现异步版的HTTPClient，业务模块复用同样的依赖结构。

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