[Effective Java]16.public 클래스에서는 public 필드가 아닌 접근자 메서드를 사용하라

이따금 인스턴스 필드들을 모아놓는 일 외에는 아무 목적도 없는 퇴보한 클래스를 작성하려 할 때가 있다.

// 코드 16-1 이처럼 퇴보한 클래스는 public 이어서는 안된다!
class Point {
    public double x;
    public double y;
}

 

이런 클래스는 데이터 필드에 직접 접근할 수 있으니 캡슐화의 이점을 제공하지 못한다.

API를 수정하지 않고는 내부 표현을 바꿀 수 없고, 불변식을 보장할 수 없으며, 외부에서 필드에 접근할 때 부수 작업을 수행할 수도 없다.

철저한 객체 지향 프로그래머는 이런 클래스를 상당히 싫어해서 필드들을 모두 private으로 바꾸고 public 접근자(getter)를 추가한다.

 

// 코드 16-2 접근자와 변경자(mutator) 메서드를 활용해 데이터를 캡슐화한다. (102쪽)
class Point {
    private double x;
    private double y;

    public Point(double x, double y) {
        this.x = x;
        this.y = y;
    }

    public double getX() { return x; }
    public double getY() { return y; }

    public void setX(double x) { this.x = x; }
    public void setY(double y) { this.y = y; }
}

 

public 클래스에서라면 이 방식이 확실히 맞다.

패키지 바깥에서 접근할 수 있는 클래스라면 접근자를 제공함으로써 클래스 내부 표현 방식을 언제든 바꿀 수 있는 유연성을 얻을 수 있다.

public 클래스가 필드를 공개하면 이를 사용하는 클라이언트가 생겨날 것이므로 내부 표현 방식을 마음대로 바꿀 수 없게 된다.

 

하지만 package-private 클래스 혹은 private 중첩 클래스라면 데이터 필드를 노출한다 해도 하등의 문제가 없다.

그 클래스가 표현하려는 추상 개념만 올바르게 표현해주면 된다.

이 방식은 클래스 선언 면에서나 이를 사용하는 클라이언트 코드 면에서나 접근자 방식보다 훨씬 깔끔하다. - 1

클라이언트 코드가 이 클래스 내부 표현에 묶이기는 하나, 클라이언트도 어차피 이 클래스를 포함하는 패키지 안에서만 동작하는 코드일 뿐이다. - 2

따라서 패키지 바깥 코드는 전혀 손대지 않고도 데이터 표현 방식을 바꿀 수 있다.

private 중첩 클래스의 경우라면 수정 범위가 더 좁아져서 이 클래스를 포함하는 외부 클래스까지로 제한된다. - 3

 

ex)

package-private 상황을 예로 들어보겠다.

class Product {
    public String productName;
    public int cost;
}

class Client {
    private List<Product> products;
    private int balance;

    public Client(int balance) {
        this.balance = balance;
        products = new ArrayList<>();
    }

    public void buyProduct(Product product) {
        if(balance < product.cost) {
            throw new IllegalArgumentException();
        }
        balance -= product.cost;
        products.add(product);
    }
}

 

1 - Getter, Setter가 없으니까 깔끔하다고 볼 수 있다.

2 - Product가 cost라는 필드 이름을 바꾸고싶다고 한다면 Client의 필드 이름을 바꿔야 하므로 내부표현으로 묶이는 것이고 어차피 내부의 클래스만 수정하면 된다.

3 - private 중첩 클래스의 필드는 어차피 그 클래스를 포함하는 톱레벨 클래스에서만 접근가능하기 때문에 package-private 보다 더 수정할게 없어진다.

 

 

자바 플랫폼 라이브러리에도 public 클래스의 필드를 직접 노출하지 말라는 규칙을 어기는 사례가 종종 있다.

대표적인 예가 java.awt.package 패키지의 Point와 Dimension 클래스다.

이 클래스들을 흉내 내지 말고, 타산지석으로 삼길 바란다.

아이템 67에서 설명하듯 내부를 노출한 Dimension 클래스의 심각한 성능 문제는 오늘날까지도 해결되지 못했다.

 

public 클래스의 필드가 불변이라면 직접 노출할 때의 단점이 조금은 줄어들지만, 여전히 좋은 생각이 아니다.

API를 변경하지 않고는 표현 방식을 바꿀 수 없고, 필드를 읽을 때 부수 작업을 수행할 수 없다는 단점은 여전하다.

단, 불변식은 보장할 수 있게 된다.

예컨대 다음 클래스는 각 인스턴스가 유효한 시간을 표현함을 보장한다.

 

// 코드 16-3 불변 필드를 노출한 public 클래스 - 과연 좋은가? (103-104쪽)
public final class Time {
    private static final int HOURS_PER_DAY    = 24;
    private static final int MINUTES_PER_HOUR = 60;

    public final int hour;
    public final int minute;

    public Time(int hour, int minute) {
        if (hour < 0 || hour >= HOURS_PER_DAY)
            throw new IllegalArgumentException("Hour: " + hour);
        if (minute < 0 || minute >= MINUTES_PER_HOUR)
            throw new IllegalArgumentException("Min: " + minute);
        this.hour = hour;
        this.minute = minute;
    }

    // 나머지 코드 생략
}

 

핵심 정리

public 클래스는 절대 가변 필드를 직접 노출해서는 안 된다.

불변 필드라면 노출해도 덜 위험하지만 완전히 안심할 수는 없다. 하지만 package-private 클래스나 private 중첩 클래스에서는 종종 (불변이든 가변이든) 필드를 노출하는 편이 나을 때도 있다.

 

 

출처

https://github.com/Java-Bom/ReadingRecord/issues