[22.05.15] TIL

이진·2022년 5월 15일

TIL

목록 보기

12/22

과제 코드 리뷰

기업과제 코드리뷰를 받아 생각나는 내용을 적어본다. 감사합니다..

1. dl, dt, dd

dt와 dd는 무조건 dl로 감싸야한다.
dt와 dd는 1대1 매칭이 되게 해야한다.

<dl>
  <dt>title</dt>
  <dd>year</dd>
  <dd>type</dd>
</dl>

====

<dt>Year</dt> <dd>{year}</dd>

2. 조건 처리

로딩이 끝났을 때 영화 리스트가 없다면 보여주는 조건 -> 에러 메시지의 유무 조건

// 조건이 더 직관적이게
!movieList.length
===
movieList.length === 0

movieList.length === 0
===
movieList.length > 0

// 렌더 안에 괄호가 들어가 있으면 안됨(차피 boolean)
const isFavorite = () => { return list.filter(...) }
===
const isFavorite = list.filter(...)

3. .finally

axios는 Promise async await 불필요

const getMovie = async () => {
  setIsLoading(true)
  await axios.get(...).then(....).catch(....)
  setIsLoading(false)
}

===
  
// .finally를 이용해라
const getMovie = () => {
  setIsLoading(true)
  axios.get(...).then(....).catch(....).finally(setIsLoading(false))
}