Open
Conversation
YoungJJun
approved these changes
Apr 7, 2026
Collaborator
There was a problem hiding this comment.
컨버터 관련해서 조사해주신 내용은 모두 맞는 내용입니다.
약간의 추가설명을 드리자면 공통 로직의 분리와 비슷하게 들리실 수 있는데 컨버터가 없으면 controller나 service 등에서 '입력을 원하는 형태로 변경' 하거나 '결과를 return하고 싶은 형태로 변경' 하는 등의 책임을 맡게됩니다.
즉 컨트롤러가 변환이라는 책임을 지는것인데 이는 SRP 관점에서 좋지 않아서 컨트롤러에서 변환의 책임을 제거하고 컨버터만이 변환의 책임을 맡도록 하는 이유에서 사용되기도 합니다!
추가로 커밋 메시지가 mission으로 되어있는데 해당 내용은 keyword에 해당하니까 keyword: 로 시작하는 컨벤션으로 메시지 작성해주시면 됩니다.
앞으로도 해당 레포지토리에는 미션 없이 키워드만 제출하게 되는 경우가 많을테니 참고해주세요.
레오 4주차도 잘 조사해주셨습니다. 머지전에 피어리뷰 추가 해주세요.
시험 화이팅입니다~🍎
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
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.
✅ 워크북 체크리스트
✅ 컨벤션 체크리스트