1

我需要(从互联网或我的缓存)下载很多图像。因此,我决定创建一个下载器线程,它将图像排入队列并在下载图像时通知调用者。该线程下载队列中的任何图像,并等待下载更多图像。在add方法中,我唤醒线程重新开始下载:

public class ImageDownloader implements Runnable {

private boolean continueFetching;
private List<Image> images;

private static ImageDownloader instance = new ImageDownloader();

private ImageDownloader() {
    continueFetching = true;
    images = new ArrayList<Image>();

    new Thread(this).start();
}

public static ImageDownloader getInstance() {
    return instance;
}

@Override
public void run() {
    synchronized (this) {
        while (continueFetching) {
            fetchAvailableImages();
            try {
                this.wait();
            } catch (InterruptedException e) {
            }
        }
    }
}

private void fetchAvailableImages() {
    while (images.size() != 0) {
        Image image = images.get(0);

                    Bitmap bitmap = downloadImage(image);
                    image.onImageDownloadCompleteListener.onImageDownloadComplete(bitmap);

        images.remove(0);
    }
}

public void stop() {
    synchronized (this) {
        continueFetching = false;
        notify();
    }
}

public void add(Image image) {
    images.add(image);
            notify;
}

public interface OnImageDownloadCompleteListener {

    public void onImageDownloadComplete(Bitmap bitmap);
}
}

当我同步add方法时,UI线程挂起,因为它需要等待当前图像完成下载。所以我删除了同步块,但现在我得到了java.lang.IllegalMonitorStateException: object not locked by thread before notify().

我该如何解决这个问题?

4

3 回答 3

1

更好的解决方案是使用这个为您处理图像下载的好库:)

用户易于使用且开源: https ://github.com/nostra13/Android-Universal-Image-Loader

为什么要发明两次轮子?:)

于 2012-12-11T14:57:39.103 回答
0

我使用这个源https://github.com/thest1/LazyList 效果很好,异步加载图像;)

希望对你有帮助

于 2012-12-11T14:53:29.720 回答
0
  1. 通知修改为 notifyAll..
  2. 当点击 UI/action 时,创建一个新线程......但是,性能很慢,因为一旦动作创建一个新线程......
于 2012-12-11T15:12:24.200 回答