반응형 Mizix131 [리팩터링 카탈로그] 1-6. 함수 선언 바꾸기 절차 변수로의 접근과 갱신을 전담하는 캡슐화 함수들을 만든다. 정적 검사를 수행한다. 변수를 직접 참조하던 부분을 모두 적절한 캡슐화 함수 호출로 바꾼다. 하나씩 바꿀 때마다 테스트 한다. 변수의 접근 범위를 제한단다. 테스트 한다. 변수 값이 레코드라면 레코드 캡슐화하기를 적용할지 고려해본다. 예시 Before: let defaultOwner = { firstName: "John", lastName: "Doe" }; After: let defaultOwnerData = { firstName: "John", lastName: "Doe" }; export function defaultOwner() { return defaultOwnerData; } export function defaultOwner(dat.. 2021. 12. 6. [리팩터링 카탈로그] 1-5. 함수 선언 바꾸기 간단한 절차 매개 변수를 제거하려거든 먼저 함수 본문에서 제고 대상 매개변수를 참조하는 곳은 없는지 확인한다. 메서드 선언을 원하는 형태로 바꾼다. 기존 메서드 선언을 참조하는 부분을 모두 찾아서 바뀐 형태로 수정한다. 테스트 한다. 예시 Before: function circum(radius) { return 2 * Math.PI * radius; } After: function circumference(radius) { return 2 * Math.PI * radius; } 마이그레이션 절차 이어지는 추출 단계를 수원하게 만들어야 한다면 함수의 본문을 적절히 리팩터링한다. 함수 본문을 새로운 함수로 추출한다. 추출한 함수에 매개변수를 추가해야 한다면 ‘간단한 절차를’따라 추가한다. 테스트한다. 기존 .. 2021. 12. 5. [리팩터링 카탈로그] 1-4. 변수 인라인하기 절차 변수가 불변으로 선언되지 않았다면 불변으로 만든 후 테스트 한다. 이 변수를 가장 처음 사용하는 코드를 찾아서 대입문 우변의 코드로 바꾼다. 테스트한다. 변수를 사용하는 부분을 모두 교체할 때까지 이 과정을 반복한다. 변수 선언문과 대입문을 지운다. 테스트 한다. 예시 Before: let basePrice = anOrder.basePrice; return (basePrice > 1000); After: return anOrder.basePrice > 1000; 출처 : 리팩터링 2판 – Chapter6 2021. 12. 5. [리팩터링 카탈로그] 1-3. 변수 추출하기 추출하려는 표현식에 부작용은 없는지 확인한다 불변 변수를 하나 선언하고 이름을 붙일 표현식의 복제본을 대입한다. 원본 표현식을 새로 만든 변수로 교체한다. 테스트한다. 표현식을 여러 곳에서 사용한다면 각각을 새로 만든 변수로 교체한다. 하나 교체할 때마다 테스트한다. 예시 Before: function price(order) { return ( order.quantity * order.itemPrice - Math.max(0, order.quantity - 500) * order.itemPrice * 0.05 + Math.min(order.quantity * order.itemPrice * 0.1, 100) ); } After: function price(order) { const basePrice = .. 2021. 12. 5. 이전 1 2 3 4 5 6 ··· 33 다음 반응형