Cleans AI-generated code slop …

# PR冗余代码清理指南

一、重要前提说明

您提到的 main.ts 中 OrderService 类的具体代码未随任务附上，我无法直接修改。以下提供可直接套用的清理清单与规范，您贴出代码后我可逐行处理。

二、可安全删除的冗余模式清单

1. 多余的防御性判断

• 重复非空校验：函数入口和调用处都对同一参数做了 if (!x) return，保留一层即可
• 类型已保证的判断：TypeScript 严格模式下，已声明类型的参数再判 typeof 属冗余
• 不可能为 false 的分支：如对刚初始化的变量判空
• 对框架/运行时已保证的行为重复校验（如已过中间件鉴权的接口再查登录态）

2. 重复的错误处理

• 同一模块多处 catch 后执行完全相同的日志+抛出逻辑 → 抽成统一的错误处理函数或中间件
• 上层已统一捕获，下层再层层捕获仅做 console.log 的可删
• 错误码/错误消息重复定义 → 收敛到常量或错误枚举

3. 无用的 try-catch 包装

• 纯同步代码包 try-catch 且 catch 后仅原样抛出 → 直接删除
• catch 后 throw e 无任何附加处理 → 删除整个 try-catch
• Promise 链中同时 .catch 又外层 try-catch → 保留一种
• catch 后吞掉错误返回默认值且未记录日志（除非业务明确需要降级）

三、执行步骤建议

1. 跑通测试基线：修改前确认现有测试全部通过

2. 小步提交：每删一类冗余单独 commit，方便 review 与回滚

3. 对照验证：

• 功能逻辑不变（输入输出、副作用一致）
• 日志行为一致（删错误处理时确认日志不丢失）

4. PR 描述中列出删除项：便于评审者快速确认

四、注意事项

• 判断“防御性校验是否多余”需确认调用方，跨边界数据（API 入参、外部服务返回）的校验不建议删
• 涉及金额、支付等订单核心逻辑时，宁可保守保留校验

请贴出 OrderService 的完整代码，我将按上述规则输出清理后的版本及逐项说明。