Java 有人破坏了我对列表的排序-现在选择哪种方法：返回不可修改的列表还是一起返回新列表？

我使用JSFWeb服务器上的列表，例如从网页访问数据模型。但是，对这些列表的访问也可以从各种其他地方（web服务、工具）完成

有一段代码被我返回的列表打破了。我正在谈论我的开发团队中的某个人——我们是唯一使用此代码的人。我有大约300个关于这个函数的引用，很好地进行修复可能与性能有关：

列表可以是1-10000个条目，通常我会有10-100个这样的列表。事实上，我可能经常会有20个左右的列表，每个列表有8个条目——所以没什么大不了的。但有时我可以拥有更多

顺便说一句，我说的是这样一个函数：

public List<MyObject> getMyObjectList() {
    if (this.myObjects== null) {
        myObjects = new ArrayList<MyObject>(myObjectsMap.values());
    }
    return myObjects;
}

    public List<MyObject> getMyObjectList() {
        if (this.myObjects== null) {
            myObjects = new ArrayList<MyObject>(myObjectsMap.values());
        }
        return Collections.unmodifiableList(myObjects );
}

public List getMyObjectList（）{
if（this.myObjects==null）{
myObjects=newArrayList（myObjectsMap.values（））；
}
返回对象；
}

现在我当然可以这样做了：

public List<MyObject> getMyObjectList() {
    if (this.myObjects== null) {
        myObjects = new ArrayList<MyObject>(myObjectsMap.values());
    }
    return myObjects;
}

    public List<MyObject> getMyObjectList() {
        if (this.myObjects== null) {
            myObjects = new ArrayList<MyObject>(myObjectsMap.values());
        }
        return Collections.unmodifiableList(myObjects );
}

public List getMyObjectList（）{
if（this.myObjects==null）{
myObjects=newArrayList（myObjectsMap.values（））；
}
返回集合。不可修改列表（MyObject）；
}

但这种情况最终会在不同项目/应用程序的几个地方出现。 返回不可修改、添加javadoc并修复所有损坏的内容将是最干净的。但这是工作。我可能需要测试大约10个应用程序

另一方面，我可以返回一个新列表，例如

public List<MyObject> getMyObjectList() {
    return new ArrayList<MyObject>(myObjectsMap.values());
}

public List getMyObjectList（）{
返回新的ArrayList（myObjectsMap.values（））；
}

这不是一件简单的工作，但是这会带来什么性能问题呢？除此之外，如果有人正在从我返回的列表中删除内容，它将自动中断应用程序

因此： 性能问题是什么？这是个问题吗

---

你会怎么做？

如果你有能力测试这些应用程序，我会选择不可修改列表。它将使您在将来避免其他相关问题

你会怎么做

如果我没有弄错的话，这是一个用于多个应用程序的产品库。不管你喜欢与否，

getMyObjectList（）

的实际约定是用户可以对列表进行排序，而不会出现错误或异常

我会立即更改此方法并返回防御副本：

// good idea
public List<MyObject> getMyObjectList() {
    return new ArrayList<MyObject>(myObjectsMap.values());
}

您已经创建了一种情况，使得MyObject很容易与myObjectsMap不同步。（当有人调用getMyObjectList后将一个项添加到myObjectsMap时会发生什么情况？）同时，每次有人调用该方法时，您都会复制该列表。所以你就放弃了你最初获得的理论上的性能提升

---

不管怎样，祝你好运。希望这能有所帮助。

您应该在典型负载下对其进行测试，以获得明确的答案。但是创建100个新的ArrayList，每个ArrayList的引用少于10k，可能不会花费超过几毫秒的时间。所以，除非您的方法每秒被调用多次，否则您可能不会注意到差异。我认为答案应该更多地涉及到如果我们更改列表（或）使其不可修改会产生什么影响，它在您的应用程序中是如何使用的？性能只是成为问题时的问题。意思是你甚至应该在有人抱怨之前考虑性能。@ GijsOvervliet，你的意思是我不应该这么想吗？@托斯坎，我的意思是，你可以轻松地创建一个性能测试，在那里你拷贝很多包含10000个对象的数组，并看平均需要花费多长时间，你可能会发现它并不多。请确认。我正在对

myObjectsMap.put（）

或

remove（）

进行无效处理-我将运行一些测试并从那里开始，谢谢。您的方法的唯一问题是返回的列表不再是原始列表。如果某个应用程序正在运行，比如-

getMyObjectList（）.removeAll（）

实际上什么都不会发生，但它仍然被破坏。有问题吗？这是一个好处！你的收藏是安全的。如果希望允许removeAll，请向类中添加一个removeAll方法，该方法只在映射中调用removeAll。您的类保持对集合的完全控制。