5

我有一个显示部门文档的项目。我将所有文档（从数据库中获取）存储在一个静态 arrayList 中。每隔 X 小时，我就会根据数据库中的新文档（如果有）重建该 arrayList。还有一个静态变量可以控制是否重建该数组，在执行重建任务的方法中设置和取消设置。每个访问服务器的 Web 浏览器都会创建此类的实例，但 doc arrayList 和该控制变量在所有类实例之间共享。

Find-Bugs 工具抱怨“从实例方法 someClassMethod 写入静态字段 someArrayName 和 someVariableName”。似乎这不是一件好事（让类实例方法写入静态字段）。有没有人有很好的建议如何解决这个问题？谢谢。

4

5 回答 5

7

根据FindBugs 错误描述：

ST：从实例方法写入静态字段（ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD）

此实例方法写入静态字段。如果正在操纵多个实例，这很难纠正，而且通常是不好的做法。

除了并发问题，这意味着 JVM 中的所有实例都在访问相同的数据，并且不允许两组单独的实例。如果您有一个单例“管理器”对象并将其作为构造函数参数或至少作为setManager()方法参数传递给每个实例，那就更好了。

至于并发问题：如果你必须使用静态字段，你的静态字段应该是最终的；显式同步很困难。（如果您要初始化非最终静态字段，还有一些棘手的方面，超出了我对 Java 的了解，但我想我已经在 J​​ava Puzzlers 书中看到了它们。）至少有三种处理方法（警告，未经测试的代码如下，使用前先检查）：

  1. 使用线程安全的集合，例如Collections.synchronizedList包裹在不以任何其他方式访问的列表周围。

    static final List<Item> items = createThreadSafeCollection();


static List<Item> createThreadSafeCollection()
{
   return Collections.synchronizedList(new ArrayList());
}

    然后当您从一个实例替换此集合时：

    List<Item> newItems = getNewListFromSomewhere();
items.clear();
items.add(newItems);

    这样做的问题是，如果两个实例同时执行此序列，您可能会得到：

    实例1：items.clear(); Instance2: items.clear(); Instance1: items.addAll(newItems); Instance2: items.addAll(newItems);

    并获得一个不满足所需类不变量的列表，即静态列表中有两组 newItems。因此，如果您将整个列表作为一个步骤清除，然后将项目作为第二个步骤添加，则此方法不起作用。（但是，如果您的实例只需要添加一个项目，items.add(newItem)则可以安全地从每个实例中使用。）

  2. 同步对集合的访问。

    您需要一个显式的同步机制。同步方法不起作用，因为它们在“this”上同步，这在实例之间并不常见。你可以使用：

    static final private Object lock = new Object();
static volatile private List<Item> list;
// technically "list" doesn't need to be final if you
// make sure you synchronize properly around unit operations.


static void setList(List<Item> newList)
{
  synchronized(lock)
  {
      list = newList;
  }
}

  3. 使用原子参考

    static final private AtomicReference<List<Item>> list;


static void setList(List<Item> newList)
{
  list.set(newList);
}
于 2010-12-16T03:43:25.060 回答
1

如果我正确理解您从 Find Bugs 发布的消息，这只是一个警告。

如果要隐藏警告，请从静态方法进行修改。Find Bugs 会警告您，因为这通常是一个错误。程序员认为他们正在改变一些实例状态，但实际上他们正在改变一些影响每个实例的状态。

于 2010-12-16T03:31:38.993 回答
0

您不需要每次都删除列表。如上所述，您将不得不处理多个线程，但您可以创建一次 ArrayList，然后使用 clear() 和 addAll() 方法进行擦除和重新填充。FindBugs 应该对此非常满意，因为您没有设置静态。

伙计们-如果这种技术有任何问题，请随时加入:-)

第二个想法是通过休眠从数据库中驱动东西。所以不要维护一个列表，hibernate 有内置的缓存，所以它几乎一样快。如果您在数据库级别更新数据（这意味着 hibernate 不知道），那么您可以告诉 hibernate 清除它的缓存并在下一次查询时从数据库中刷新。

于 2010-12-16T03:10:41.743 回答
0

使用单例设计模式是一种方法。您只能拥有一个包含所需值的对象实例，并通过全局属性访问该实例。优点是，如果您以后想要更多实例，则对预先存在的代码的修改较少（因为您没有将静态字段更改为实例字段）。

于 2010-12-16T02:52:39.893 回答
-1

你不想这样做。每个请求都在自己的线程中运行。如果在浏览器操作上执行的代码修改了列表，那么两个请求可能同时修改列表，并破坏数据。这就是为什么从非静态上下文访问静态资源不是一个好主意的原因，并且可能是您的工具警告您的原因。

看这个

http://download.oracle.com/javase/6/docs/api/index.html?java/util/concurrent/package-summary.html

特别是关于 ArrayList 如何不同步的部分。另请注意，我提到的段落有一个解决方案，特别是

List list = Collections.synchronizedList(new ArrayList(...));

这是一种方法。但这仍然不是一个好主意，即因为它可能很慢。如果它不是商业级应用程序，并且您的交易量不高，那么您可能会通过不改进它来获得。如果这是每天只被点击几次的应用程序类型，您可以忽略警告，但要了解如果两个请求相互冲突，可能会发生不好的事情。

一个更好的解决方案：既然你有数据库，我会在你需要的时候从数据库中获取信息，即当请求进来时。你可以使用一些缓存技术来提高性能。

我不喜欢单例模式想法的原因是，即使它使警告消失，它本身也不能解决基本的同步问题。但是，有线程安全的http://en.wikipedia.org/wiki/Singleton_pattern#Traditional_simple_way_using_synchronization，在这种情况下可能有效。

于 2010-12-16T02:55:56.147 回答