你可能偶尔需要编写退化类,目的只是为了集中实例域:
// Degenerate classes like this should not be public!
class Point {
public double x;
public double y;
}
标准写法是:
// Encapsulation of data by accessor and mutators
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类暴露了其数据域,则要想在将来改变内部表示是不可能的,因为他的客户端代码可能已经遍布各处了。
然而,如果类是package-private或是private嵌套类,那么把数据域暴露出去并没有本质的错误——假设这些数据域充分描述了该类提供的抽象。无论是在类定义中,还是在客户端代码中,这种方法相对于访问方法更不会产生视觉混乱。虽然客户端代码与类的内部表示紧密相连,但这些代码被限定在同一个包中。如果需要改变内部数据表示,你不必修改包外的任何代码。如果是private嵌套类,则甚至不需要修改类外的任何代码。
除非修改其API,否则你不能修改其内部数据表示,而且当别人读取该字段时 你不能采取任何附加行动,但是你可以强加约束条件(——因为只有你自己可以set该字段)
补救办法: public域加入约束条件
// public class with exposed immutable fields - questionable
public final class Time {
private static final int HOUR_PER_DAY = 24;
private static final int MINUTES_PER_HOUR = 60;
public final int hour;
public final int minute;
public Time() {
//强加约束条件
if (hour < 0 || hour >= HOUR_PER_DAY) {
throw new IllegalArgumentException();
if (minute < 0 || minute >= MINUTES_PER_HOUR) {
throw new IllegalArgumentException();
this.hour = hour;
this.minute = minute;
}
}
Summary
public类永远不要暴露其可变的字段。而public类暴露其不可变字段虽然是有问题,但危害性要小一些。然而,package-private或者private的嵌套类暴露其字段则是可行的,无论该字段是可变还是非可变。