[리팩터링 2판] 11.13 예외를 사전확인으로 바꾸기

0. 언제 쓰면 좋을까?

예외도 (더 이상 좋지 않을 정도까지) 과용되곤 한다. 예외는 '뜻밖의 오류'라는, 말 그대로 예외적으로 동작할 때만 쓰여야 한다.

함수 수행시 문제가 될 수 있는 조건을 함수 호출 전에 검사할 수 있다면,

예외를 던지는 대신 호출하는 곳에서 조건을 검사하도록 해야 한다.

1. 절차

(1) 팩터리 함수를 만든다. 팩터리 함수의 본문에서는 원래의 생성자를 호출한다.

(2) 생성자를 호출하던 코드를 팩터리 함수 호출로 바꾼다.

 

2. 예제 코드

(1) 리팩터링 전

해당 코드의 문제점 : 전달받은 인덱스를 통해서 배열의 특정한 값을 반환하는 것인데,

인덱스를 전달받을 때, 인덱스가 0보다 작거나 또는 인덱스가 주어진 배열의 사이즈보다 넘어가는건 실패 케이스로 충분히 예상(expected)가능하다. 따라서 배열 안에, 값을 찾지 못했다고 에러를 던지는 것은 예외처리를 남용하는 것이다.

const values = [];
function getValueForPeriod(periodNumber) {
	const value = values[periodNumber];
	if (!value) {
		throw new Error('value is undefined');
	}
	return value;
}

try {
	getValueForPeriod(-10);
} catch (error) {
	console.log('에러 발생');
}

(1) 리팩터링 후보1

const values = [];
function getValueForPeriod(periodNumber) {
	if (periodNumber < 0 || periodNumber >= values.length) {
		return 0;
	}
	return values[periodNumber];
}

getValueForPeriod(-10);

(2) 리팩터링 후보2

 

자바스크립트에서는 배열의 인덱스를 벗어난다고 해서, 크래쉬되지 않아 에러가 발생하지 않기 때문에,

아래와 같이 조금 더 간편하게 만들어볼 수 있다. 

const values = [];
function getValueForPeriod(periodNumber) {
	return values[periodNumber] ?? 0;
}

console.log(getValueForPeriod(-10));