java.lang.IllegalMonitorStateException: objeto no bloqueado por subproceso antes de wait()

Question

Pero me estoy sincronizando en el objeto 'roster' en todos los sitios donde se actualiza. Cómo ?

El código erróneo:

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 "recibe new'ed" quiere decir que crea un nuevo objeto lista ?

¿Estás seguro de que estás sincronizando correctamente? La sincronización ocurre en instancias, no en variables. Entonces, si lo hace, p.

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

Entonces sólo sincroniza con la edad , no el nuevo roster.

Así que el siguiente código debe producir el mismo error:

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 no sucede en el nombre de la variable , sino en los contenidos . ¡Si sobrescribe los contenidos, no se vuelve a sincronizar con el nuevo valor!