Skip to content

Week6/sori m2#66

Open
soyun0318 wants to merge 21 commits into
mainfrom
week6/sori-m2
Open

Week6/sori m2#66
soyun0318 wants to merge 21 commits into
mainfrom
week6/sori-m2

Conversation

@soyun0318
Copy link
Copy Markdown
Collaborator

📚 주차 / 미션

  • 6주차 2번째 미션

📌 작업 내용

  • 어떤 작업을 했는지 한 줄 요약

✨ 상세 작업 내용


📸 스크린샷


❓ 리뷰어가 알아야 할 사항 / 질문


✅ 체크리스트

  • 기능 정상 작동 확인
  • 불필요한 주석 삭제
  • 해당 주차 키워드 내용 이해

Copy link
Copy Markdown
Collaborator

@yewon20804 yewon20804 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

6주차 2번째 미션도 수고하셨습니다 ! useInfiniteQuery 무한스크롤 + 스켈레톤 Ui를 진행해주셨는데요 ! 아래 코멘트 확인 부탁드립니다 👍


1. myPage.tsx — useState 초기값 타입 강제 캐스팅

// 현재
const [data, setData] = useState<ResponseMyInfoDto>([] as unknown as ResponseMyInfoDto);

// 수정
const [data, setData] = useState<ResponseMyInfoDto | null>(null);

배열 []을 as unknown as ResponseMyInfoDto 로 강제 캐스팅하면 초기 렌더링 시 data.data?.name 접근에서 런타임 에러가 발생할 수 있습니다! null 초기값 + optional chaining으로 처리해주는 게 안전합니다 !


2. 파일명 컨벤션 — myPage.tsx → MyPage.tsx

React 컴포넌트 파일은 PascalCase가 관례입니다! MyPage.tsx 로 변경하면 프로젝트 전반의 네이밍 컨벤션 일관성을 맞추기 더 좋을 것 같습니다 !


3. 이전 코멘트 내용

SignupPage.tsx — 사용하지 않는 임포트 2개

// 현재
import { data } from "react-router-dom";  
import { is } from "zod/locales";

존재하지 않는 경로는 import 자체가 에러가 나기 때문에 지워주시면 좋을 것 같습니다 !

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