코드 냄새 156 - 암시적 Else

프로그래밍 첫 날에 if/else를 배웁니다. 그런 다음 우리는 다른 것을 잊어 버립니다

TL;DR: Be explicit. Even with Else.



문제


  • 가독성
  • 인지 부하
  • 예상치 못한 상황

  • 솔루션


  • 명시적 else 쓰기

  • 문맥



    IF 문장에서 조기에 복귀하면 else 부분을 생략할 수 있습니다.

    그 후, 우리는 다형성을 사용합니다.

    실제 사례가 그리울 때입니다.

    샘플 코드



    잘못된




    function carBrandImplicit(model) {
      if (model === 'A4') {
        return 'audi';
      }
      return 'Mercedes-Benz';
    }
    

    오른쪽



    function carBrandExplicit(model) {
      if (model === 'A4') {
        return 'audi';
      }
      if (model === 'AMG') {
        return 'Mercedes-Benz';
      }
    
      // Fail Fast
      throw new Exception('Model not found);
    }
    

    발각



    [X] 자동

    구문 트리를 확인하고 구문 분석하고 누락된 항목에 대해 경고할 수 있습니다.

    우리는 그것들을 다시 작성하고 돌연변이 테스트를 수행할 수도 있습니다.

    태그


  • 조건부

  • 결론



    이런 종류의 냄새는 많은 공개 토론과 혐오를 불러일으킨다.

    우리는 의견을 교환하고 각각의 장단점을 소중히 여겨야 합니다.

    처지











    더 많은 정보



    Stop Using Implicit Else

    When To Use Implicit Else









    학점



    사진 제공: Elena Mozhvilo on Unsplash


    The biggest issue on software teams is making sure everyone understands what everyone else is doing.



    마틴 파울러






    이 기사는 CodeSmell 시리즈의 일부입니다.


    좋은 웹페이지 즐겨찾기