[Book] Refactoring: 9 데이터 조직화
리팩터링
출처: 리팩터링 2판, 마틴 파울러
09 테이터 조직화
데이터 구조는 프로그램에서 중요한 역할을 수행한다.
9.1 변수 쪼개기
Split Variable
- 1판에서의 이름: 매개변수로의 값 대입 제거
- 1판에서의 이름: 임시변수 분리
// before
let temp = 2 * (height + width);
console.log(temp);
temp = height * width;
console.log(temp);
// after
const perimeter = 2 * (height + width);
console.log(perimeter);
const area = height * width;
console.log(area);
배경
역할이 둘 이상인 변수가 있다면 쪼개야 한다. 예외는 없다. 역할 하나당 변수 하나다.
절차
- 변수를 선언한 곳과 값을 처음 대입하는 곳에서 변수 이름을 바꾼다.
- 가능하면 이때 불변
immutable으로 선언한다. - 이 변수에 두 번째로 값을 대입하는 곳 앞까지의 모든 참조(이 변수가 쓰인 곳)를 새로운 변수 이름으로 바꾼다.
- 두 번째 대입 시 변수를 원래 이름으로 다시 선언한다.
- 테스트한다.
- 반복한다. 매 반복에서 변수를 새로운 이름으로 선언하고 다음번 대입 때까지의 모든 참조를 새 변수명으로 바꾼다. 이 과정을 마지막 대입까지 반복한다.
9.2 필드 이름 바꾸기
Rename Field
// before
class Organization {
get name() {...}
}
// after
class Organization {
get title() {...}
}
배경
이름은 중요하다. 레코드 구조체의 필드 이름들은 특히 더 중요하다. 데이터 구조는 프로그램을 이해하는 데 큰 역할을 한다.
절차
- 레코드의 유효 범위가 제한적이라면 필드에 접근하는 모든 코드를 수정한 후 테스트한다. 이후 단계는 필요 없다.
- 레코드가 캡슐화되지 않았다면 우선 레코드를 캡슐화한다.
- 캡슐화된 객체 안의 private 필드명을 변경하고, 그에 맞게 내부 메서드들을 수정한다.
- 테스트한다.
- 생성자의 매개변수 중 필드의 이름이 겹치는 게 있다면 함수 선언 바꾸기로 변경한다.
- 접근자들의 이름도 바꿔준다.
9.3 파생 변수를 질의 함수로 바꾸기
Replace Derived Variable with Query
// before
get discountedTotal() {return this._discountedTotal}
set discountedTotal(aNumber) {
const old = this._discount;
this._discount = aNumber;
this._discountedTotal += old - aNumber;
}
// after
get discountedTotal() {return this._baseTotal - this._discount}
set discountedTotal() {this._discount = aNumber}
배경
가변 데이터는 서로 다른 두 코드에서 연쇄 효과를 일으켜 다른 쪽 코드에 원인을 찾기 어려운 문제를 야기하기도 한다. 가변 데이터를 완전히 배제하기란 불가능하므로 유효 범위를 가능한 좁혀야 한다.
절차
- 변수 값이 갱신되는 지점을 모두 찾는다. 필요하면 변수 쪼개기를 활용해 각 갱신 지점에서 변수를 분리한다.
- 해당 변수의 값을 계산해주는 함수를 만든다.
- 해당 변수가 사용되는 모든 곳에 어서션을 추가하여 함수의 계산 결과가 변수의 값과 같은지 확인한다.
- 테스트한다.
- 변수를 읽는 코드를 모두 함수 호출로 대체한다.
- 테스트한다.
- 변수를 선언하고 갱신하는 코드를 죽은 코드 제거하기로 없앤다.
9.4 참조를 값으로 바꾸기
Change Reference to Value
- 반대 리팩터링: 값을 참조로 바꾸기
// before
class Product {
applyDiscount(arg) {
this._price.amount -= arg;
}
}
// after
class Product {
applyDiscount(arg) {
this._price = new Money(this._price.amount - arg, this.price.currency);
}
}
배경
필드를 값으로 다룬다면 내부 객체의 클래스를 수정하여 값 객체Value object로 만들 수 있다. 값 객체는 대체로 자유롭게 활용하기 좋은데, 특히 불변이기 때문이다. 값 객체는 분산 시스템과 동시성 시스템에서 특히 유용하다.
절차
- 후보 클래스가 불변인지, 혹은 불변이 될 수 있는지 확인한다.
- 각각의 세터를 하나씩 제거한다.
- 이 값 객체의 필드들을 사용하는 동치성 비교 메서드를 만든다.
9.5 값을 참조로 바꾸기
Change Value to Reference
- 반대 리팩터링: 참조를 값으로 바꾸기
// before
let customer = new Customer(customerData);
// after
let customer = customerRepository.get(customerData.id);
배경
하나의 데이터 구조 안에 논리적으로 똑같은 제3의 데이터 구조를 참조하는 레코드가 여러 개 있을 때 값이나 참조로 다룰 수 있다. 값으로 다룬다면 데이터가 각각 복사되고, 참조로 다룬다면 단 하나의 데이터 구조를 참조하게 된다.
절차
- 같은 부류에 속하는 객체들을 보관할 저장소를 만든다(이미 있다면 생략).
- 생성자에서 이 부류의 객체들 중 특정 객체를 정확히 찾아내는 방법이 있는지 확인한다.
- 호스트 객체의 생성자들을 수정하여 필요한 객체를 이 저장소에서 찾도록 한다. 하나 수정할 때마다 테스트한다.
9.6 매직 리터럴 바꾸기
Replace Magic Literal
- 1판에서의 이름: 마법 숫자를 기호 상수로 전환
function potentialEnergy(mass, height) {
return mass * 9.81 * height;
}
const STANDARD_GRAVITY = 9.81;
function potentialEnergy(mass, height) {
return mass * STANDARD_GRAVITY * height;
}
배경
코드 자체가 뜻을 분명하게 드러내는 게 좋다. 상수를 정의하고 숫자 대신 상수를 사용하도록 바꾸면 된다.
방법
- 상수를 선언하고 매직 리터럴을 대입한다.
- 해당 리터럴이 사용되는 곳을 모두 찾는다.
- 찾은 곳 각각에서 리터럴이 새 상수와 똑같은 의미로 쓰였는지 확인하여, 같은 의미라면 상수로 대체한 후 테스트한다.