카테고리 도메인 리팩토링 - 패키지 위치, 도메인 로직, 쿼리 개선

[le2sky]

구현 내용

구현 요약

도메인 레이어 개선

• repository 패키지 위치 개선
  • domain 내부로 수정했습니다.
• 카테고리 도메인 규칙, 로직 개선
  • 카테고리 네이밍 제약 강화(최대 20글자)
  • 시퀀스 범위 제약 강화(0-4까지 숫자)

응용 레이어 개선

• category_limit_count(카테고리 생성 제한 수)를 클래스 상수로 변경
  • 기존 하드 코딩된 값을 상수화 했습니다.
• category_shuffle 테스트 개선
  • auto_increment로 인해 테스트가 깨지는 것을 개선했습니다.
• CategoryService.shuffle 매개변수 중 changeCategorySequenceDtos → changedCategories로 네이밍 개선(기존에 xxDtos로 설정되어 있었음) 했습니다.
• ShuffleSequenceService이라는 도메인 서비스를 신규 개발해 도메인 레이어 내부로 카테고리 순서 변경 로직을 이동했습니다.

영속성 레이어 개선

• repository 쿼리 개선
  • 기존 findCategoryEntityGraphById는 카테고리 ID를 받아서 UserJpaEntity가 로딩된 카테고리를 반환합니다.
  • 특정 사용자가 다른 사용자의 카테고리에 접근하는 것을 방지하기위해 해당 쿼리를 응용 영역에서 사용합니다.
  • 하지만,select * from category where category.id = id and category.user_id = user_id 쿼리로도 충분히 방어 로직 작성이 가능하기 때문에 수정했습니다.
  • 수정 이후 findCategoryEntityGraphById -> findUserCategory로 네이밍 개선했습니다.

문서 업데이트

• 리포지터리 위치를 개선했기 때문에 패키지 구조 문서를 업데이트했습니다.

관련 이슈

close #155

구현 내용

• [ ] Bug fix (non-breaking change which fixes an issue)
• [ ] New feature (non-breaking change which adds functionality)
• [x] Breaking change (fix or feature that would cause existing functionality to not work as expected)

[le2sky]

@akalswl14 코드 변경이 많아서! 제가 선리뷰 먼저 작성하고 나서 코드 리뷰 요청하도록 하겠습니다! 🙇🏻‍♂️🙇🏻‍♂️

[sonarcloud[bot]]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[le2sky]

@akalswl14 리뷰 감사합니다! 테스트 관련해서 해당 conversation에 커멘트 남겨놨습니다!