模型调用封装类的代码审查

按「正确性 / 资源 / 安全 / 可维护」四个维度审查一个模型调用封装类,每个问题都要给出改法。

更新于 4天前

适用场景

  • 团队里有人写了个模型调用工具类,要合并进主干前先过一遍
  • 接手别人封装的 SDK,判断能不能直接用
  • 要给新人示范一次结构化的代码审查怎么写
PROMPT
团队要把一个同事写的 DeepSeek 调用封装类合进主干,需要一份能直接贴到 PR 评论里的审查意见时用这个。

【需求】
审查下面这个类的代码(我会贴出来),按四个维度逐条给意见:
1. 正确性:参数默认值、异常分支、返回值是否可能为 null、流式与非流式是否混用;
2. 资源:HTTP 连接是否复用、超时是否设置、流式响应体有没有关闭;
3. 安全:API Key 是否硬编码、日志里会不会打印完整 prompt、用户输入有没有直接拼进系统提示词;
4. 可维护:配置是否外部化、是否便于换模型、单测能不能覆盖。

每条意见按「问题 / 影响 / 改法(给代码)」三行写,不要只指出问题不给改法。

代码要点:一个 LlmClient 类,构造函数里 new 了 OkHttpClient,send() 方法没设 readTimeout,catch 了 Exception 只打了 printStackTrace,apiKey 从常量读,日志里打印了完整 messages。

【输出要求】
1. 按四个维度分节,每节下面用编号列表;
2. 每个问题必须给一段可直接替换的改后代码;
3. 最后给一个「阻断合并 / 可后续改」的结论清单,标明哪几条必须这次改完;
4. 不要写客套话,直接给意见。

---
标签
代码审查 封装 规范