java.lang.IllegalMonitorStateException: oggetto non bloccato da thread before wait()

Question

Ma io sto sincronizzando sull'oggetto 'roster' ovunque venga new'ed. Come mai ?

Il codice incriminato:

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

Con "viene new'ed" si intende la creazione di un nuovo oggetto roster ?

Sei sicuro di essere sincronizzato correttamente? Synchronizsation succede su istanze, non su variabili. Quindi se lo fai, ad es.

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

Poi si sincronizzati solo al vecchio, non il nuovaroster.

Così il seguente codice dovrebbe produrre lo stesso errore:

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 non avviene sul nome della variabile, ma sui contenuti . Se si sovrascrivono i contenuti, non si risincronizza con il nuovo valore!