2

我正在使用 jsf Web 服务器上的列表来例如从网页访问数据模型。对这些列表的访问也可以从其他各种地方(Web 服务、工具)完成。

有一段代码被我返回的列表破坏了。我说的是我的开发团队中的某个人——我们是唯一使用此代码的人。我对这个函数有大约 300 个引用,它可能与性能相关,可以很好地进行修复:

该列表可以是 1 到 10'000 个条目之间的任何地方,通常我可能会有 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 );
}

但这最终会在不同项目/应用程序的几个地方中断。恕我直言,返回不可修改、添加 javadoc 并修复所有损坏的东西是最干净的。但是 :-D 这是工作。我可能需要测试大约 10 个应用程序。

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

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

这不是什么小工作——但是这方面的性能问题呢?除此之外 - 如果有人从我返回的列表中删除东西,它会默默地破坏应用程序。

那么:性能问题是什么?这是一个问题吗?

你会怎么做?

4

2 回答 2

2

你会怎么做?

如果我理解正确,这是一个用于多个应用程序的生产库。而且,不管你喜不喜欢,事实上的约定getMyObjectList()是用户可以对列表进行排序而不会出现错误或异常。

我会立即更改此方法并返回一个防御性副本:

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

您现在已经解决了有人对您的内部收藏进行分类并且您没有违反合同的问题。事实上,您甚至可以更新 Javadoc 并告诉用户他们可以对副本做任何他们想做的事情。

这可能会或可能不会导致性能问题。请记住,集合中的对象没有被复制——它们仍然被共享。您只是在创建一个新的数组列表以及跟踪对象所需的任何内部对象。

如果事实证明这些副本会导致性能问题,那么您可以考虑增强您的类以包含内部集合的只读缓存。要访问它,您必须给该方法一个新名称 - 例如getMySharedObjectList,您可以逐步更新客户端代码以根据性能需要使用此新方法。

但不要这样做。我认为这种方法特别糟糕:

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

您已经创建了 myObjects 很容易与 myObjectsMap 不同步的情况。(在有人调用 getMyObjectList之后将项目添加到 myObjectsMap 会发生什么情况?)同时,每次有人调用该方法时,您都在制作列表的副本。因此,您首先放弃了任何理论上的性能提升。

无论如何,祝你好运。希望这可以帮助。

于 2012-10-29T14:38:19.477 回答
0

如果您有能力测试应用程序,我会选择 unmodifiableList。它将在将来使您免于其他相关问题。

于 2012-10-29T14:08:07.453 回答