Study/JavaScript(Clean code)

[JavaScript][clean-code] Default Case고려하기

갈푸라떼 2022. 5. 30. 16:07

* Default Case고려하기

- JS는 사용자와 상호작용하는 언어이다. 따라서 사용자의 입력을받을수도있고 사용자가 만질수도 있다.
 그럴때마다 값이 변경되고 런타임 프로그램이 동작되는 시기도 무언가 결정이 될 수 있다.

- 사용자에게 값을 얻지 못하였을때 어떠한 기본값을 둘지 정책을 정할 수 있다.
(기본값을 정해놓을것은 매우 중요하다.)

- 이를 edge case라고도 한다.

- team의 core한 유틸리티 함수나 core한 라이브러리를 개발하는 팀은 항상 이런부분을 염두해 두어야 한다.
(그래야 더욱더 안전하고 확장성 높은 코드를 작성할 수 있다.)


* 기본값 설정 예시 코드

function sum(x, y) {
  x = x || 1
  y = y || 1

  return x + y;
}

sum();
  • or연산자를 이용하여서 단축평가한다.
  • 유저가 실수로 값을 안넣을경우 default값인 1이 반환된다.

* 기본값 설정 예시코드2

function createElement(type, height, width) {
  const element - document.createElement(type || 'div');

  element.style.height = height || 100;
  element.style.width = width || 100;

  return element;
}

createElement();

* 기본값 설정 예시코드3

function registerDay(userInputDay) {
  switch (userInputDay) {
    case '월요일': // some code
    case '화요일': // some code
    case '수요일': // some code
    case '목요일': // some code
    case '금요일': // some code
    case '토요일': // some code
    case '일요일': // some code
    default:
      throw Error('입력값이 유효하지 않습니다.')
  }
}
  • 절대 변하지 않는값인 월 ~ 일 날짜값이여도 사용자가 입력을 잘못하게 되면 Error를 발생시킬 수 있다.
  • 따라서 사용자의 오입력 및 미입력을 방지하고자 Default Case를 고려해야한다.
  • 물론 try ... catch ... 를 이용해도 되지만 명시적으로 Error를 선언하는건 중요한 포인트이다.

* Default Case예시문

const Root = () => {
  <Router history={browserHistory}>
    <Switch>
      <Route exact path="/" component={App}/>
      <Route path="/" component={Welcome}/>
      <Route component={NotFound}/>
    </Switch>
  </Router>
};
  • 리액트 라우터의 Switch구문
  • 프로그래밍 언어적으로 switch case문으로 변경되는것은 아니다.
  • 결론은 Switch를 쓰면 마지막에는 Edge case를 고려할 수 있다.
  • witch문을 쓸 때도 default case를 고려하듯이 프로그래밍을 작성할 때 항상 이런 case를 고려해야한다.


* parseInt()또한 기본값이 10진수가 아니므로 꼭 10진수를 명시해줘야한다.

ex) parseInt(2, 10)

 

- 혹은 parseInt를 안전하게 함수를 만들어주는 방법

functin safeParseInt(number, radix) {
  return parseInt(number, radix || 10);
}

 

 

(참조) 어떤 라이브러리든 default값을 중요하게 생각한다.