Spring Data JpaRepository“JOIN FETCH”返回重复项

Question

我正在编写一个简单的 Spring Data JPA 应用程序。我使用 MySQL 数据库。有两个简单的表：

• 部门
• 员工

每个员工都在某个部门工作 (Employee.department_id)。

@Entity
public class Department {
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    @Id
    private Long id;

    @Basic(fetch = FetchType.LAZY)
    @OneToMany(mappedBy = "department")
    List<Employee> employees;
}

@Entity
public class Employee {
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    @Id
    private Long id;

    @ManyToOne
    @JoinColumn
    private Department department;
}

@Repository
public interface DepartmentRepository extends JpaRepository<Department, Long> {
    @Query("FROM Department dep JOIN FETCH dep.employees emp WHERE dep = emp.department")
    List<Department> getAll();
}

getAll 方法返回一个包含重复部门的列表（每个部门重复的次数与该部门的员工数量一样多）。

问题 1：我是否认为这是一个与 Spring Data JPA 无关但与 Hibernate 相关的功能？
问题2：修复它的最佳方法是什么？ （我发现至少有两种方法：1）使用Set<Department> getAll(); 2）在@Query注解中使用"SELECT DISTINCT dep"）

Answer 1

我不确定在这里使用@Query。如果您通过注释 @OneToMany 和 @ManyToOne 映射这两个实体，则此连接是多余的。 第二个时刻，为什么@Basic(fetch = FetchType.LAZY) 在这里？也许您应该在 @OneToMany 中设置惰性初始化？ Check out this resource, I hope you'll find it helpfull

Answer 2

FROM Department dep JOIN FETCH dep.employees emp 表达式生成本机查询，该查询返回纯结果Department-Employee。每个Department 将返回dep.employees.size() 次。这是 JPA 提供者的预期行为（在您的情况下为 Hibernate）。

使用distinct 去除重复项似乎是一个不错的选择。 Set<Department> 作为查询结果，无法得到有序结果。

Answer 3

据我所知，您提到的用例不需要定义自己的方法，而是使用继承自 PagingAndSortingRepository 的 repository.findAll，它扩展了 CrudRepository 并由 SimpleJpaRepository 实现。见here

所以只需将存储库界面留空，如下所示，

@Repository
public interface DepartmentRepository extends JpaRepository<Department, Long> {

}

然后在任何需要的地方注入并使用它，例如让我们说 DepartmentService

public interface DepartmentService {
    
    List<Link> getAll();    
}

@Component
public class DepartmentServiceImpl implements DepartmentService {

    private final DepartmentRepository  repository;    

    // dependency injection via constructor
    @Autowired
    public DepartmentServiceImpl(DepartmentRepository repository) {
        this.repository = repository;
    }

    @Override
    public List<Department> getAll() {  
        return repository.findAll();
    }
}