Skip to content

Conversation

@doitchuu
Copy link
Member

@doitchuu doitchuu commented Apr 15, 2025

close #15

5주차 기다려주셔서 감사합니다 👍
한 주 쉰덕에 더 보기 수월했던 것 같아요!

감기 조심하세요 🥇 🏠 🍄‍🟫

@github-actions github-actions bot requested a review from tjd985 April 15, 2025 14:32
@tjd985 tjd985 added the 리뷰해주세욥 나는 지금 리뷰가 고프다 label Apr 16, 2025
Copy link
Contributor

@tjd985 tjd985 left a comment

Choose a reason for hiding this comment

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

잘읽었습니다~~~

오타 두개만 수정해주세요!
이번주도 고생 많으셨습니당ㅎㅎㅎ

Copy link
Contributor

Choose a reason for hiding this comment

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

type Obj = { [key: string]: number; };

type KeyType = keyof Obj; // string | number;

이 예제에서 key는 string | number이며, 이는 자바스크립트 객체 키가 항상 문자열로 강제 지정되므로 객체[0]은 항상 객체["0"]와 동일하기 때문이네요!

덕분에 하나 배워갑니다!! 👍🏻

Copy link
Member Author

Choose a reason for hiding this comment

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

오호 저도 몰랐었는데 덕분에 배워요!!!!!

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
이때 타입스크립트는 키가 유효하지 않더라도 타입상으로는 문제가 없기 떄문에 오류를 표시하지 않는다.
이때 타입스크립트는 키가 유효하지 않더라도 타입상으로는 문제가 없기 때문에 오류를 표시하지 않는다.

Copy link
Member Author

Choose a reason for hiding this comment

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

수정 완료했습니다!

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
> 이럴 때는 개발 중에 유효하지 않은 키가 사용되었는 지 또는 undefined일 수 있는 값이 있는 지 등 사전에 타입스크립트 기능을 활용헤 파악하는 것이 중요하다.
> 이럴 때는 개발 중에 유효하지 않은 키가 사용되었는 지 또는 undefined일 수 있는 값이 있는 지 등 사전에 타입스크립트 기능을 활용해 파악하는 것이 중요하다.

Copy link
Member Author

Choose a reason for hiding this comment

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

수정 완료했습니다!

@tjd985 tjd985 added 오타있어요! 오타 뭐에용😟 잘 봤습니다 👍🏻 굿잡.. 빠른 수정 감사합니다 ^__^ 우사인 볼트도 울고 갈 수정 속도 머지 하세요! 이제 당신에게 남은건 merge뿐이야.. 고생하셨습니다 🫡 고생한 당신.. 쉬어라! and removed 리뷰해주세욥 나는 지금 리뷰가 고프다 오타있어요! 오타 뭐에용😟 labels Apr 17, 2025
Copy link
Contributor

@tjd985 tjd985 left a comment

Choose a reason for hiding this comment

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

겁나게 빠른 오타 수정.. 감사합니다!!

@doitchuu doitchuu merged commit b4e6a81 into main Apr 17, 2025
@doitchuu doitchuu deleted the week5_seulgi branch April 17, 2025 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

고생하셨습니다 🫡 고생한 당신.. 쉬어라! 머지 하세요! 이제 당신에게 남은건 merge뿐이야.. 빠른 수정 감사합니다 ^__^ 우사인 볼트도 울고 갈 수정 속도 잘 봤습니다 👍🏻 굿잡..

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5주차 정리

3 participants