Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[고도화] 엔티티 코드 리팩토링 - equals(), hashcode()에서 필드 접근을 getter로 바꾸기 #60

Merged
merged 1 commit into from
Jan 6, 2023

Conversation

lmw7414
Copy link
Owner

@lmw7414 lmw7414 commented Jan 6, 2023

이 pr은 엔티티의 equals(), hashcode()가 값 비교를 하기 위해 필드에 직접 접근하려는 것을 getter 접근으로 바꾼다.
프록시 객체를 사용하는 하이버네이트의 지연로딩을 고려하여, 값 비교를 제대로 수행하지 못하는 일이 없도록 한다.
This closes #59

여기서 필드에 직접 접근하면 하이버네이트가 지연 로딩하려고 만든 프록시 객체를 다룰 때 필드 값이 `null`일 수 있음
그러면 제대로 비교 로직을 수행할 수 없기 때문에 getter를 이용함.
이것으로 프록시 객체를 타더라도 제대로 값을 읽어올 수 있음.
@lmw7414 lmw7414 added the enhancement New feature or request label Jan 6, 2023
@lmw7414 lmw7414 added this to the 게시판 서비스 만들기 milestone Jan 6, 2023
@lmw7414 lmw7414 self-assigned this Jan 6, 2023
Copy link
Owner Author

@lmw7414 lmw7414 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@lmw7414 lmw7414 merged commit d79b0c1 into main Jan 6, 2023
@lmw7414 lmw7414 deleted the feature/#59-equals-hashcode-refactoring branch January 6, 2023 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[고도화] 엔티티 코드 리팩토링 - equals(), hashcode()에서 필드 접근을 getter로 바꾸기
1 participant