게시글 수정 기능 추가 및 작성 폼 확장 #41
Conversation
…-sample-monorepo-simple-crud into feature/post-update
| const updatedData: UpdatePostDTO = { | ||
| title: formData.get('title') | ||
| ? validateFormField(formData.get('title'), 'title') | ||
| : undefined, | ||
| content: formData.get('content') | ||
| ? validateFormField(formData.get('content'), 'content') | ||
| : undefined, | ||
| author: formData.get('author') | ||
| ? validateFormField(formData.get('author'), 'author') | ||
| : undefined, | ||
| }; |
There was a problem hiding this comment.
현재 updateData 객체를 생성하면서 값 추출과 밸리데이션이 함께 이루어지고 있는데, 객체를 생성하면서 바로 밸리데이션 검증을 진행하기보다는,
밸리데이션을 별도의 단계로 나눠서 유효성 검사를 진행하고, 문제가 없다는 것이 확인된 후에 updateData 객체를 생성하거나 다음 로직을 실행하는 흐름으로 구성하면 가독성이 더 좋아질 것 같습니다. ✨
이렇게 하면 코드 흐름이 더 직관적으로 보이고, 각 단계가 명확하게 구분되어 유지보수에도 도움이 될 것 같아요. 🛠️
이런 방향은 어떠신가요? 🤔
There was a problem hiding this comment.
@choi1five 님 좋은 리뷰 감사합니다!🙏
피드백 주신 대로 밸리데이션 검증을 별도로 분리하는 방향으로 개선했습니다.
기존에는 updateData 객체를 만들면서 동시에 validateFormField를 호출했는데,
이를 별도의 유효성 검사 함수getValidatedField로 분리하여 먼저 검증을 수행한 후,
reduce를 활용해 validatedData 객체를 생성하는 방식으로 변경했습니다.
getValidatedField의 경우 공용으로 사용할 수 있어 shared/util로 분리할 수 있었지만,
아직 post 관련 부분에서만 사용하고 있고 validateFormField과 함꼐 사용하고 있기 떄문에
entities/post/lib/form-validation.ts안에 넣어두었습니다.
이렇게 수정하면서 코드의 가독성이 향상되었고, 각 단계가 명확히 구분되었습니다.🔧
혹시 더 개선할 부분이 있다면 피드백 부탁드립니다. 감사합니다! 😊
github-actions
bot
added
status: done
관련 이슈 번호
Resolves #36
핵심 변경 사항 및 이유
이외 기타 변경 사항
PR 시 참고 사항