[feature] 메뉴 카테고리를 등록, 수정, 삭제할 수 있다. #39
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
- 메뉴의 id는 자동으로 증가하는 auto increment로 저장하게 하낟. - 외부로 노출되는 id는 uuid로 형태로 저장한다.
- 들어오는 값을 쉽게 검증할 수 있는 spring-boot-starter-validation 라이브러리 추가한다.
- uuid와 id를 명확하게 하기 위해, 기존 필드의 id에서 uuid로 변경한다.
- 결합도를 줄이기 위해 repository 인터페이스로 분리한다.
- 향후 분리와 CQRS패턴을 편리하게 적용하기 위해서 이벤트를 기반으로 데이터를 처리하도록 한다.
- 서비스를 통해 repository에 접근하여 저장하도록 한다.
- 메뉴 카테고리 생성을 요청받으면, 요청값들을 검증하고 이를 서비스에 넘겨준다.
- 생성일과 변경일을 JPA가 관리하도록 @EnableJpaAuditing을 추가해준다.
- @PathVariable은 기본적으로 null이 될 수 없기 때문에 이 어노테이션을 제거한다.
- 필드명은 일반적으로 소문자로 시작해야 하므로, `MenuCategoryUuid`를 `menuCategoryUuid`로 변경한다.
- JpaRepository를 상속받고 있으므로, `save`메서드를 제거한다.
- MenuCategoryCreatedEvent객체를 불변하도록 변경한다.
- 기존 복잡해진 repository는 데이터베이스를 처리하는 역할만 하고, 이벤트를 발행하고 처리하는 곳은 서비스로 이동시킨다.
…ory-c [feature] 메뉴 카테고리를 등록할 수 있다
- 같은 모듈내에서는 event를 발생시켜 처리하는 방식대신, 직접 처리하고 검색시스템에서 이벤트를 받아 동작하도록 변경한다.
- 기존 MenuCategoryDto -> MenuCategoryCreateDto로 변경한다.
- uuid를 기준으로 값을 찾고, entity에 값을 변경하여 반환하도록 한다.
- 변경된 값의 id와 정보들을 모두 포함하도록 한다.
…기능 추가 - 요청 받은 정보를 service에 넘겨주고, service에서 로직을 처리하도록 한다.
…기능 추가 - 요청 받은 정보를 service에 넘겨주고, service에서 로직을 처리하도록 한다.
- `MenuCategory` -> `menuCategory`로 변경한다.
- create 정적 팩토리 메서드로 값을 생성하도록 한다. - @Setter를 삭제하고, update()로만 값을 변경하도록 한다.
…ory-ud [feature] 메뉴 카테고리를 수정, 삭제할 수 있다.
2 tasks
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.
#️⃣ 연관된 이슈
📝 작업 내용
💬 리뷰 요구사항(선택)
RCA 룰
| 코드 리뷰 멘토가 PR작성자에게 어떤 의도로 전달 되길 바라는지 알파벳으로 표현해주세요!