@Singleton bean 是否应该返回内部 List<> 成员？

Question

这可能是一个非常简单的问题，但我并不完全清楚。

我有一个豆子如下：

@Singleton
@Startup
@Lock(READ)
public class SomeDataBean {
  List<Foo> foos;

  @PostConstruct
  public void init() {
    // Build foos;
  }

  public List<Foo> getFoos() {
    return foos;
  }

  @Lock(WRITE)
  public void modifyFoos() {
    // This could be potentially called too
  }
}

随后，这个 bean 被注入（使用 CDI - 环境是 Wildfly 8.2），注入到各种其他 bean（请求/会话/视图等）中。现在他们都访问了这个 Foos 列表。我的问题是，当另一个 bean 可能调用 modifyFoos() 时，按原样返回列表是否安全。或者这是永远不会发生并且由容器保证的事情？

我已经阅读了有关容器管理事务等的一些文档，但尚不完全清楚它在这种情况下是如何工作的。例如，用并发容器替换列表是否更好？

Answer 1

不，这不安全，因为一个线程可以读取列表，而另一个线程修改它。

1. 返回列表的副本（如果有很多读取，则效率不高）。
2. 使用copy-on-write 列表。
3. 将列表的新实例存储在modifyFoos 中。