适用场景
- 团队里有人写了个模型调用工具类,要合并进主干前先过一遍
- 接手别人封装的 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. 不要写客套话,直接给意见。 ---