Skip to content

[5주차] 워크북 미션 - toy#9

Open
jinseong-yoon wants to merge 3 commits into
mainfrom
feat/week5-toy
Open

[5주차] 워크북 미션 - toy#9
jinseong-yoon wants to merge 3 commits into
mainfrom
feat/week5-toy

Conversation

@jinseong-yoon

Copy link
Copy Markdown
Collaborator

워크북과 미션을 동시에 진행해서 뒤죽박죽입니다.
5시간 안에 끝날 줄 알았는데 기한 넘겨서 제출했습니다..

@drghdtjr

drghdtjr commented May 1, 2026

Copy link
Copy Markdown
Contributor

코드 전반적으로 잘 확인했습니다!
특히 DTO 설계에 Java의 Record를 활용하신 점이 인상적인데, 코드의 가독성과 불변성을 동시에 챙긴 좋은 선택인 것 같습니다.
현재는 DTO의 필드값이 비어있는 상태로 들어올 경우에 대한 처리가 보이지 않는데, @Valid와 같은 어노테이션을 활용해 검증 로직을 추가해 보시면 예상치 못한 null 값이나 빈 문자열로 인해 발생할 수 있는 에러를 컨트롤러 단에서 미리 방지할 수 있어 더 좋을 것 같습니다.
수고하셨습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants