search

2njeong / next.js_todolist

0 stars 0 forks source link

[NextJS] 개인과제 리뷰 - 최원장 튜터 #1

Open wonee09 opened 3 months ago

wonee09 commented 3 months ago

별도 타입 분리 굳

image 타입이 특정 컴포넌트에 종속되는 경우 관리가 매우 어려울 수 있습니다. 좋은 습관이에요.

커스텀 훅 분리 굳

image 마찬가지로 hooks도 별도 분리해주셨어요. 특히 useQuery, useMutation의 경우 반복되는 로직이 매우 많고 컴포넌트 단에서 '선언' -> '실행'이 되는 경우 복잡할 수 있어서 분리해주시는 게 좋은데, 이 부분을 잘 작성해주셨습니다.

고급문법도 촥촥

image suspense, memoization 등 어려운 개념일 수 있는데, 아무렇지 않게 툭툭 던져놓는 부분 넘 멋있네요!! :)

타입추론을 위한 타입명시 필요

image 가능하다면 타입 명시를 해주시는게 협업 측면에서, 휴먼에러 방지 차원에서 좋을 수 있겠습니다.

dehydration

image 인정님! 왜케 퍼포먼스가 좋아요??? ㅋㅋ 대단한데 react-query에서 dehydration을 사용한 분은 첨봐요. 일케 하면 초기 로딩에 대한 부분을 보장받을 수 있으니 더없이 좋은 적요이죠!! 굳굳 다만, 연습하려고 한 것인지, About 페이지는 CSR로 작성될 필요는 없을 것 같아요 ㅎㅎ

민감정보 숨기기

image 아마 알고있겠지만, env 파일에 민감한 정보는 숨기도록 합시닷

컴포넌트 위치

image 개인마다 스탈은 다르긴 하지만, 저는 별도 컴포넌트는 components로 아에 분리시키는 편이에요!

완성도 넘 높은 과제였어요. 고생 정말 많았습니다!!

2njeong commented 3 months ago

감사합니다 튜터님!! 시험삼아 써본 건데 처음에 제 코드 아닌 줄 알았어요ㅋㅋㅋㅋ 오랜만에 보니까 너무 낯설네요 다시 잘 공부해보겠습니다. 주말에도 리뷰해주시는 튜터님ㅠㅠ 정말 감사합니다!

2024년 3월 22일 (금) 오전 10:19, wonee09 @.***>님이 작성:

별도 타입 분리 굳

image.png (view on web) https://github.com/2njeong/next.js_todolist/assets/62840864/8964ebbc-2527-4f21-ab74-6969b89d27ed 타입이 특정 컴포넌트에 종속되는 경우 관리가 매우 어려울 수 있습니다. 좋은 습관이에요. 커스텀 훅 분리 굳

image.png (view on web) https://github.com/2njeong/next.js_todolist/assets/62840864/eb37373e-cf9c-43ff-8691-54c0f3cedb4d 마찬가지로 hooks도 별도 분리해주셨어요. 특히 useQuery, useMutation의 경우 반복되는 로직이 매우 많고 컴포넌트 단에서 '선언' -> '실행'이 되는 경우 복잡할 수 있어서 분리해주시는 게 좋은데, 이 부분을 잘 작성해주셨습니다. 고급문법도 촥촥

image.png (view on web) https://github.com/2njeong/next.js_todolist/assets/62840864/0fd24986-0fd0-4cfd-ad16-57268fd3db0a suspense, memoization 등 어려운 개념일 수 있는데, 아무렇지 않게 툭툭 던져놓는 부분 넘 멋있네요!! :) 타입추론을 위한 타입명시 필요

image.png (view on web) https://github.com/2njeong/next.js_todolist/assets/62840864/36bb8b2f-db5c-444d-b1f1-3fd954cb282f 가능하다면 타입 명시를 해주시는게 협업 측면에서, 휴먼에러 방지 차원에서 좋을 수 있겠습니다. dehydration

image.png (view on web) https://github.com/2njeong/next.js_todolist/assets/62840864/6c87953d-670a-42b6-b670-15a8a3f96872 인정님! 왜케 퍼포먼스가 좋아요??? ㅋㅋ 대단한데 react-query에서 dehydration을 사용한 분은 첨봐요. 일케 하면 초기 로딩에 대한 부분을 보장받을 수 있으니 더없이 좋은 적요이죠!! 굳굳 다만, 연습하려고 한 것인지, About 페이지는 CSR로 작성될 필요는 없을 것 같아요 ㅎㅎ 민감정보 숨기기

image.png (view on web) https://github.com/2njeong/next.js_todolist/assets/62840864/1a7f8327-e96b-40d7-afd8-efe8afcf7965 아마 알고있겠지만, env 파일에 민감한 정보는 숨기도록 합시닷 컴포넌트 위치

image.png (view on web) https://github.com/2njeong/next.js_todolist/assets/62840864/34c17490-f242-4fcb-a242-62bd208dd305 개인마다 스탈은 다르긴 하지만, 저는 별도 컴포넌트는 components로 아에 분리시키는 편이에요!

완성도 넘 높은 과제였어요. 고생 정말 많았습니다!!

— Reply to this email directly, view it on GitHub https://github.com/2njeong/next.js_todolist/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/BE2TIXLD2DK5KQGJOC7JIZDYZOBKNAVCNFSM6AAAAABFCPGCMWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGIYDCNBZGIZTMMA . You are receiving this because you are subscribed to this thread.Message ID: @.***>