Combine 代码审查

快速参考

审查清单

• 所有 sink 闭包在 self 拥有 cancellable 时均使用 [weak self]
• 不使用 assign(to:on:self)（应改用 assign(to: &$property) 或 sink）
• 所有 AnyCancellable 均存储于 Set 或属性中（不可被丢弃）
• Subjects 通过 eraseToAnyPublisher() 暴露为 AnyPublisher
• 正确使用 flatMap（当需 map + switchToLatest 时不应使用 flatMap）
• 在 flatMap 内部进行错误处理，以保持主链持续运行
• tryMap 后接 mapError，以恢复错误类型
• 在 UI 更新前调用 receive(on: DispatchQueue.main)
• 使用 PassthroughSubject 表达事件，CurrentValueSubject 表达状态
• Future 在与 retry 配合使用时需包裹在 Deferred 中

何时查阅参考文档

• 审查 Subjects 或发布者选型 → publishers.md
• 审查操作符链或发布者组合逻辑 → operators.md
• 审查订阅行为或内存问题 → memory.md
• 审查错误处理或 try* 操作符 → error-handling.md

硬性准入条件（提交审查发现前必须满足）

请按顺序完成。前一条件未通过时，不得跳过执行后续条件。

1. 范围确认 — 通过： 至少明确指出一个正在审查的文件或类型，该文件/类型导入了 Combine 或使用了快速参考中所列 API（例如 AnyPublisher、@Published、PassthroughSubject）。若无任何适用项，则终止审查并标注“超出范围”。
2. 订阅持有检查 — 通过： 对范围内每个 sink、assign 和 store(in:) 调用，明确说明其对应的 AnyCancellable 存储位置（属性、Set、任务生命周期内），或标记为 临时性（ephemeral） 并附一行简要理由（例如：同步一次性调用，其生命周期不可能超过调用方）。若仅凭代码片段无法判断，请标注 未知（unknown） 并请求提供上下文中的存储结构，切勿假设安全。
3. 循环引用断言 — 通过： 对已 确认 的内存泄漏问题，须明确列出捕获链（例如：self → 存储的 cancellable → 闭包强引用 self）；对疑似情况应标注为 风险（risk） 或 待验证（verify），而非直接断言为已确认泄漏；若主张某处安全，须注明所依赖的 [weak self]、[unowned self] 或非捕获模式。
4. UI / 主线程调度 — 通过： 若发布者链最终触发 UIKit/SwiftUI 的 UI 更新，则须指出在 UI 工作前存在 receive(on: DispatchQueue.main)、@MainActor 或等效机制；否则，须以 file:line 格式标记缺失调度的问题点。
5. 严重性与清单对应 — 通过： 每一项 高（high） 或 严重（critical） 级别问题，均须包含 file:line（或精确粘贴的源码行）并指明其违反的是哪一条 审查清单 条目；低严重性备注可省略行号，但仍须确保能根据所命名的文件复现问题。

审查问题

1. 所有订阅是否都被正确持有？（检查是否存在被丢弃的 AnyCancellable）
2. 任意 sink 或 assign 是否可能与 self 构成循环引用？
3. 搜索/自动补全类场景中，flatMap 是否应替换为 switchToLatest？
4. 当该发布者失败时会发生什么？（是否会中断主链？）
5. 在使用 try* 操作符后，错误类型是否被保留或经由 mapError 正确映射？