我的设计中有一个有听众的对象.这些侦听器会在某个事件上被触发,该事件每秒可能发生多达一百次.

我做的是这样的：

private void notifyListeners(ObjectEvent o) {
    synchronized (this.listeners) {
        for (ObjectListener l: this.listeners)
            l.eventFired(o);
    }
}

这里的问题是有人可以实现一个eventFired方法,然后转向并等待同步一个对象,该对象由另一个试图添加或删除监听器并等待同步(this.listeners)行的线程持有.

所以,我这样修改了notifyListeners方法：

private ObjectListener[] getObjectListeners() {
    synchronized (this.listeners) {
        return this.listeners.toArray(new ObjectListener[this.listeners.size()]);
    }
}

private void notifyListeners(ObjectEvent o) {
    ObjectListener[] listeners = this.getObjectListeners();
    for (ObjectListener l: listeners)
        l.eventFired(o);
}

我担心每次触发对象时创建此数组的影响,以及这将对应用程序的内存使用量产生的影响.

所以我在这里问是否有更好的方法.不过,我可能刚刚发现了它.每次添加或删除一个监听器时都很容易创建ObjectListener [],然后只是在每次事件触发时都没有同步地遍历它.我现在要做出改变,然后看看这里是否有更好的想法.

解决方法:

请改用CopyOnWriteArrayList.观察结果是,听众列表通常比修改后更频繁地进行检查.