Multithreading (Prüfungsvorbereitung)

[Ishildur]

Hallo zusammen
Ich habe nächste Woche eine Prüfung in Java Multithreading und habe hierzu ein Frage (und weitere werden vermutlich folgen )

Ich habe ein Verständnisproblem im Zusammenhang mit den Monitors und zwar kann ich folgendes nicht so recht verstehen: Wenn ich eine synchronisierte Collection benötige, so muss ich ja meine Zugriffsmethoden nicht synchronisieren, solange ich die Methoden einfach nur delegiere:

public void add(MyObject obj){
 this.syncList.add(obj); // syncList ist ein java.util.Vector
}

Aber in diesem Fall darf ich dies ja nicht mehr so machen:

public void add(MyObject obj){
 if(this.syncList.contains(obj)) this.syncList.add(obj); // zwischen dem contains und add könnte der Thread unterbrochen werden... PFUI!!
}

Nun habe ich es folgendermassen in den Büchern gelesen:

public void add(MyObject obj){
 synchronized(this.syncList){
   if(this.syncList.contains(obj)) this.syncList.add(obj); // alles paletti
 }
}

Aber das kann ich nun icht verstehen...

Wieso kann man es nicht einfach so machen:

public synchronized void add(MyObject obj){
  if(this.syncList.contains(obj)) this.syncList.add(obj); // ist offentbar ungültig, weil mehrere Monitore
}

Könnte mir das jemand erklären? Wäre echt nett!!

Mfg Ishildur

P.S.
By the way. Ich höre immer wieder, Vector sei schlecht! Wie macht man den eine synchronisierte Liste korrekt? Ich habe keine ConcurrentList gefunden?

 

[Gast]

> Wie macht man den eine synchronisierte Liste korrekt?

Collections#synchronizedList(List<T> list)

 

[Murray]

Sollte es noch irgendwo außerhalb Deiner Wrapper-Klasse Referenzen auf den Vector geben, dann könnte man darüber ja ggfs. aus anderen Threads den Vector verändern. Das würde durch das Sychronisiren auf die Wrapper-Klasse nicht verhindert werden.

 

[Ishildur]

Hehe, stimmt! Kann mir noch jemand die Sache mit den Monitors erklären? Stehe da echt auf dem Schlauch!

 

[Marco13]

Du meinst das letzte? Wenn ich nicht gerade einen (auf Hunger und Müdigkeit zu schiebenden :wink: ) Denkfehler habe, kann da ein Deadlock auftreten:

public synchronized void add(MyObject obj)
{
XXX  
  if(this.syncList.contains(obj)) 
  {
    this.syncList.add(obj); 
  }
}

public void doSomething add(MyObject obj)
{
  synchronized (syncList)
  {
YYY
    add(obj);
  }
}

Ein Thread könnte in der Zeile XXX stehen - er hält den Monitor "this", und wartet darauf, dass der Monitor "syncList" freigegeben wird, damit er "contains" aufrufen kann.

Ein anderer Thread steht in Zeile YYY - er hält den Monitor "syncList", und wartet darauf, dass der Monitor "this" freigegeben wird, damit der "add" aufrufen kann.

.... ???:L kann mich auch irren, aber ich denke, das würde zu einem Deadlock führen.

 

[Ishildur]

Hmm, bedeutet dies, dass jedes Objekt einen Monitor haben kann?

Also wäre folgender Code semantisch identisch?

public synchronized void doSomething(){
 // my code
}

und

public void doSomething(){
 synchronized(this){
  // my code
 }
}

 

[Murray]

Richtig.