[이펙티브 자바] Item16. public 클래스에서는 public 필드가 아닌 메서즈 접근자를 사용하라.

 

[이펙티브 자바] Item16. public 클래스에서는 public 필드가 아닌 메서드 접근자를 사용하라. 

 

public class Point
{
   public double x;
   public double y;
   
   public static void main(String[] args)
   {
      Point point = new Point();
      point.x = 10;
      point.y = 20;
   }
}

위에 예시처럼 public 클래스에서 public 필드를 통해 사용한다면 데이터 필드에 직접 접근할 수 있어 캡슐화의 이점을 제공하지 못한다. 

[이펙티브 자바] Item15. 클래스와 멤버의 접근 권한을 최소화하라. (tistory.com)

[이펙티브 자바] Item15. 클래스와 멤버의 접근 권한을 최소화하라.

-> 캡슐화의 이점은 여기서 확인할 수 있다. 

 

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

 

 

public 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 setY(double y)
   {
      this.y = y;
   }
   
   public void setX(double x)
   {
      // 부가 작업 가능
      this.x = x;
   }
}

따라서 필드 접근을 public 필드가 아닌 메서드 접근자를 사용해야 한다. 필드를 모두 private로 변경하고 public 접근자(getter) 추가!! 

 

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

public 클래스가 필드를 공개했더라면 이를 사용하는 클라이언트가 생겨서 내부 표현 방식을 마음대로 바꿀 수 없게 되기 때문이다. 

 

 

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

클라이언트 코드가 이 클래스 내부 표현에 묶이기는 하겠지만, 클라이언트도 어차피 이 클래스를 포함하는 패키지 안에서만 동작하기 때문이다. 하지만 강의를 들을 때 강사님은 package-private 클래스라고 하더라도 private 로 필드를 접근권한을 주고, 메서드 접근자를 사용하는 것이 더 좋을 것이라고 했고, 내 생각에도 메서드 접근자를 사용하는 것이 더 좋겠다고 생각했다. 그리고 그렇게 사용하고 있었기도 하고.. ! 

 

 

 

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

대표적인 예로는 java.awt.package 패키지의 Dimension 클래스가 있다. 

Dimension 클래스를 보면 width, height를 직접 노출하고 있고, 이로 인한 심각한 성능 문제는 오늘날까지도 해결되지 못했다고 한다.... 

 

public static void main(String[] args)
{
   Button button = new Button("Hello Button");
   button.setBounds(0, 0, 20, 10);
   
   Dimension size = button.getSize();
   System.out.println("size.height = " + size.height);
   System.out.println("size.height = " + size.height);
   
   doSomething(size);
}

이렇게 size.height, size.width에 직접 접근할 수 있는데.. 예를 들어 doSomethng()이라는 메서드를 수행하고 난 후 그 다음 width와 height의 크기가 언제 어디서 변경되었을지 예측하기 힘들어지며 코드가 불안해지고,

private static void doSomething(Dimension size)
{
   // 값을 copy 해서 사용해야 한다. -> 근데 한번으로 성능 저하가 일어나지는 않겠지만..이렇게 copy하는 자체가 불편하다!!
   Dimension dimension = new Dimension();
   dimension.width = size.width;
   dimension.height = size.height;
}

따라서 위와 같이 값을 copy해서 사용해야 하는데 이게 불필요한 인스턴스를 만드는 작업이기 때문에 한번 정도 copy해서 사용하는건 성능 문제로 어이지지는 않겠지만.. 수백만번? 이렇게 호출하게 된다면... 성능상의 심각한 문제를 일으킬 수 있다. 

근데 성능 문제 뿐만 아니라 이렇게 새로운 인스턴스를 생성하고.. 복사해서 사용하는것 자체가 불편하다!! 

 

 

 

public class Time
{
   private static final int HOURS_PER_DAY = 24;
   private static final int MINUTES_PER_HOUR = 60;
   
   // public 클래스의 필드가 불변이라면 직접 노출할 때 단점이 조금 줄어들지만 이게 좋은 생각은 아니다.
   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;
   }
}

hour, minute처럼 public 클래스의 필드가 불변이라면 직접 노출할 때의 단점이 조금 줄어들지만 이렇게 사용하는것이 여전히 좋은 생각은 아니다. 

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

 

 

 

 

📚 정리 

 

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

불변 필드라면 노출해도 덜 위험하지만 완전히 안심할 수는 없다.