Spring Boot支付系统：带Type字段的DTO请求处理最佳实践问询

项目背景 #

我正在开发一个大学项目，用Spring Boot搭建后端处理前端请求，核心功能之一是管理多种类型的用户支付并生成数据库日志，通过专属支付控制器调用支付服务实现。目前的实现思路是把各支付类型的具体逻辑封装成独立方法，通过一个接收GenericPaymentRequestDTO（包含Type字段）的通用方法统一处理：先校验权限，再根据Type调用对应支付方法，最后统一生成并保存PaymentLog。

问题1：通用DTO存在大量非必填字段，需手动做空值校验，是否应改为为每种支付类型创建专属DTO，通过@NotNull注解实现校验？ #

建议为每种支付类型创建专属DTO，理由如下：

• 符合单一职责原则：每个DTO只对应一种支付场景的参数需求，结构清晰，后续维护时不会混淆字段用途。
• 利用JSR-380校验注解（如@NotNull）自动完成参数校验，省去手动写空值判断的冗余代码，还能通过Spring全局异常处理器统一返回校验失败的响应。
• 减少接口歧义性，避免前端传递不必要的字段，降低参数传递错误的概率。

举个示例，针对不同支付类型创建专属DTO：

@Data
public class PaymentXRequestDTO {
    @NotNull
    private PaymentLogType type = PaymentLogType.TYPE_X; // 固定类型，避免前端传错
    @NotNull
    private Long managerId;
    @NotNull
    private String message;
    @NotNull
    private Double price;
    @NotNull // 该场景必填，直接注解校验
    private Long userId;
    @NotNull
    private Long couponId;
}

@Data
public class PaymentYRequestDTO {
    @NotNull
    private PaymentLogType type = PaymentLogType.TYPE_Y;
    @NotNull
    private Long managerId;
    @NotNull
    private String message;
    @NotNull
    private Double price;
    @NotNull
    private Long itemId;
}

后续可以在控制器层接收不同DTO，或者结合策略模式优化支付逻辑分发，提升代码扩展性。

问题2：日志生成时通过orElseThrow()获取关联实体（item、coupon等），但具体支付方法已处理实体不存在异常，此代码是否冗余？ #

这部分代码确实冗余，还存在性能问题：

• 冗余性：既然processPaymentX、processTypeY等方法已经验证了实体存在（通过orElseThrow抛出异常），日志阶段再次查询实体完全没必要——前面方法执行成功就意味着实体肯定存在。
• 性能问题：重复执行数据库查询会增加数据库负载，高并发场景下影响更明显。

优化方案 #

1. 让支付方法返回实体对象
   修改具体支付方法，返回需要存入日志的实体，避免重复查询：

private User processPaymentX(Long userId, Long couponId) {
    User user = userRepository.findById(userId).orElseThrow(/*异常处理*/);
    Coupon coupon = couponRepository.findById(couponId).orElseThrow(/*异常处理*/);
    // 执行支付X的业务逻辑
    return user; // 或者返回包含user和coupon的自定义对象
}

然后在通用方法中直接使用返回的实体：

case TYPE_X->  {
    User user = processPaymentX(request.getUserId(), request.getCouponId());
    Coupon coupon = ...; // 从支付方法返回结果中获取
    log.setUser(user);
    log.setCoupon(coupon);
}

2. 使用JPA的getReferenceById避免实际查询
   如果不需要实体的具体属性，只是维护外键关联，可以用getReferenceById生成代理对象，不会立即查询数据库：

log.setUser(userRepository.getReferenceById(request.getUserId()));

JPA保存日志时只需要ID来维护外键，这样既避免重复查询，又保证日志正确性。

原核心代码片段 #

通用处理方法

@Transactional
public void processGenericPayment(GenericPaymentRequestDTO request) throws AccessDeniedException, IllegalArgumentException{
    //0. 权限校验
    User manager = userRepository.findById(email).orElseThrow(//异常处理);
    if(!userService.checkAdminAccess(email)) {
        throw new AccessDeniedException("Not allowed");
    }
    
    //1. 创建日志
    PaymentLog log = new PaymentLog();
    //2. 根据Type调用对应支付方法
    switch(request.getType()) {
        case TYPE_X->  {
            if(request.getUserId() == null || request.getCouponId() == null) throw new IllegalArgumentException("Something");
            processPaymentX(request.getUserId(), request.getCouponId());
            log.setUser(userRepository.findById(request.getUserId()).orElseThrow(//异常处理)
            log.setCoupon(couponRepository.findById(request.getCouponId()).orElseThrow(//异常处理))
        }
        case TYPE_Y-> {
            if(request.getItemId() == null) throw new IllegalArgumentException("Something");
            processTypeY(request.getItemId());
            log.setItem(itemRepository.findById(request.getItemId()).orElseThrow(//异常处理))
        }
        case TYPE_Z-> {
            if(request.getUserId() == null) throw new IllegalArgumentException("Something");
            processTypeZ(request.getUserId());
            log.setItem(itemRepository.findById(request.getUserId()).orElseThrow(//异常处理))
        }
        // 其他类型
    }

    //3. 完善日志信息
    log.setDate(LocalDateTime.now());
    log.setMessage(request.getMessage());
    // 其他字段赋值

    //4. 保存日志
    logRepository.save(log);
}

GenericPaymentRequestDTO

@Getter
@Setter
@NoArgsConstructor
@Data
public class GenericPaymentRequestDTO {
    @NotNull
    private PaymentLogType type;
    @NotNull
    private Long managerId; // 处理支付的管理员ID，服务端校验权限
    @NotNull
    private String message;
    @NotNull
    private Double price;
    private Long userId; // 支付类型X使用
    private Long itemId; // 支付类型Y使用
    private Long couponId; // 支付类型X、Z使用
    // 其他所需字段...
}

PaymentLog实体

@Getter
@Setter
@Entity
@Table(name = "payment_logs")
public class PaymentLog implements Serializable {
    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    @Column(name = "log_id", nullable = false)
    private Long logId;
    @NotNull
    @Column(name = "price")
    private Double price;
    @NotNull
    @Column(name = "date", nullable = false)
    private LocalDateTime date;
    @NotNull
    @Column(name = "message")
    private String message;
    @NotNull
    @Column(name="type", nullable = false, columnDefinition = "payment_log_type")
    private PaymentLogType type;
    @NotNull
    @ManyToOne(fetch = FetchType.LAZY, optional = false)
    @JoinColumn(name = "manager_id", nullable = false)
    private User manager;
    // 根据支付类型，以下字段可为空
    @ManyToOne(fetch = FetchType.LAZY)
    @JoinColumn(name = "item_id")
    private InventoryItem item;
    @ManyToOne(fetch = FetchType.LAZY)
    @JoinColumn(name = "coupon_id")
    private Coupon coupon;
    @ManyToOne(fetch = FetchType.LAZY)
    @JoinColumn(name = "user_id")
    private User user;
    // 其他字段已省略
}

内容的提问来源于stack exchange，提问作者minicrub