너무 많은 일을하는 커스텀훅 어떻게 처리해야 할까? #649
hwangstar156
started this conversation in
refactor
Replies: 1 comment
-
const {
isLoading,
content,
handleSubmitButtonClick,
title,
isValidTitleInput,
setTitle,
titleInputRef,
categoryOption,
setCategoryOption,
hashTags,
setHashTags,
} = usePostWritingArticles({ category, isAnonymous }); 현재 WritingArticles내부의 usePostWritingArticles입니다. 동작만 보면 post했을경우의 로직을 처리하는것 같지만 실제로는 모든 로직을 처리하고 있습니다. 리액트 쿼리가 이미 훅으로 되있는데 불필요하게 빼고 있는것 같습니다. 리액트 쿼리는 컴포넌트 단에서 사용하고 비동기통신 외의 로직은 다른 커스텀 훅으로 빼는게 좋을것 같습니다. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
현재 하나의 커스텀 훅이 모든 로직을 담고 있습니다. 분리가 필요해보입니다.
Beta Was this translation helpful? Give feedback.
All reactions