이펙티브 자바
아이템 16. public classes에는 public fields를 사용하지 말고 접근 메소드를 사용해라.
1
2
3
4
5
6
7
8
9
|
package com.github.sejoung.codetest.accessor;
public class Point {
public double x;
public double y;
}
|
1
2
3
4
5
6
7
8
9
10
11
12
13
14
|
package com.github.sejoung.codetest.accessor;
public class PointTest {
public static void main(String[] args) {
Point p = new Point();
p.x = 1;
p.x = 2;
System.out.println(p.x+" "+p.y);
}
}
|
위와 같은 클래스는 public fields를 통해서 접근이 가능하다 하지만 저런식으로 작성하면 캡슐화의 이점을 살리지 못하고
이렇게 되면 클라이언트 코드를 수정하지 않고 해당 정보를 수정할수 없다
그러면 어떻게 해야 되냐면
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
|
package com.github.sejoung.codetest.accessor;
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 setX(double x) { this.x = x; }
public void setY(double y) { this.y = y; }
}
|
1
2
3
4
5
6
7
8
9
10
11
12
13
14
|
package com.github.sejoung.codetest.accessor;
public class PointTest {
public static void main(String[] args) {
Point p = new Point(10,9);
p.setX(1);
p.setY(2);
System.out.println(p.getX()+" "+p.getY());
}
}
|
바꾼 코드에서는 x와 y를 private으로 접근을 제한했고 getter와 setter를 통해서 접근및 수정을 가능하게 해
외부 클라이언트 수정할 필요 없이 로직 수적이 가능하다.
하지만 package-private 클래스나 private 중첩클래스(Nested-Class)라면 데이터 필드를 노출해도 하등에 문제가 없다
클래스가 표현하고자 하는 추상개념을 잘표현 하면 된다.
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
|
package com.github.sejoung.codetest.accessor;
public class NestedClass {
public void Test(){
A a = new A();
a.a = "1";
}
private class A{
public String a;
}
}
|
위에서 중첩클래스인 A는 NestedClass에서만 접근이 가능하기 때문에 추상개념을 잘구현하면 문제가 없다는것이다.
1
2
3
4
5
6
7
8
9
10
11
|
package com.github.sejoung.codetest.accessor;
class B {
public String a;
}
|
위에 package-private 클래스에서는 같은 패키지 내에서만 해당 클래스를 접근하기 때문에 영향이 퍼지는것을 최소화 할수 있다
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
|
package com.github.sejoung.codetest.accessor;
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;
}
}
|
그러면 위에 클래스처럼 불편 필드는 노출해도 괜찮을까 가변필드보다는 덜 위험하지만 쓸때 없이 불변필드를 노출하면 안된다.
참조
