多个锁下java的对象可见性

时间:2014-06-28 17:30:05

标签: java multithreading concurrency visibility locks

这是我面临的问题。在运行下面的代码时,我有时会在打印B的内容时收到过时的数据。我无法理解为什么会发生这种情况,因为更新和接收B的内容受到B锁的保护。

请注意,在更新到B期间,A和B的锁都被保留。 访问B的内容时,只保留一个锁。

另请注意,B.updated()未同步,但我不认为这是问题,因为我确保更新的计数器设置为2,然后才能访问B的内容。

class B {
  Object data
  int updated = 0

  // get and set protected by B's monitor
  synchronized Object getData() {
    return data;
  }
  synchronized setData(Object d) {
    data = d;
    updated += 1;
  }
  // deliberately not synchronized
  int updated() { return updated; }
}

class A {
    B b
    A() {
        new Thread t = new Thread(new UpdaterThread());
        t.start();
    }

   // when B is updated both A and B monitors are held
    synchronized String setB(Object data) {
      b.setData(data);
   }

   class UpdateThread implements Runnable {
     public void run() {
       // updating B to new Value
       setB("B2");
     }
   }
}

public static void main(String[] _) {
   B b = new B();  
   b.setData("B1");
   A a = new A();
   while (b.updated() == 1) {
     // wait for updated to reach 2
   } 
   println(b.getData());  // prints "B1" sometimes, but very rarely.
}

1 个答案:

答案 0 :(得分:5)

您希望代码为sequentially consistent。但是,如果代码中没有数据竞争,则Java仅保证顺序一致性

  

另请注意,B.updated()未同步,但我不认为这是问题,因为我确保更新的计数器设置为2,然后才能访问B的内容。

这就是问题所在。更新的变量上有数据竞争,因为一个线程写入它,另一个线程从中读取而没有任何同步。

如果您创建变量volatile,代码将起作用,因为volatile变量上没有数据竞赛