Skip to content

Commit

Permalink
docs: README.md 업데이트
Browse files Browse the repository at this point in the history
  • Loading branch information
pitangland authored Jan 20, 2025
1 parent 4ed92e9 commit 633eaed
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -191,3 +191,13 @@ App.tsx는 관심사가 모두 모여있는 구조로 리팩토링이 필요할

타입 안전성 관련
- ref.current?.value as T에서 이것은 잘못된 값이 반환되어도 컴파일러가 경고하지 않게 만들 수 있다고 찾아보았습니다. 따라, as T를 사용하는 방식이 안전한지에 대해 검토받고싶습니다.

고생하셨습니다 원표님!

> as T를 사용하는 방식이 안전한지에 대해 검토받고싶습니다.
네 고민하신 부분이 맞습니다. 타입스크립트를 사용하는 이유는 타입을 추론하게 하여 트랜스파일단계에서 타입에러를 사전에 체크하기 위함인데요. 타입단언은 추론을 못하게 만드는 장치이다보니 정말 추론이 어려운 써드파티의 코드들에서만 사용을 권장하곤합니다.
현재의 코드를 제네릭으로 구현하시다보니 반환코드에서 타입 린트에러가 나셨던 듯 하네요. 내부 로직에서 undefined 가드를 해보셔서 단언을 없애보시면 좋을 듯 합니다.

기본과제 피드백으로 갈음하겠습니다!
취소선에 있는 리팩토링 계획 기대할게요 ㅎㅎ!! 리팩토링후에 개인적으로 리뷰 주셔도 좋습니다~

0 comments on commit 633eaed

Please sign in to comment.