如何避免JPA Repository返回的List出现null值？

问题根源 #

你遇到的null值和seqNo间隙问题，核心是对@OrderColumn的工作机制理解偏差，加上双向关联未正确同步导致的：

1. @OrderColumn的自动维护缺陷：Hibernate的@OrderColumn是通过内存中列表的索引位置来更新seqNo字段的。手动调用Collections.swap仅改变列表内存顺序，Hibernate只会检测到被交换的两个元素的位置变化，不会自动同步其他元素的seqNo；当产品移动到新分类时，旧分类的seqNo不会自动重新编号，自然出现间隙。
2. 双向关联未同步：移动产品时，若只修改某一侧的关联关系（比如仅从旧分类列表移除产品，未更新产品的parentCategory），会导致持久化上下文状态不一致，进而出现null值或数据异常。
3. orphanRemoval=true的干扰：该配置会在产品从分类列表移除时，若未同步设置parentCategory=null，Hibernate可能误判为需要删除产品，进一步加剧数据混乱。

修复方案 #

放弃依赖@OrderColumn自动维护顺序，改为手动控制seqNo字段，同时封装双向关联的同步逻辑，确保数据一致性。

1. 修改实体类 #

移除@OrderColumn，改用@OrderBy指定查询时的排序规则，并在Category类中封装顺序调整、产品移动的方法：

Category类

@Data
@Entity
public class Category {
    @Id private String id;

    private String displayName;

    @OneToMany(
            mappedBy = "parentCategory",
            cascade = CascadeType.ALL,
            orphanRemoval = true
    )
    @OrderBy("seqNo ASC") // 查询时按seqNo升序排列
    private List<Product> products = new ArrayList<>(); // 初始化空列表避免空指针

    // 交换两个产品的顺序
    public void swapProducts(int index1, int index2) {
        if (index1 < 0 || index1 >= products.size() || index2 < 0 || index2 >= products.size()) {
            throw new IllegalArgumentException("索引超出范围");
        }
        Product p1 = products.get(index1);
        Product p2 = products.get(index2);
        // 交换seqNo值
        int tempSeq = p1.getSeqNo();
        p1.setSeqNo(p2.getSeqNo());
        p2.setSeqNo(tempSeq);
        // 同步内存列表顺序（可选，因为@OrderBy会按seqNo排序）
        Collections.swap(products, index1, index2);
    }

    // 将产品移动到目标分类的指定位置
    public void moveProductTo(Product product, Category targetCategory, int targetPosition) {
        // 从当前分类移除产品并同步关联
        this.products.remove(product);
        product.setParentCategory(null);
        // 重新编排当前分类的seqNo，消除间隙
        reindexSeqNo();

        // 添加到目标分类并设置关联
        targetCategory.getProducts().add(targetPosition, product);
        product.setParentCategory(targetCategory);
        // 重新编排目标分类的seqNo
        targetCategory.reindexSeqNo();
    }

    // 重新编排当前分类下所有产品的seqNo，生成连续编号
    private void reindexSeqNo() {
        for (int i = 0; i < products.size(); i++) {
            products.get(i).setSeqNo(i);
        }
    }
}

Product类

@Data
@Entity
public class Product {
    @Id
    private String id;

    private String displayName;
    
    @ToString.Exclude
    @JsonIgnore
    @ManyToOne(fetch = FetchType.LAZY)
    private Category parentCategory;

    @JsonIgnore
    private int seqNo;
}

2. 业务层操作规范 #

• 所有顺序调整、产品移动操作，必须通过Category类提供的封装方法执行，禁止直接修改products列表。
• 操作必须在事务内完成，确保所有seqNo修改和关联同步能被持久化。

3. 示例调用 #

// 交换分类中两个产品的顺序
@Transactional
public void swapProductOrder(String categoryId, int index1, int index2) {
    Category category = categoryRepository.findById(categoryId).orElseThrow();
    category.swapProducts(index1, index2);
    categoryRepository.save(category);
}

// 移动产品到新分类
@Transactional
public void moveProduct(String sourceCategoryId, String targetCategoryId, String productId, int targetPos) {
    Category source = categoryRepository.findById(sourceCategoryId).orElseThrow();
    Category target = categoryRepository.findById(targetCategoryId).orElseThrow();
    Product product = productRepository.findById(productId).orElseThrow();
    source.moveProductTo(product, target, targetPos);
    categoryRepository.save(source);
    categoryRepository.save(target);
}

关键说明 #

• 手动维护seqNo彻底避免了@OrderColumn的自动维护缺陷，确保seqNo始终连续无间隙。
• 封装关联操作逻辑，保证双向关联的一致性，避免Hibernate持久化时出现状态混乱。
• @OrderBy确保查询分类下的产品列表时，始终按seqNo顺序返回。

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