search

brother1-4752 / marvel-characters-client

0 stars 0 forks source link

[빡준] : 네모 과제리뷰 #1

Open wnsguddl789 opened 1 year ago

wnsguddl789 commented 1 year ago

graphQL 관련 리뷰

전회사에서 작성했던 방법대로 공유할게요

저라면 src/graphql/**/queries, mutations 폴더를 만들고 관리할것같습니다~ 네모님의 코드는 graphql.ts에 모든 타입과 쿼리문을 작성하셨더라고요

개인적으로 헝가리안표기법(IComic)을 선호하지는 않습니다만, 이건 개발자간의 취향차이라...! 더이상 말하지않고 allMarvels는 식별자이름이 모호한거같아요. FETCH_MARVEL_CHARACHER_LIST가 어떨까요?

https://stackoverflow.com/questions/62633904/react-apollo-usequery-hook-with-typescript

상기 링크를 확인해보면 useQuery가 반환하는 data의 타입을 미리 지정해둘수도있습니다~

wnsguddl789 commented 1 year ago

그리고 굳이 다크모드를 위해 리코일을 도입한 이유가 무엇인가요?

context api로도 충분히 가능한것같은데 확장성을 위한것인지 이유가 궁금하네요?

wnsguddl789 commented 1 year ago

나머지 리뷰에 대해서는 폴더라던가 관심사 비슷한것끼리 뭉쳐두면 좋겠어요

제 추천은

configs -> client(apolloClient.ts).ts atoms -> 상태에 따라 네이밍 예를들면 darkMode.atom.ts theme -> 테마 관심사끼리