java - java.lang.IllegalMonitorStateException：对象在等待（）之前未被线程锁定

Question

但是我在“名册”对象上同步它得到新的任何地方。怎么来的 ？

违规代码：

 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

用“gets new'ed”你的意思是你创建一个新的名册对象？

你确定你同步正确吗？同步发生在实例上，而不是变量上。所以如果你这样做

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

然后你只同步到旧的，而不是新 roster的。

所以下面的代码应该会产生同样的错误：

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!
}

同步不会发生在变量的名称上，而是发生在内容上。如果覆盖内容，则不会重新同步到新值！