[TIL 210823] 코드 리뷰 시 다시 한번 체크할 것

Ko Seoyoung·2021년 8월 23일
0

✅ Today I Learned

목록 보기
8/16

1. 조건을 체크하여 boolean을 리턴하는 함수 네이밍

// Hmm 🤔
  const beOutOfBoundary = (index: number): boolean => {
    return index < 0 || index > itemLength - 1;
  };

// Good 👍
  const checkIsOutofBoundary = (index: number): boolean => {
    return index < 0 || index > itemLength - 1;
  };

일반적이지 않은 beOutOfBoundary보다 checkIsOutOfBoundary가 더 이해하기 쉽다.

2. onChange -> onCurrentChange

onChange는 뭔가 event를 반환할 것으로 착각할 수도 있다. 좀더 명확한 네이밍이 좋을 것같다.


profile
Web Frontend Developer 👩🏻‍💻 #React #Nextjs #ApolloClient

0개의 댓글