[리팩터링 2판] 12.7 서브클래스 제거하기

1. 언제 사용하면 좋을까

더이상 쓰이지 않는 서브클래스와 마주하는 프로그래머는 가치 없는 것을 이해하느라 에너지를 낭비할 것이다.

이런 정도까지 되면 서브클래스를 슈퍼클래스의 필드로 대체해 제거하는 게 최선이다.

불필요한 서브 클래스를 제거하고, 대신에 외부에서 전혀 알 필요없는 내부로직들을 잘 캡슐화해두고, 

외부에서 사용하기 쉽게 create라는 팩토리 메소드를 제공해주고, 기타 필요한 메소드를 잘 챙겨주도록 하자. 

 

2. 절차

(1) 서브클래스의 생성자를 팩터리 함수로 바꾼다.

-> 생성자를 사용하는 측에서 데이터 필드를 이용해 어떤 서브클래스를 생성할지 결정한다면 그 결정 로직을 슈퍼클래스의 팩터리 메서드에 넣는다.

(2) 서브클래스의 타입을 검사하는 코드가 있다면 그 검사 코드에 함수 추출하기와 함수 옮기기를 차례로 적용하여 슈퍼클래스로 옮긴다. 하나 변경할 때마다 테스트한다.

(3) 서브클래스의 타입을 나타내는 필드를 슈퍼클래스에 만든다.

(4) 서브클래스를 참조하는 메서드가 방금 만든 타입 필드를 이용하도록 수정한다.

(5) 서브클래스를 지운다.

(6) 테스트한다.

 

3.  리팩토링 전 전체 코드

class Person {
	#name;
	constructor(name) {
		this.#name = name;
	}

	get name() {
		return this.#name;
	}

	get genderCode() {
		return 'X';
	}
}

class Male extends Person {
	get genderCode() {
		return 'M';
	}
}

class Female extends Person {
	get genderCode() {
		return 'F';
	}
}

function loadFromInput(data) {
	const result = [];
	data.forEach((record) => {
		let person;
		switch (record.gender) {
			case 'M':
				person = new Male(record.name);
				break;
			case 'F':
				person = new Female(record.name);
				break;
			default:
				person = new Person(record.name);
		}
        result.push(person);
	});
    return result;
}

const people = loadFromInput([
	{ name: '은하', gender: 'F' },
	{ name: '신비', gender: 'F' },
	{ name: '진우', gender: 'M' },
	{ name: '태현', gender: 'M' },
	{ name: '태연', gender: 'F' }
]);

출력값

4.  리팩토링 후 전체 코드

class Person {
	#name;
	#genderCode;
	constructor(name, genderCode) {
		this.#name = name;
		this.#genderCode = genderCode;
	}

	get name() {
		return this.#name;
	}

	get genderCode() {
		return this.#genderCode;
	}

	get isMale() {
		return this.#genderCode === 'M';
	}

	static create(record) {
		switch (record.gender) {
			case 'M':
				return new Person(record.name, 'M');
				break;
			case 'F':
				return new Person(record.name, 'F');
				break;
			default:
				return new Person(record.name, 'X');
				break;
		}
	}
}

const people = loadFromInput([
	{ name: '은하', gender: 'F' },
	{ name: '신비', gender: 'F' },
	{ name: '진우', gender: 'M' },
	{ name: '태현', gender: 'M' },
	{ name: '태연', gender: 'F' }
]);

function loadFromInput(data) {
	const result = [];
	data.forEach((element) => {
		result.push(Person.create(element));
	});
	return result;
}

const numberOfMales = people.filter((p) => p.isMale).length;
console.log(numberOfMales);

출력값