자바로 배우는 리팩토링 입문 - 유키 히로시 리뷰 (1)

코드에서 나는 악취를 나타내는 말

• 겹치잖아!
• 너무 길어!
• 너무 많아!
• 이름이 안 맞잖아!
• 너무 공개적이잖아!
• 객체 지향답지 않아!

 

---

리팩토링 방법

1. 매직 넘버를 기호 상수로 치환

if (100 < input.length()) {
	...
}

↓

public class Something {
	public static final int MAX_INPUT_LENGTH = 100;
	...
}

...

if (Something.MAX_INPUT_LENGTH < input.length()) {
	...
}

 

 

 

2. 제어 플래그 삭제

public class FindInt {
	public static boolean find(int[] data, int target) {
    	boolean flag = false;
        for (int i = 0; i < data.length && !flag; i++) {
        	if (data[i] == target) {
            	flag = true;
            }
        }
        return flag;
    }
}

↓

public class FindInt {
	public static boolean find(int[] data, int target) {
        for (int i = 0; i < data.length; i++) {
        	if (data[i] == target) {
            	return true;
            }
        }
        return false;
    }
}

 

변수 이름을 짓기 힘들 때, 추천하는 flag 변수명

• initialized: 초기화가 끝났음
• debug: 디버깅 여부
• error: 에러 발생했음
• done: 처리가 완료됨
• interrupted: 인터럽트가 발생했음
• recurse: 재귀 적용 여부

 

Assertion(어서션) 도입

• Assertion: 프로그래머의 의도를 확실히 밝히면서도 실행 시 조건이 반드시 성립함을 보장한다

void method(int value) {
	// 여기서 value는 참이어야 함ㅋ
	...
}

↓

void method(int value) {
	assert value > 0;
    ...
}

- value > 0 인 경우, 아무 일도 발생하지 않음

- value > 0 이 아닌 경우, java.lang.AssertionError 예외 발생

 

 

'프로그램에서 여기는 절대로 실행되지 않을 것' 을 표시하고 싶다면,

assert false: "Never reached";

 

 

 

4. 널 객체 도입

• 아무것도 처리하지 않는 널 객체를 도입해 '이 변수는 현재 null인가?'라는 확인을 생략할 수 있다.

if (_name != null) {
	_name.display();
}

↓

_name.display();

 

[ 도입 방법 ]

1) 널 객체 클래스 작성

public class NullSomething extends Something {
	
}

 

2) isNull 메서드 작성

public class Something {
	...
    public boolean isNull() { return false; }
}

 

3) isNull 오버라이드 - true 반환

public class NullSomething extneds Something {
	@Override
    public boolean isNull() {
    	return true;
    }
}

 이제 null을 사용하는 대신 NullSomething 인스턴스로 치환해 코드를 리팩토링할 수 있다.

 

 

 

 

[널 객체 도입 시 사용하면 좋은 패턴]

팩토리 메서드 패턴

•  Null 객체를 생성할 때 널 객체 생성용 메서드를 따로 만들어 new NullSomething() 표현식을 감춰보도록 하자.

public static Something newNull() {
	return new NullSomething();
}

 

• 패턴 사용 방법

// 기존 사용 방법
new NullSomething();

// 팩토리 메서드 사용 방법
Something.newNull();

 

싱글톤 패턴

•  어차피 널 객체는 아무것도 안하니까.. 싱글톤 패턴을 사용하는게 좋다.

 

 

중첩 클래스

public class Something {
	...
    
    private static class NullSomthing extends Something {
    
    ...
    
    }
    
}

 중첩 클래스를 사용하면 널 객체를 도입할 때마다 클래스 개수가 늘어난다는 단점을 보완해준다.

 

 

 

→ 그러나 무엇이든 과유불급.. 모든 null 확인을 널 객체로 바꿀 필요는 없다. 알아서 잘 유도리있게 사용하자.