kết hợp một thực thể tách rời hoặc mới với đối tượng hiện có trong câu hỏi thực hành tốt nhất hibernate/jpa

Question

Khi lớp doanh nghiệp tạo một thực thể mới.), đây có phải là phương pháp sáp nhập thực hành xấu không?

public User add(User user){ 

    User existingUser = getUserDao().findByBusinessKey(user.getBusinessKey(), false); 
    user.setId(existingUser.getId()); 

    user = getUserDao().merge(user); 

    return user; 
} 

Tôi hỏi vì thiết lập ID một cách rõ ràng trên thực thể tách rời cảm thấy khá xa lạ với tôi, nhưng dù bằng và hashCode phương pháp của đối tượng người dùng được thực hiện một cách thích hợp, thiết lập ID ở đây là cách duy nhất để đảm bảo việc hợp nhất diễn ra.

Có thực tiễn nào tốt hơn không?

Có những hạn chế cụ thể nào đối với phương pháp này có thể khiến tôi khó chịu sau này không?

Cảm ơn bạn đã xem!

Answer 1

mã mà sẽ việc, nhưng thiết ID một cách rõ ràng trên thực thể tách ra không cần thiết. Ứng dụng Hibernate điển hình có phương thức 'lưu' xử lý hai trường hợp:

1. Người dùng muốn tạo Người dùng mới, do đó ứng dụng sẽ tạo đối tượng Người dùng có 'null' làm ID.
2. Người dùng được truy vấn cho một danh sách người dùng và đang chọn một danh sách để chỉnh sửa. Trong trường hợp này, ứng dụng thực hiện truy vấn và truyền đối tượng đến phương thức 'lưu'. Đối tượng sẽ có một ID và mã sẽ áp dụng các giá trị mới cho nó.

Có vẻ như nội dung nào đó trong mã của bạn không hoạt động trong trường hợp thứ hai theo cách thông thường. Nếu đối tượng 'người dùng' đến từ một số truy vấn Hibernate trước (được kích hoạt bởi người dùng nhấp vào 'chỉnh sửa người dùng' hoặc một cái gì đó tương tự), thì nó sẽ có một ID. Do đó, chỉ cần gọi số merge(user).

Tôi thường làm điều gì đó như thế này:

if (user.getId() == null) 
    em.persist(user); 
else 
    user = em.merge(user); 

Sau đó, tôi thêm mã để xử lý các vấn đề lạc quan khóa (một phiên cập nhật các đối tượng) và các vấn đề hạn chế duy nhất (phiên khác đã cố gắng để tồn tại một cái gì đó với phím kinh doanh cùng).

Các khung như Seam có thể làm điều này đơn giản hơn vì chúng truyền bá phiên Hibernate giữa các phương thức của trình điều khiển. Vì vậy, ngay cả 'hợp nhất' là không cần thiết.

Answer 2

Nếu thực thể của bạn là một thực thể tách rời, điều duy nhất bạn thực sự cần làm là gọi entityManager.merge (người dùng). Bạn không cần phải thực hiện bất kỳ phương pháp tìm. Nếu thực thể của bạn không bị tách rời mà là mới (không có id được chỉ định), bạn nên tìm thực thể thích hợp trong cơ sở dữ liệu trước khi thực hiện bất kỳ thao tác sửa đổi nào trên thực thể đó và hợp nhất nó sau đó. tức là:

User user = userDao.findBySomething(Criteria c); 

//stuff that modifies user 

user = userDao.merge(user);