리팩토링 006 - 결과 변수 이름 바꾸기

'결과'는 매우 나쁜 일반 이름입니다. 그냥 고쳐

TL;DR: Use the last call as a semantic guide.



해결된 문제


  • 변수의 잘못된 이름 지정

  • 관련 코드 냄새











    단계


  • 마지막 함수 호출과 같은 이름으로 변수 이름을 지정합니다.

  • 샘플 코드



    전에




    function doubleFavoriteNumber(n) {
        return this.favoriteNumber * n;
    }
    
    var result = doubleFavoriteNumber(2);
    
    // Many lines after we have no idea what does 
    // result holds
    
    // var result ???
    


    후에




    function doubleFavoriteNumber(n) {
        return this.favoriteNumber * n;
    }
    
    const favoriteNumberDoubled = doubleFavoriteNumber(2);
    
    // Many instructions after
    
    // We can use favoriteNumberDoubled knowing its semantics
    


    유형



    [X] 반자동

    많은 이름 휴리스틱과 마찬가지로 변수를 다른 리팩터링 이름 바꾸기 변수로 바꿀 수 있습니다.

    왜 코드가 더 나은가요?



    변수 범위는 오래 지속될 수 있습니다.

    할당과 사용은 서로 매우 멀리 떨어져 있을 수 있습니다.

    태그


  • 네이밍

  • 또한보십시오



    학점



    HeungSoonPixabay 이미지


    이 문서는 리팩토링 시리즈의 일부입니다.

    좋은 웹페이지 즐겨찾기