Question

我想使下面的类不可变，即使它有一个可变的List成员。我怎样才能做到这一点？

class MyImmutableClass {
        private List<Integer> data;
        public MyImmutableClass(List<Integer> data) {
            this.data = data;
        }
        public List<Integer> getData() {
            return data;
        }
    }

下面是测试类，其main()函数修改了对象状态。

class TestMyImmutableClass{
    public static void main(String[] args) {
        List<Integer> data = new ArrayList<Integer>();
        data.add(2);
        data.add(5);
        MyImmutableClass obj = new MyImmutableClass(data);
        obj.getData().add(3);
        System.out.println("Data is; " + obj.getData());
    }
}

O / P：Data is; [2, 5, 3]

Answer 1

使用getData方法，而不是像List那样返回对{...}}的引用。

public List<Integer> getData() {
    return data;
}

您可以返回不可修改的List而不是

public List<Integer> getData() {
    return Collections.unmodifiableList(data);
}

有关详细信息，请参阅Collections.unmodifiableList ...

<强>更新

正如user949300所指出的，您还应该复制原始列表。

public MyImmutableClass(List<Integer> data) {
    this.data = new ArrayList<Integer>(data);
}

这将阻止任何有权访问原始列表的人对其进行修改。

Answer 2

在构造函数中，转到

this.data = Collections.unmodifiableList(new ArrayList(data));

为了完整性和清晰度，请将数据字段声明为final。

Answer 3

如果您使用的是Eclipse Collections，则可以将MyImmutableClass.data的类型更改为ImmutableList。

class MyImmutableClass {
    private ImmutableList<Integer> data;
    public MyImmutableClass(List<Integer> data) {
        this.data = Lists.immutable.withAll(data);
    }
    public ListIterable<Integer> getData() {
        return data;
    }
}

Eclipse Collection的ImmutableList和ListIterable类型在合同中是不可变的，这意味着它们没有add()和remove()方法。这样，从getData()的返回类型可以清楚地看出，返回的列表无法更改。

ImmutableList和MutableList都扩展ListIterable，因此您可以在不更改API的情况下将实施从ImmutableList更改为不可修改的列表。

class MyImmutableClass {
    private MutableList<Integer> data;
    public MyImmutableClass(List<Integer> data) {
        this.data = Lists.mutable.withAll(data);
    }
    public ListIterable<Integer> getData() {
        return data.asUnmodifiable();
    }
}

注意：我是Eclipse集合的提交者。

尽管有一个可变成员，我怎么能让这个类不可变？

3 个答案: