[전송] PHP Talk '리팩토링 - 기존 코드 디자인 개선' 1부 기능 재구성

동기:

짧고 이름이 잘 지정된 함수: --finely graned

1. 재사용 가능성이 높습니다.

2. 이 함수는 일련의 주석처럼 읽혀집니다.

　3. 기능 덮어쓰기가 쉽습니다.

핵심 포인트: 함수 길이의 핵심은 함수 이름과 함수 온톨로지 사이의 의미론적 거리에 있습니다. 개선 작업을 통해 코드의 명확성이 향상된다면 그렇게 하십시오.

방법:

1. 새 함수를 만들고 함수의 목적에 따라 이름을 지정합니다. "어떻게" 수행하는지보다는 "무엇을" 수행하는지에 따라 이름을 지정합니다.

　=》 메시지나 함수 호출 등 추출 함수가 매우 간단하더라도 새 함수가 코드 의도를 더 나은 방식으로 표현할 수 있는 한 추출해야 합니다. 하지만 더 의미 있는 이름이 생각나지 않는다면 그대로 두십시오.

2. 소스 함수에서 추출 코드를 새 함수로 이동합니다.

2. 인라인 방식

메소드 이름만큼 메소드 본문이 명확하고 이해하기 쉬우면 인라인 메소드를 사용하세요.

3. 인라인 온도

임시변수는 단순 표현식으로 한 번만 할당되며, 할당 후에는 한 번만 사용됩니다. ——온도를 인라인으로 올려주세요

4. 임시를 쿼리로 대체

임시 변수에 표현식이 포함된 경우 표현식 추출 방법이 사용됩니다. ——이것이 바로 쿼리식, 쿼리

동기:

　1. 지역 변수는 코드를 수정하기 어렵게 만듭니다.

　2. 임시 변수를 사용하면 더 긴 코드를 작성해야 합니다. 쿼리 메소드로 변경하면 클래스 하위 메소드에서 해당 정보를 얻을 수 있다. - 더 깔끔한 코드를 작성하겠습니다.

　3. Extract Method를 사용하기 전에 Temp를 Query로 바꾸는 것이 필수적인 단계인 경우가 많습니다.

방법:

1. 한 번만 할당된 임시 변수를 찾습니다.

　　=>　임시 변수가 두 번 이상 할당된 경우 임시 변수 분할을 사용하여 여러 변수로 분할하는 것을 고려해보세요.

　2. 임시변수 할당의 오른쪽 부분을 독립된 함수로 추출합니다.

=> 메소드를 비공개로 선언한 다음 나중에 다른 클래스에서 사용하는 경우 공개 또는 보호를 해제합니다.

동기:

1. 임시 변수가 여러 역할을 담당하는 경우 여러 임시 변수로 대체해야 합니다. 각 변수에는 하나의 책임만 있습니다.

2. 동일한 임시 변수가 두 가지 다른 일을 담당하므로 리뷰가 혼란스러워집니다.

7. 메소드를 메소드 객체로 대체

큰 함수에서 지역 변수를 사용할 때는 Extract Method를 사용할 수 없습니다. 그런 다음 이 메소드를 별도의 객체에 넣어 지역 변수가 객체의 필드가 되도록 한 다음 동일한 객체에서 큰 함수를 여러 개의 작은 메소드로 분해합니다.

동기:

1. 대규모 메소드에서 상대적으로 독립적인 코드를 추출하면 코드의 가독성이 크게 향상될 수 있습니다.

　2. Method에는 지역 변수가 너무 많아서 이 함수를 분해하기가 매우 어렵습니다.

　3. 메소드를 메소드 객체로 대체하면 모든 지역 변수가 객체의 값 범위로 변경됩니다. 그런 다음 이 새 객체에 대해 Extract Method를 수행합니다.