job51-gitlab-cr-node-jt-1 2.3.1 → 2.3.2
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
|
@@ -96,6 +96,14 @@
|
|
|
96
96
|
> - 基本类型（int, boolean）：不可能为 null
|
|
97
97
|
> - 明确返回空集合的方法（如 `return Collections.emptyList()`）
|
|
98
98
|
> - 有 `@NotNull` 注解或 `Objects.requireNonNull()` 保护的方法
|
|
99
|
+
> - **早期返回模式**：
|
|
100
|
+
> - 方法开头已有判空/边界检查并提前返回的代码，后续逻辑**不应报告**空指针问题
|
|
101
|
+
> - 示例：`if (query == null || list.isEmpty()) return emptyList;` 后续有正常业务逻辑，**不应报告**问题
|
|
102
|
+
> - **禁止报告"代码简化建议"类问题**：
|
|
103
|
+
> - **禁止**建议"移除变量声明，直接 return xxx.stream()..."
|
|
104
|
+
> - **禁止**建议"合并变量声明和返回语句"
|
|
105
|
+
> - **禁止**对正常的早期返回模式（Early Return）提出简化建议
|
|
106
|
+
> - 这类建议属于代码风格偏好，不是真正的缺陷
|
|
99
107
|
|
|
100
108
|
1. **仅输出一份代码审查报告**，以 `<REPORT>` 开始，`</REPORT>` 结束的 Markdown 格式生成；
|
|
101
109
|
2. **不得输出任何解释、问候、总结或额外文本**；
|
|
@@ -242,3 +250,8 @@
|
|
|
242
250
|
> - [ ] **已确认建议代码与项目原有风格一致**（参数校验方式、异常处理方式、注解使用习惯等）
|
|
243
251
|
> - [ ] **已执行跨块去重**：同一文件同一根本原因的问题只报告一次
|
|
244
252
|
> - [ ] **已验证问题位置**：问题报告在真正会执行出错的行上，而非 package/import/类定义/方法签名等位置
|
|
253
|
+
> - [ ] **已确认报告不自相矛盾**：
|
|
254
|
+
> - 如果代码已有 `if (obj != null)` 判空检查，**禁止**报告"未判空"问题
|
|
255
|
+
> - 如果分析结论包含"当前代码逻辑正确"、"无需修改"、"已有 XX 保护"等描述，**禁止**将该问题报告为严重问题
|
|
256
|
+
> - **报告前自检**：问题描述、错误代码、修改建议三者逻辑必须一致，不得出现"有判空但说未判空"的矛盾
|
|
257
|
+
> - **建议写法**：如果确实想提醒注意某行为，但代码已正确处理，应省略该问题，不输出报告
|