Java Reviewer Tool Free

SKILL.md

---
name: "java-reviewer-tool-free"
description: "Java 代码变更审查工具，按 6 大维度生成结构化审查报告与修复建议。"
license: Proprietary
allowed-tools: read exec
compatibility: "Requires LLM with tool-use capability"
metadata:
  displayName: "Java代码审查免费版"
  version: "1.0.0"
  summary: "Java 代码变更审查工具，按 6 大维度生成结构化审查报告与修复建议。"
  tags:
    - "开发工具"
    - "Java"
    - "代码审查"
    - "代码质量"
  source: "SkillHub"
  converted_at: "2026-07-22T17:58:36"
---

# Java 代码审查工具（免费版）

## 概述

本工具为 Java 开发者提供代码变更的结构化审查能力，按 6 大维度逐项检查代码质量，标记问题严重程度，并提供可运行的修复代码。通过自然语言指令驱动，帮助开发者在提交前发现潜在缺陷、安全隐患和性能问题。免费版聚焦个人开发者的代码变更审查场景，提供标准化的审查模板与修复建议。

## 核心能力

| 审查维度 | 描述 | 常见问题示例 |
| --- | --- | --- |
| 代码风格与命名 | 命名规范、格式、注释 | 变量名 `int d`、魔法值 `status = 1` |
| 异常处理 | 捕获策略、错误处理 | 空 catch 块、返回 null 而非抛异常 |
| 安全隐患 | 敏感信息、注入风险 | SQL 拼接、硬编码密码 |
| 性能问题 | 循环优化、资源复用 | 循环内字符串拼接、N+1 查询 |
| 设计合理性 | 职责单一、模块化 | 过长函数、重复代码 |
| 资源管理 | 资源释放、内存管理 | 未关闭连接、ThreadLocal 未清理 |
**技术实现要点**：核心能力基于`input_params`参数与`output_format`配置实现,支持创建/查询/修改/删除等操作模式,通过`config_options`进行运行时配置。

### 核心功能执行
用`input_params`参数进行配置。

**输入**: 用户提供核心功能执行所需的指令和必要参数。
**处理**: 按照skill规范执行核心功能执行操作,遵循单一意图原则。
**输出**: 返回核心功能执行的执行结果,包含操作状态和输出数据。
- 执行此能力时使用`input_params`参数,支持创建/查询/导出操作

### 参数配置与调用
用`config_options`参数进行配置。

**输入**: 用户提供参数配置与调用所需的指令和必要参数。
**处理**: 按照skill规范执行参数配置与调用操作,遵循单一意图原则。
**输出**: 返回参数配置与调用的执行结果,包含操作状态和输出数据。
- 执行此能力时使用`config_options`参数,支持修改/重置/导入操作

### 结果处理与输出
用`output_format`参数进行配置。

**输入**: 用户提供结果处理与输出所需的指令和必要参数。
**处理**: 按照skill规范执行结果处理与输出操作,遵循单一意图原则。
**输出**: 返回结果处理与输出的执行结果,包含操作状态和输出数据。
- 执行此能力时使用`output_format`参数,支持导出/保存/转换操作
**能力覆盖范围**：本skill的核心能力覆盖以下场景关键词：Java、代码变更审查工具、大维度生成结构化、审查报告与修复建、开发者的代码审查、自动生成结构化审、查报告、大维度代码审查、级严重程度分级、Critical、Major、Minor、Suggestion、代码修复建议与前、后对比、Markdown、格式审查报告输出等。这些关键词对应description中声明的使用场景,均已在上述能力点中提供对应的操作支持。

## 使用场景

### 场景一：审查 Git Diff 变更

开发者提交前需要审查代码变更。

```text
用户：审查以下代码变更
[粘贴 git diff 输出]

助手：生成审查报告

## 不适用场景

以下场景Java代码审查免费版不适合处理：

- 无明确技术栈的模糊需求
- 纯架构设计决策
- 运维部署管理

## 触发条件

需要代码生成、编程辅助、调试测试、开发部署时使用。不适用于非本工具能力范围的需求。

## 概述
- 审查文件：3 个
- 问题总数：5 个
  - Critical: 1
  - Major: 2
  - Minor: 2

## 问题详情

### 1. [Critical] 未释放数据库连接
文件：UserService.java:45

问题描述：
方法中获取了数据库连接但未确保关闭。

问题代码：
```java
public User findById(Long id) {
    Connection conn = dataSource.getConnection();
    // ...
}
```

修复后代码：
```java
public User findById(Long id) {
    try (Connection conn = dataSource.getConnection()) {
        // ...
    }
}
```
```

### 场景二：审查单个源文件

开发者需要对一个 Java 源文件进行全面审查。

```text
用户：审查 UserService.java 文件

助手：按 6 维度逐项检查

### 代码风格与命名
- [Minor] 第 12 行：变量名 `d` 不清晰，建议改为 `duration`
- [Suggestion] 第 25 行：魔法值 `1` 建议提取为常量

### 异常处理
- [Major] 第 38 行：空 catch 块，应记录日志或重新抛出

### 安全隐患
- [Critical] 第 52 行：SQL 拼接存在注入风险，应使用 PreparedStatement

### 性能问题
- [Major] 第 67 行：循环内字符串拼接，应使用 StringBuilder

### 设计合理性
- [Minor] 第 80 行：方法超过 50 行，建议拆分

### 资源管理
- [Critical] 第 95 行：FileInputStream 未关闭，应使用 try-with-resources
```

### 场景三：生成修复建议

开发者需要针对审查发现的问题获取修复代码。

```java
// 问题代码：SQL 拼接
public User findByUsername(String username) {
    String sql = "SELECT * FROM users WHERE username = '" + username + "'";
    Statement stmt = conn.createStatement();
    ResultSet rs = stmt.executeQuery(sql);
    // ...
}

// 修复后代码：参数化查询
public User findByUsername(String username) {
    String sql = "SELECT * FROM users WHERE username = ?";
    try (PreparedStatement stmt = conn.prepareStatement(sql)) {
        stmt.setString(1, username);
        try (ResultSet rs = stmt.executeQuery()) {
            if (rs.next()) {
                return mapToUser(rs);
            }
        }
    } catch (SQLException e) {
        logger.error("查询用户失败: {}", username, e);
        throw new RuntimeException("查询用户失败", e);
    }
    return null;
}
```

## 快速开始

### 严重程度定义

| 级别 | 说明 | 示例 |
| --- | --- | --- |
| Critical | 必须修复，可能导致崩溃/安全漏洞/数据丢失 | SQL 注入、空指针崩溃、硬编码密码 |
| Major | 强烈建议修复，影响质量/可维护性/性能 | 空 catch 块、N+1 查询、过长函数 |
| Minor | 建议改进，不影响功能但有优化空间 | 魔法值未提取、命名不清晰 |
| Suggestion | 可选优化，用于代码美化或最佳实践 | 添加 Javadoc、提取工具方法 |

### 审查流程

```text
1. 用户提供输入：
   - 必填：git diff 输出或 Java 源文件
   - 可选：需求文档（用于一致性检查）

2. 执行审查：
   - 按 6 个维度逐项检查
   - 标记问题及严重程度

3. 生成报告：
   - 使用标准模板
   - 输出 Markdown 格式
```

### 报告结构

```markdown
## 概述
- 审查文件：N 个
- 问题总数：N 个
  - Critical: N
  - Major: N
  - Minor: N
  - Suggestion: N

## 问题详情
（按 Critical → Major → Minor → Suggestion 排序）

### N. [严重程度] 问题标题
文件：`文件名:行号`

**问题描述**：简短描述

**问题代码**：
```java
// 修复前代码（至少 3 行上下文）
```

**修复后代码**：
```java
// 修复后代码（必须可运行）
```

**参考规则**：[对应规则编号和名称]
```

## 示例

### 审查规则速查

**代码风格规则**：
| 规则 | 描述 |
| --- | --- |
| NAM-001 | 变量名应具有描述性，禁止单字母（循环变量除外） |
| NAM-002 | 类名使用 UpperCamelCase |
| NAM-003 | 常量使用全大写下划线 |
| STY-001 | 避免魔法值，提取为常量 |
| STY-002 | 删除注释掉的代码 |

**异常处理规则**：
| 规则 | 描述 |
| --- | --- |
| EXC-001 | 禁止空 catch 块 |
| EXC-002 | 捕获具体异常而非 Exception |
| EXC-003 | 不要返回 null，应抛出异常 |
| EXC-004 | 异常信息应包含上下文 |

**安全规则**：
| 规则 | 描述 |
| --- | --- |
| SEC-001 | 禁止 SQL 拼接，使用参数化查询 |
| SEC-002 | 禁止硬编码密码/密钥 |
| SEC-003 | 校验用户输入 |
| SEC-004 | 敏感信息不写入日志 |

**性能规则**：
| 规则 | 描述 |
| --- | --- |
| PERF-001 | 循环内禁止字符串拼接 |
| PERF-002 | 避免 N+1 查询 |
| PERF-003 | 复用数据库连接 |
| PERF-004 | 合理设置集合初始容量 |

**设计规则**：
| 规则 | 描述 |
| --- | --- |
| DES-001 | 方法不超过 50 行 |
| DES-002 | 方法参数不超过 3 个 |
| DES-003 | 避免重复代码 |
| DES-004 | 单一职责原则 |

**资源规则**：
| 规则 | 描述 |
| --- | --- |
| RES-001 | 使用 try-with-resources 释放资源 |
| RES-002 | ThreadLocal 使用后必须 remove |
| RES-003 | 及时关闭连接/流/文件 |
| RES-004 | 避免内存泄漏 |

## 最佳实践

1. **提交前审查**：所有代码变更提交前必须经过审查

2. **优先修复 Critical**：Critical 级别问题必须修复后才能提交

3. **修复代码必须可运行**：修复建议中的代码必须完整可用

4. **按严重程度排序**：报告按 Critical → Major → Minor → Suggestion 排序

5. **提供上下文**：问题代码至少包含 3 行上下文

6. **引用规则编号**：每个问题引用对应的规则编号

## 常见问题

### Q1：如何提供审查输入？

```bash
# 方式一：提供 git diff
git diff > changes.diff
# 将 diff 内容粘贴给审查工具

# 方式二：提供源文件
# 直接粘贴或指定 Java 文件路径

# 方式三：审查特定提交
git show COMMIT_ID --stat
git diff COMMIT_ID~1 COMMIT_ID
```

### Q2：空 catch 块如何修复？

```java
// 错误
try {
    // ...
} catch (Exception e) {
    // 空
}

// 修复方式一：记录日志
try {
    // ...
} catch (Exception e) {
    logger.error("操作失败", e);
    throw new BusinessException("操作失败", e);
}

// 修复方式二：重新抛出
try {
    // ...
} catch (SpecificException e) {
    throw new RuntimeException(e);
}
```

### Q3：如何避免 N+1 查询？

```java
// 错误：N+1 查询
List<Order> orders = orderRepository.findAll();
for (Order order : orders) {
    User user = userRepository.findById(order.getUserId());  // N 次查询
}

// 修复：批量查询
List<Order> orders = orderRepository.findAll();
Set<Long> userIds = orders.stream()
    .map(Order::getUserId)
    .collect(Collectors.toSet());
Map<Long, User> userMap = userRepository.findByIds(userIds)
    .stream()
    .collect(Collectors.toMap(User::getId, u -> u));
for (Order order : orders) {
    User user = userMap.get(order.getUserId());  // 内存查找
}
```

### Q4：魔法值怎么提取？

```java
// 错误
if (user.getStatus() == 1) {
    // ...
}

// 修复
public static final int STATUS_ACTIVE = 1;
if (user.getStatus() == STATUS_ACTIVE) {
    // ...
}

// 更好：使用枚举
public enum UserStatus {
    ACTIVE(1), INACTIVE(0), BANNED(-1);
    private final int code;
    UserStatus(int code) { this.code = code; }
}
```

### Q5：如何正确释放资源？

```java
// 错误
Connection conn = null;
try {
    conn = dataSource.getConnection();
    // ...
} finally {
    if (conn != null) conn.close();  // 可能抛异常
}

// 正确：try-with-resources
try (Connection conn = dataSource.getConnection();
     PreparedStatement stmt = conn.prepareStatement(sql);
     ResultSet rs = stmt.executeQuery()) {
    // ...
}
```

### Q6：ThreadLocal 如何正确清理？

```java
// 错误：未清理导致内存泄漏
private static final ThreadLocal<UserContext> context = new ThreadLocal<>();

public void handle() {
    context.set(new UserContext());
    // ... 忘记 remove
}

// 正确
public void handle() {
    context.set(new UserContext());
    try {
        // 业务逻辑
    } finally {
        context.remove();  // 必须清理
    }
}
```

## 错误处理

| 错误场景 | 可能原因 | 处理方式 | 优先级 |
|----------|----------|----------|--------|
| 输入参数缺失 | 用户未提供必要参数 | 提示用户提供所需参数后重试 | P0 |
| 执行超时 | 处理时间过长 | 检查输入数据量,分批处理 | P1 |
| 输出格式错误 | 结果不符合预期格式 | 检查`output_format`参数配置 | P1 |

## 依赖说明

### 运行环境
- **Agent 平台**: 支持读取 SKILL.md 的任意 AI Agent（Claude Code / Cursor / Codex / Gemini CLI 等）
- **操作系统**: Windows / macOS / Linux
- **JDK 版本**: 建议 8 及以上

### 依赖详情

| 依赖项 | 类型 | 是否必需 | 获取方式 |
|:-------|:-----|:---------|:---------|
| JDK | 编译器/运行时 | 推荐 | oracle.com 或 openjdk.net 下载 |
| Git | 命令行工具 | 推荐 | 系统包管理器安装 |
| LLM API | API | 必需 | 由 Agent 内置 LLM 提供 |

### API Key 配置
- 本工具为纯 Markdown 指令驱动，无需额外 API Key
- 代码编译验证需要 JDK 环境

### 可用性分类
- **分类**: MD+EXEC（Markdown 指令 + 命令行执行）
- **说明**: 通过自然语言指令驱动 Agent 执行代码审查，编译验证需要 JDK 执行能力

## 已知限制

- 需LLM支持,无LLM环境不可用
- 复杂业务场景建议结合人工经验判断
- 执行效率受模型能力与网络环境影响
- 当前为免费版本,如需完整功能请升级到付费版获取全部能力