为什么这段代码不抛出NullPointerException？

Question

我想了解为什么一段代码不会抛出NullPointerException。

请考虑以下代码：

public class Agent {
  public List files = new ArrayList();

  public void deliver() {
    if( files != null && files.iterator().hasNext() ) {
      File file = (File)files.iterator().next();
    }

    files = new ArrayList();
  }
}

deliver方法被重复调用，同时以下代码在单独的线程中运行：

  public void run() {
    agent.files = null;
  }

只有一个agent实例。

从不引发NullPointerException。

但是，当Deliver方法暂停时，即使暂停0毫秒，也会按预期引发NullPointerException：

  public void deliver() {
    if( files != null ) {
      Thread.currentThread().sleep( 0 );

      if( files.iterator().hasNext() ) {
        File file = (File)files.iterator().next();
      }
    }

    files = new ArrayList();
  }

我的理解是，在理论上，在检查文件==null和调用文件.Iterator().hasNext()之间存在竞争条件。在实践中，如果不引入暂停（即从后续方法调用中拆分空检查），我就无法触发竞态条件。

为什么第一个deliver方法在相同语句中合并了null检查和用法时不抛出异常？

Answer 1

两件事：

>

thread.sleep（0）仍然停止执行（可能超过0毫秒）。基本上，即使是0睡眠也会导致线程上的执行短暂停止，然后重新启动。这给了另一个线程一个运行和完成的机会，这就是为什么您能够触发竞态条件。

文件应该是volatile，否则允许JVM以这样一种方式进行优化，即您可能永远不会注意到它在改变值，因为它认为它不需要维护线程之间的一致性。