Wyjaśnienie java.lang.IllegalMonitorStateException: obiekt nie jest zablokowany przez wątek przed odczekiem()

Question

Ale synchronizuję obiekt "roster" wszędzie, gdzie jest on nowy. Dlaczego ?

Kod wykraczająca:

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

Z "dostaje new'ed" znaczy utworzyć nowy obiekt zaplanowany?

Czy jesteś pewien, że synchronizujesz poprawnie? Synchronizowanie odbywa się na instancjach, a nie na zmiennych. Więc jeśli zrobisz np.

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

Wtedy tylko zsynchronizowana z starego, a nie nowyroster.

więc następujący kod powinien produkować ten sam błąd:

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 nie zdarza imienia zmiennej, ale na treści. Jeśli zastąpisz zawartość, nie dokonasz ponownej synchronizacji z nową wartością!