Spring/JPA自定义查询异常：订单号按账户自增逻辑问题求助

我在基于ORM的电商项目中遇到了自定义查询的问题，项目包含客户账户、商品列表、当前购物车、交易历史四张表。需求是结账时将购物车所有商品转存到交易历史，且每个账户的订单号需按账户ID单独递增，但现有自定义查询逻辑无法实现该需求，附上相关代码请帮忙指出问题。

Cart DAO 查询方法 #

public void deleteAllByAccountId(int accountId);

public void deleteByBookIdAndAccountId(int bookId, int accountId);

public List<CartEntity> findAllByAccountId(int accountId);

@Query ("SELECT MAX(orderNo) FROM current_cart WHERE accountid = :#{customer_accounts.accountid}")
public CartEntity findByAccountId(@Param("accountid") int accountId);

Cart 实体类 #

@Table(name = "current_cart")
public class CartEntity {

    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY) // Id是自动生成的主键
    @Column(name ="orderno")
    private int orderNo;

    @Column(name = "accountid")
    private int accountId;

    @Column(name = "cost")
    private int bookCost;

    @Column(name = "quantity")
    private int quantity;

    @Column(name = "booktitle")
    private String bookTitle;

    @Column(name = "bookid")
    private int bookId;
}

@Table(name = "customer_accounts")
public class AccountEntity {

    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY) // Id是自动生成的主键
    @Column(name = "accountid")
    private int accountId;

    @Column(name = "email")
    private String email;

    @Column(name = "password")
    private String password;

    @Column(name = "firstname")
    private String firstname;

    @Column(name = "lastname")
    private String lastname;
}

结账逻辑 #

public void Checkout(int accountId) {

    List<CartEntity> FetchedCartEntities = cartDao.findAllByAccountId(accountId);

    List<TransactionHistoryEntity> transactionsToCopy = new ArrayList<TransactionHistoryEntity>();
    for (CartEntity cartEntity : FetchedCartEntities) {
        TransactionHistoryEntity transaction = new TransactionHistoryEntity();

        BeanUtils.copyProperties(cartEntity, transaction);

        transactionsToCopy.add(transaction);
    }
    transactionHistoryDao.saveAllAndFlush(transactionsToCopy);

    cartDao.deleteAllByAccountId(accountId);
}

问题分析与修正方案 #

1. 自定义查询方法的语法与类型错误

• 返回类型不匹配：SELECT MAX(orderNo) 查询的是整数类型的最大值，但方法返回CartEntity，类型完全不匹配，应改为返回Integer。
• JPQL表达式错误：:#{customer_accounts.accountid} 是错误的Spring EL用法，直接使用@Param定义的参数:accountId即可，且JPQL应引用实体类而非数据库表名，正确写法如下：

@Query("SELECT MAX(c.orderNo) FROM CartEntity c WHERE c.accountId = :accountId")
public Integer findMaxOrderNoByAccountId(@Param("accountId") int accountId);

2. 订单号生成逻辑错误

当前CartEntity的orderNo使用IDENTITY自增策略，这是全局自增，无法实现按账户ID单独递增的需求。需修改为手动生成订单号：

• 移除CartEntity中orderNo字段的@GeneratedValue(strategy = GenerationType.IDENTITY)注解；
• 在新增购物车商品时，先通过上述修正后的查询方法获取该账户的最大订单号：
  • 若返回null（该账户无购物车记录），则订单号设为1；
  • 若有值，则订单号为最大值+1；
• 将计算后的订单号赋值给CartEntity的orderNo字段后再保存。

3. 结账逻辑的缺失

现有结账逻辑仅直接复制购物车的订单号到交易历史，未处理交易历史的订单号递增需求（若交易历史也需按账户递增订单号）。需在转存时重新计算交易历史的订单号，或确保购物车的订单号已按规则生成，直接复用即可。

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