삼항 연산자 단순하게 하기

가독성

삼항 연산자를 복잡하게 사용하면 조건의 구조가 명확하게 보이지 않아서 코드를 읽기 어려울 수 있어요.

📝 코드 예시

다음 코드는 A조건과 B조건에 따라서 "BOTH", "A", "B" 또는 "NONE" 중 하나를 status에 지정하는 코드예요.

const status =
  (A조건 && B조건) ? "BOTH" : (A조건 || B조건) ? (A조건 ? "A" : "B") : "NONE";

👃 코드 냄새 맡아보기

가독성

이 코드는 여러 삼항 연산자가 중첩되어 사용되어서, 정확하게 어떤 조건으로 값이 계산되는지 한눈에 파악하기 어려워요.

✏️ 개선해보기

다음과 같이 조건을 if 문으로 풀어서 사용하면 보다 명확하고 간단하게 조건을 드러낼 수 있어요.

const status = (() => {
  if (A조건 && B조건) return "BOTH";
  if (A조건) return "A";
  if (B조건) return "B";
  return "NONE";
})();