用 AI 做代码审查：效率提升 5 倍的实践

审查以下 Java 代码，重点关注：
1. 空指针风险
2. 并发安全
3. SQL 注入
4. 性能问题
5. 代码规范

[粘贴代码]

# 审查最近一次 commit
claude > 审查最近一次 git commit 的代码

# 审查特定文件
claude > 审查 src/service/OrderService.java

# 审查整个 PR
claude > 对比 main 分支和当前分支的改动，审查所有变更

# .github/workflows/ai-review.yml
name: AI Code Review
on: [pull_request]
jobs:
  review:
    runs-on: ubuntu-latest
    steps:
      - uses: anthropics/claude-code-action@v1
        with:
          anthropic-api-key: ${{ secrets.ANTHROPIC_API_KEY }}

审查这段代码，从以下角度：
1. 正确性：逻辑是否正确
2. 安全性：有无漏洞
3. 性能：有无瓶颈
4. 可维护性：代码是否清晰
5. 边界情况：是否处理了异常输入

对于每个发现，给出严重程度（高/中/低）和具体修复建议。

对这段代码进行安全审查，重点检查：
1. SQL 注入
2. XSS 攻击
3. CSRF 防护
4. 敏感信息泄露
5. 权限校验缺失
6. 文件上传漏洞

给出 OWASP Top 10 相关的安全建议。

审查这段代码的性能问题：
1. 数据库查询是否高效
2. 是否有 N+1 查询
3. 缓存使用是否合理
4. 是否有内存泄漏风险
5. 并发性能如何

给出具体的优化建议和代码示例。

审查这段 Java 代码，遵循以下规范：
- Google Java Style Guide
- SOLID 原则
- Spring Boot 最佳实践

重点关注：
1. 异常处理是否合理
2. 资源是否正确关闭
3. 线程安全
4. Optional 使用
5. Stream API 使用是否恰当

public String getUserName(Long userId) {
    User user = userRepository.findById(userId);
    return user.getName();
}

问题：高 - 空指针风险
findById 可能返回 null，直接调用 getName() 会 NPE。

修复：
public String getUserName(Long userId) {
    return userRepository.findById(userId)
        .map(User::getName)
        .orElse("Unknown");
}

public List<User> searchUsers(String keyword) {
    String sql = "SELECT * FROM users WHERE name LIKE '%" + keyword + "%'";
    return jdbcTemplate.query(sql, new UserRowMapper());
}

问题：高 - SQL 注入
直接拼接用户输入到 SQL 中，存在注入风险。

修复：
public List<User> searchUsers(String keyword) {
    String sql = "SELECT * FROM users WHERE name LIKE ?";
    return jdbcTemplate.query(sql, new UserRowMapper(), "%" + keyword + "%");
}

public void deductStock(Long productId, int quantity) {
    Product product = productRepository.findById(productId);
    if (product.getStock() >= quantity) {
        product.setStock(product.getStock() - quantity);
        productRepository.save(product);
    }
}

问题：高 - 并发安全
非原子操作，高并发下可能超卖。

修复：使用乐观锁或悲观锁
@Lock(LockModeType.PESSIMISTIC_WRITE)
Product findByIdWithLock(Long id);

或使用 CAS：
@Modifying
@Query("UPDATE Product p SET p.stock = p.stock - :qty WHERE p.id = :id AND p.stock >= :qty")
int deductStock(Long id, int qty);