Code Smell 170 - 기능 변경으로 리팩토링

개발은 훌륭합니다. 리팩토링은 놀랍습니다. 동시에 만들지 마세요

TL;DR: Don't change functionally and refactor at the same time.



문제


  • 솔루션을 검토하기 어려움
  • 병합 충돌

  • 솔루션


  • 리팩터링하는 동안 기능을 변경하지 않음

  • 문맥



    추가 개발을 위해 리팩토링이 필요한 경우가 있습니다.

    우리는 배움의 전문가입니다.

    솔루션을 보류해야 합니다. 리팩토링 작업을 하고 솔루션을 계속 진행합니다.

    샘플 코드



    잘못된




    factorial(n) {
      return n * factorial(n-1);
    }
    


    오른쪽




    getFactorial(n) {
      return n * getFactorial(n);
    }
    
    // Change
    
    getFactorial(n) {
      return n * getFactorial(n-1);
    }
    
    // Run the tests
    
    factorial(n) {
      return n * factorial(n-1);
    }
    
    // Rename
    


    발각



    이것은 리팩토링 냄새입니다.

    [X] 수동

    태그


  • 리팩토링

  • 결론



    물리적 토큰을 사용해야 합니다.

    우리는 리팩토링 단계에 있거나 개발 단계에 있습니다.

    부인 성명



    Code Smells is just my opinion .

    학점



    사진 제공: Dannie Jing on Unsplash


    When I’m studying code, refactoring leads me to higher levels of understanding that I would otherwise miss. Those who dismiss comprehension refactoring as useless fiddling with the code don’t realize they never see the opportunities hidden behind the confusion.



    마틴 파울러

    %[ https://maximilianocontieri.com/software-engineering-great-quotes ]


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

    %[ https://maximilianocontieri.com/how-to-find-the-stinky-parts-of-your-code ]

    좋은 웹페이지 즐겨찾기