아이템7 다 쓴 객체 참조를 해제하라

• GC를 지원하는 언어로 개발한다고 해서 메모리 관리에 더 이상 신경 쓰지 않아도 된다고 오해할 수 있는데, 절대 사실이 아니다.

메모리 누수가 일어나는 위치는 어디인가?

public class Stack {
    private Object[] elements;
    private int size = 0;
    private static final int DEFAULT_INITIAL_CAPACITY = 16;
    public Stack() {
        elements = new Object[DEFAULT_INITIAL_CAPACITY];
    }
    public void push(Object e) {
        ensureCapacity();
        elements[size++] = e;
    }
    public Object pop() {
        if (size == 0)
            throw new EmptyStackException();
        return elements[--size];
    }
    /**
     * 원소를 위한 공간을 적어도 하나 이상 확보한다.
     * 배열 크기를 늘려야 할 때마다 대략 두 배씩 늘린다.
     */
    private void ensureCapacity() {
        if (elements.length == size)
            elements = Arrays.copyOf(elements, 2 * size + 1);
    }
}

• 위 코드는 스택이 커졌다가 줄어들었을 때 스택에서 꺼내진 객체들을 가비지 컬렉터가 회수하지 않는다. 프로그램에서 그 객체들을 더 이상 사용하지 않더라도 말이다. 이 스택이 그 객체들의 다 쓴 참조(obsolete reference)를 여전히 가지고 있기 때문이다.
• GC는 대상 객체에 하나라도 참조가 있는 경우 그 객체 뿐 아니라 그 객체가 참조하는 모든 객체를 회수해가지 못한다.
• 간단한 해법은 해당 참조를 다 썼을 때 null 처리(참조해제)하면 된다.

null 처리가 최선인가?

• null 처리를 꼼꼼히 하는 것도 이점이 있긴 하다.(잘못된 참조 NPE로 조기 발견할 수도 있다) 은 하지만 모든 객체를 다 쓰지마자 일일이 null 처리하는 것은 썩 바람직한 방법은 아니다. 그럴 필요도 없고 프로그램을 필요 이상으로 지저분하게 만들기 때문이다. 객체 참조를 null 처리하는 일은 예외적인 경우여야 한다.
• 다 쓴 참조를 해제하는 가장 좋은 방법은 그 참조를 담을 변수를 유효 범위(socpe) 밖으로 밀어내는 것이다.

핵심정리

• 메모리 누수는 겉으로 잘 드러나지 않아 시스템에 수년간 잠복하는 사례도 있다. 이런 누수는 철저한 코드 리뷰나 힙 프로파일러 같은 디버깅 도구를 동원해야만 발견되기도 한다. 그래서 이런 종류의 문제는 예방법을 익혀두는 것이 매우 중요하다.