1

我正在调试我们以前的一位开发人员编写的 Android 服务,他的使用Boolean方式如下:

public static class DownloadQueue extends LinkedHashMap  
{
    // ...
    private Boolean lock = new Boolean(false);
    // ...

    //typical notify use
    synchronized public Object addToHead(Object key, Object value)
    {
        // ...
        synchronized (lock) 
        {
            //IllegalMonitorStateException FROM HERE
            lock.notify();
        }
        // ...
        return null;
    }

    //queue machinery
    public DownloadRecord getFirst()
    {
        // we block because queue is empty
        if(this.size() == 0 || (MyApp.isInternetConnectionAvailable() == false))
        {
            try 
            {   
                lock = true;
                synchronized (lock) 
                {
                    lock.wait(30000);
                }
                lock = false;
            }
            catch (InterruptedException e) 
            {}
        //continue operating the queue
        // ...
        return value;
    }
}

我发现的文档IllegalMonitorStateException表明这是由于没有notify()synchronized块调用引起的;但是,这里显然不是这种情况。我确实想知道范围是否lock可能是一个问题,或者不同步的分配是否可能是一个问题。所有的引用lock都是上面的形式,只有一个wait(long)在队列函数中。

最后一个可能有用的细节:发生这种情况是因为,尽管我们的设备有网络,但我们的 CMS 已关闭。我们借此机会在这种情况下测试应用程序。我怀疑队列可能正在以极快的速度运行,因为失败的下载正在重新排队,所以如果争用条件可能是导致此故障的原因,那么它可能就是这里的原因。

谢谢!

4

2 回答 2

4

问题是您正在重新分配方法中lock成员变量的值getFirst

Boolean 是一种不可变类型,因此当您重新分配它的值时,您正在创建一个新对象。

这意味着您不能保证通知您同步的同一个对象(如果在两者之间发生重新分配)。

final声明锁变量以避免这种编程错误是一种很好的做法。

于 2011-12-05T16:13:07.463 回答
4

您对锁变量的同步不起作用,因为您不断重新分配它:当您这样做 lock = true;lock = false;您正在更改“锁”所指的对象时。所以当你在你的addToHead方法中获得一个锁,然后另一个线程调用getFirst,锁的引用在它下面被改变,所以它在你调用notify时指向别的东西。

于 2011-12-05T16:10:09.587 回答