[디디] JPA 미션 제출합니다. #58
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Favorite 메서드명 수정
- 정상적으로 동작하도록 메서드 수정 - data.sql 오류 제거
- EnableJdbcAuditingConfig.java 클래스 명 수정 - Favorite 생성자 파라미터명 수정 - 불필요한 fetchType 제거 - 불필요한 메서드 제거
woowahan-pjs
requested changes
Jun 18, 2020
| import lombok.Getter; | ||
| import lombok.NoArgsConstructor; | ||
|
|
||
| @MappedSuperclass |
Comment on lines
+27
to
+31
| @CreatedDate | ||
| private LocalDateTime createdAt; | ||
|
|
||
| @LastModifiedDate | ||
| private LocalDateTime updatedAt; |
There was a problem hiding this comment.
아래의 글을 읽고 Audit 기능을 적용해 보세요.
https://umanking.github.io/jpa-audit
| } | ||
|
|
||
| public void validateMember(Member member) { | ||
| if (!this.member.isSameId(member)) { |
There was a problem hiding this comment.
this.member.equals(member)와 같은 방법으로 구현할 수 있을까요?
There was a problem hiding this comment.
주신 피드백에 대해 고민을 많이 해봤습니다. 혹시 제가 이해한게 맞는지 확인해주시면 감사하겠습니다.😀
- Entity는 지연 로딩에서 프록시 객체를 반환하는데, equals는 class비교를 하므로 이를 감안하여 equals를 작성한다.
- Entity는 영속화되기 이전에 ID를 갖지 않으므로 null인 경우를 감안하여 equals를 작성한다.
이 두 가지를 생각하여, null인 경우와 Hibernate.getClass를 이용하여 equals를 작성하였습니다. 혹시 이런 경우 외에도 제가 고민해야할만한 부분이 있을까요? 말씀해주신다면 찾아보도록 하겠습니다 😄
오우 제가 너무 생각이 많았던 것 같네요. 깊은 뜻이 있는 줄 알고.. Id만 비교하도록 수정했습니다..!
- JpaAuditing - equals & hashcode
- createdBy, updatedBy 삭제
woowahan-pjs
approved these changes
Jun 19, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
안녕하세요 제이슨☺️
테스트 (동작) 주도 개발로 진행해서 코드가 매우 지저분하네요.. 일단 동작은 하네요 ..😭
지속적으로 리팩토링 하겠습니다... 이상하다 싶은 점 있다면 말씀해주시면 감사하겠습니다..