Getter 和 setter findbugs 可能会暴露内部表示 [重复]

Question

我刚开始在我的 IntelliJ 项目中使用 Firebug。

private byte[] image;

public byte[] getImage() {
    return image;
}

public void setImage(byte[] image) {
    this.image = image;
}

Firebug 给了我错误：

May expose internal representation by incorporating reference to mutable object

我真的不明白这个错误是什么意思以及如何减轻它。

编辑

我的构造函数也出现此错误：

public Design(byte[] image){
    this.image = image;
}

感谢帖子，我修复了 Getter 上的错误并理解了它的含义，但是构造函数 en setter 上的错误对我来说不是很清楚。

Answer 1

通过调用getImage()，您将可以访问私有字段image。然后，您可以以任何您喜欢的方式在外部对其进行修改，这可能会给您带来不想要的结果。

您可以通过让 getter 返回底层数组的 副本 来规避它。这样您就可以确保数据的完整性。

所以不是

public byte[] getImage() {
    return image;
}

你可以使用java.util.Arrays.copyOf(byte[] original, int newLength)

public byte[] getImage() {
    return Arrays.copyOf(image, image.length);
}

我认为这应该消除该警告并且使用起来更安全，因为您的数据现在受到保护，不会被外部操纵。