java.lang.IllegalMonitorStateException: Objekt nicht vor dem Warten durch den Thread gesperrt()

Question

Aber ich bin auf dem "Roster" -Objekt synchronisieren überall wo es neu wird. Woher ?

Der problematische Code:

public Roster getRoster() { 
    if (roster == null) { 
     return null; 
    } 

    if (!roster.rosterInitialized) { 
     try { 
      synchronized (roster) { 
       roster.reload(); 
       long waitTime = SmackConfiguration.getPacketReplyTimeout(); 
       long start = System.currentTimeMillis(); 
       while (!roster.rosterInitialized) { 
        if (waitTime <= 0) { 
         break; 
        } 
        roster.wait(waitTime); 
        long now = System.currentTimeMillis(); 
        waitTime -= now - start; 
        start = now; 
       } 
      } 
     } 
     catch (InterruptedException ie) { 
      // Ignore. 
     } 
    } 
    return roster; 
} 

Answer 1

Mit "wird new'ed" Sie meinen, Sie ein neues Roster-Objekt erstellen?

Sind Sie sicher, dass Sie richtig synchronisieren? Die Synchronisation erfolgt auf Instanzen, nicht auf Variablen. Wenn Sie z.B.

synchronized(roster) { 
    roster = new Roster(); 
    // do something 
} 

Dann nur Sie auf die alten synchronisiert, nicht die neueroster.

So ist der folgende Code sollte den gleichen Fehler erzeugen:

Roster roster = new Roster(); 
Roster othervariable = roster; 
synchronized(othervariable) { 
    roster = new Roster(); // create a new roster 
    othervariable.wait(1000); // OK, since synchronized with this instance! 
    roster.wait(1000); // NOT OK, not synchronized with *new* roster! 
} 

Synchronizsation geschieht nicht auf dem Name der Variablen, sondern auf den Inhalt . Wenn Sie den Inhalt überschreiben, synchronisieren Sie den neuen Wert nicht erneut!