同期メソッドのフラグにアクセスしますか？

Question

私は大学のクラスからいくつかのpdfの周りを探していたと私は、このコードの一部に気づいた：

public class MyMemoryElements { 
    private String x = ""; 

    public MyMemoryElements(String message){ 
     this.x = message; 
    } 

    @Override 
    public boolean equals(Object o){ 
     if(o instanceof MyMemoryElements){ 
      MyMemoryElements tmp = (MyMemoryElements)o; 
      if(tmp.toString().equalsIgnoreCase(this.x)) 
       return true; 
      return false; 
     } 
     return false; 

    } 

    @Override 
    public String toString(){ 
     return this.x; 
    } 
} 

とメインコード：

public class MainMemory { 
    private Vector<MyMemoryElements> storage; 
    private boolean canAccess = true; 
    private int counter = -1; 

    public MainMemory(){ 
     storage = new Vector<MyMemoryElements>(); 
    } 

    public synchronized MyMemoryElements take(String s) { 
     System.out.print("Method take has been invoked by "+s+". Element is:"); 
     while (!canAccess || storage.size()==counter+1) { 
      try { 
       wait(); 
      } catch (InterruptedException e) {} 
     } 
     canAccess = false; 
     counter++; 
     MyMemoryElements x = storage.elementAt(counter); 
     try { 
      Thread.sleep(10000); 
     } catch (InterruptedException ex) {} 
     notifyAll(); 
     System.out.println(x.toString()); 
     canAccess = true; 
     return x; 
    } 

    public synchronized void update(MyMemoryElements element) { 
     System.out.println("Producer is inserting element "+ element.toString()); 
     while (!canAccess) { 
      try { 
       wait(); 
      } catch (InterruptedException e) {} 
     } 
     canAccess = false; 
     this.storage.add(element); 
     notifyAll(); 
     canAccess = true; 
    } 
} 

私は考えるcanAccess変数の必要性を理解するように見えることはできませんメソッドは同期化されます（また、の後にnotifyAllの後ではありません）。

EDIT：その他の質問： このコードには何らかの点がありますか？つまり、ベクターに何かを加えて追加することだけです。それらのアクションはすでにベクター上で同期されていませんか？このコードはすべてちょうど私たちがいくつかのプリントを得ることができますか？

Answer 1

私は同意します - メソッドが同期マークされているので、これらの余分なチェックは必要ありません。

は（皮肉なことに、canAccessはスレッドセーフではないことを主張することができ

:-)。誰かが何の同期の問題がなかったことを本当に、本当に必ずしたかったように見えます。したがって、同期されているメソッドがなくても、これは必ずしも適切な代替手段ではありません。