코드 냄새 169 - 접착 방법

한 번에 둘 이상의 것을 만들지 마십시오.

TL;DR: Try to be as atomic as possible in your methods



문제


  • 결합된 코드
  • 테스트하기 어려움
  • 읽기 어려움

  • 솔루션


  • 방법을 깨십시오

  • 리팩토링






    문맥



    메서드 이름을 'And'로 지정하면 추출 및 중단 메서드 기회를 놓치고 있는 것일 수 있습니다.

    샘플 코드



    잘못된




    
    calculatePrimeFactorsRemoveDuplicatesAndPrintThem()
    
    // Three responsibilities
    
    

    오른쪽



    
    calculatePrimeFactors();
    
    removeDuplicates();
    
    printNumbers();
    
    // Three diferent methods
    
    // We can test them and reuse them
    
    

    발각



    [X] 반자동

    일부 linter는 'and'라는 용어를 포함하여 메서드에 대해 경고할 수 있습니다.

    태그


  • 커플링

  • 결론



    방법을 만들 때 고무 오리 이야기를 재생하고 우리가 일을 올바르게 만들고 있는지 스스로에게 말하는 것이 매우 중요합니다.

    처지







    부인 성명



    코드 냄새는 그냥 내 .

    학점



    사진 제공: Scott Sanker on Unsplash


    Learning the art of programming, like most other disciplines, consists of first learning the rules and then learning when to break them.



    조슈아 블로흐






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


    좋은 웹페이지 즐겨찾기