[Refactor]: 모듈화로 인한 리팩토링 적용 #20
Merged
Conversation
This file contains hidden or 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
- "Clean and Insert"방식으로 게시글태그 업데이트 - 엔티티의 수정의 경우 dto에서 수정된 엔티티클래스를 받아서 수정하는 것으로 변경
- 누락된 s3이미지 삭제 로직 추가(좀비이미지 방지)
- 기존 QueryDSL 쿼리 제거 후 재작성(Projection 사용) - 반환클래스는 api에서 작성한 걸로 사용하는 것으로 엄격히 유지
- member 조회 쿼리를 방지하기위해 memberId 값으로 필터
- QueryDSL로 게시글 마다 tag조회 쿼리 없이 한번에 조회 - 단건 조회시 QuerDSL 사용과 쿼리수가 같으므로 미사용 - 단건 조회 추가쿼리 발생위해 @entitygraph 적용
- 데드 임포트 제거
모듈화로 인한 게시글 리팩토링 완료
- 클래스 위치에 맞게 패키지 설정 변경
- 패스워드 확인에 대한 검증은 어노테이션으로 따로 설정
- SpringSecurityContext에서 전달받은 id로 조회한 객체는 소유권 검증 제거 - 삭제는 삭제 후 삭제 됐는지 memberId로 검증
refactor: 멤버정보조회 리팩토링 refactor: 멤버 수정 리팩토링 refactor: 회원정보 삭제 리팩토링
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.
Closes #18