（Java参考情况）我应该按照findbugs工具的要求去做吗？

Question

我在我的项目上运行了Findbug工具，它发现了18个此类问题：

存储对可变对象的引用-

所以我有一个类，构造函数接受Object类型的数组，并将其分配给一个私有类成员变量。下面是一个例子：

public Class HtmlCellsProcessing extends HtmlTableProcessing
{
    private Object[] htmlCells;

    public HtmlCellsProcessing(Object[] htmlCells)
    {
        this.htmlCells = htmlCells;
    }
}

以下是关于警告的进一步解释：

此代码将对外部可变对象的引用存储到对象的内部表示中。

他们给我的建议非常明显，但如果数组的大小非常大，如果我将其值复制到成员变量数组中，应用程序将占用两倍的内存，会发生什么。

在我拥有大量数据的情况下，我应该怎么做？我应该把它作为参考还是一直复制？

Answer 1

如果同时需要封装和性能，典型的解决方案是将引用传递给不可变对象。

因此，与其直接传递一个巨大的数组，不如将其封装在一个不允许数组修改的对象中：

final class ArraySnapshot {
    final Object[] array;

    ArraySnapshot(Object[] array) {
        this.array = Arrays.copyOf(array);
    }

    // methods to read from the array
}

这个对象现在可以廉价地传递，但是由于它是不可变的，封装是可以确保的。

当然，这个想法并不新鲜：这就是String为char[]所做的。

Answer 2

我想让你试试使用番石榴库中的不可变列表。看http://code.google.com/p/guava-libraries/wiki/ImmutableCollectionsExplained

Answer 3

视情况而定。你有多种顾虑，包括空间、时间和正确性。

防御副本可以帮助您保证列表项不会在持有数组的类不知道的情况下更改。但是这需要O（n）时间和空间。

对于非常大的阵列，您可能会发现防御拷贝在空间和时间上的成本对应用程序有害。如果通过访问数组来控制所有代码，那么在没有防御副本的情况下保证正确性，并抑制该类上的FindBugs警告可能是合理的。