// 代码重复示例
public class UserService {
    public User findUserById(Long id) {
        if (id == null) {
            throw new IllegalArgumentException("ID cannot be null");
        }
        
        if (id <= 0) {
            throw new IllegalArgumentException("ID must be positive");
        }
        
        // 查询用户逻辑
        return userRepository.findById(id);
    }
    
    public Order findOrderById(Long id) {
        if (id == null) {
            throw new IllegalArgumentException("ID cannot be null");
        }
        
        if (id <= 0) {
            throw new IllegalArgumentException("ID must be positive");
        }
        
        // 查询订单逻辑
        return orderRepository.findById(id);
    }
    
    public Product findProductById(Long id) {
        if (id == null) {
            throw new IllegalArgumentException("ID cannot be null");
        }
        
        if (id <= 0) {
            throw new IllegalArgumentException("ID must be positive");
        }
        
        // 查询产品逻辑
        return productRepository.findById(id);
    }
}

// 传统代码评审 vs AI辅助代码评审对比

// 传统代码评审示例
/*
人工评审员可能会发现的问题：
1. 缺少异常处理
2. 变量命名不够清晰
3. 方法过长
4. 缺少注释
*/

public class OrderService {
    public void processOrder(Order order) {
        // 复杂的业务逻辑
        if (order.getItems().size() > 0) {
            double total = 0;
            for (int i = 0; i < order.getItems().size(); i++) {
                total += order.getItems().get(i).getPrice() * order.getItems().get(i).getQuantity();
            }
            
            if (total > 1000) {
                // 大额订单处理逻辑
                applyDiscount(order, total);
            }
            
            // 保存订单
            orderRepository.save(order);
        }
    }
}

// AI辅助代码评审建议示例
/*
AI评审建议：
1. ⚠️ 缺少异常处理：数据库操作可能抛出异常
2. ⚠️ 性能问题：重复调用order.getItems().size()
3. ⚠️ 可读性问题：使用传统的for循环而非增强for循环
4. ⚠️ 安全问题：未验证订单数据的有效性
5. 💡 建议：提取计算总价的逻辑为独立方法
6. 💡 建议：添加适当的日志记录
*/

public class OrderService {
    private static final Logger logger = LoggerFactory.getLogger(OrderService.class);
    
    @Transactional
    public void processOrder(Order order) throws OrderProcessingException {
        try {
            validateOrder(order);
            
            BigDecimal total = calculateOrderTotal(order);
            
            if (total.compareTo(new BigDecimal("1000")) > 0) {
                applyDiscount(order, total);
            }
            
            orderRepository.save(order);
            logger.info("Order processed successfully: {}", order.getId());
            
        } catch (DataAccessException e) {
            logger.error("Failed to save order: {}", order.getId(), e);
            throw new OrderProcessingException("Failed to process order", e);
        }
    }
    
    private void validateOrder(Order order) throws InvalidOrderException {
        if (order == null) {
            throw new InvalidOrderException("Order cannot be null");
        }
        
        if (order.getItems() == null || order.getItems().isEmpty()) {
            throw new InvalidOrderException("Order must have items");
        }
    }
    
    private BigDecimal calculateOrderTotal(Order order) {
        return order.getItems().stream()
            .map(item -> item.getPrice().multiply(BigDecimal.valueOf(item.getQuantity())))
            .reduce(BigDecimal.ZERO, BigDecimal::add);
    }
}