私はListView
次のアダプターを持っています:
private class ListAdapter extends BaseAdapter {
private List<Item> items;
private ImageCache imageCache;
public ListAdapter(List<Item> item) {
this.items=items;
imageCache=ImageCache.getInstance();
}
//nothing special here
public View getView(int position, View convertView, ViewGroup parent) {
ViewHolder viewHolder;
if (convertView == null) {
convertView = getLayoutInflater().inflate(R.layout.listitem, parent, false);
viewHolder = new ViewHolder();
viewHolder.imgView = (ImageView) convertView.findViewById(R.id.imgview);
viewHolder.txtView = (TextView) convertView.findViewById(R.id.txtview);
convertView.setTag(viewHolder);
} else {
viewHolder = (ViewHolder) convertView.getTag();
}
Item item = getItem(position);
//this method is used to start download
imageCache.load(item.getImageURL(), viewHolder.imgView);
viewHolder.txtView.setText(item.getText());
return convertView;
}
}
以下は私のImageCache
クラスです。android.util.LruCache
Jake Wharton のDiskLruCacheに依存しています。アイテムがメモリとディスク キャッシュで見つからない場合は、AsyncTask を使用してダウンロードされます。
private Map<String,WeakReference<ImageView>> references;
//...
public void load(String imageUrl, ImageView imgView) {
final String key=getKey(imageUrl);
references.put(key, new WeakReference<ImageView>(imgView));
this.get(imageUrl, new OnImageLoadedListener() {
public void onImageLoaded(Bitmap image) {
WeakReference<ImageView> imgViewRef=references.get(key);
if(imgViewRef!=null && imgViewRef.get()!=null) {
imgViewRef.get().setImageBitmap(image);
} else {
Log.d("Set image", "Dead reference! setting bitmap cancelled");
}
}
});
}
ご覧のとおり、 ImageView がまだ生きている場合にのみイメージを設定WeakReference
し、からのビュー参照のリークを避けるために、を使用しようとしています。ただし、これは機能しないようです。画像は最初のいくつかの表示項目に対して正常にロードされますが、スクロールを開始すると、間違った画像 (以前のリスト項目で使用されたもの) が表示され、それが面倒です。どうすればこれを回避できますか? ダウンロードが終了次第、正しい画像に差し替えますが、ダウンロードに時間がかかる場合があります。ImageView
ListView
ListView
明らかな回避策は、ImageView
ダウンロードが完了するまで非表示にしてから表示することですが、他に方法はありますか?
ImageCache
編集:クラスの他のいくつかのメソッド:
//used to track requests for each image URL, key is URL
private Map<String, Queue<OnImageLoadedListener>> listeners;
//...
public void get(String imageUrl, OnImageLoadedListener listener) {
final String key=this.getKey(imageUrl);
Bitmap image;
Log.d(LOG_TAG, "image "+key+" for "+listener.hashCode()+": requesting...");
if (!listeners.containsKey(key)) {
listeners.put(key, new LinkedList<OnImageLoadedListener>());
}
Queue<OnImageLoadedListener> imageListenersQueue=listeners.get(key);
imageListenersQueue.add(listener);
//check if there are other requests for the same image
if (imageListenersQueue.size()>1) {
Log.d(LOG_TAG, "image "+key+" for "+listener.hashCode()+": already requested...");
return;
}
//proceed only if this request is the first
Log.d(LOG_TAG, "image "+key+" for "+listener.hashCode()+": proceeding...");
image=memCache.get(key);
if (image==null && diskCache!=null) {
image=diskCache.getBitmap(key);
if (image==null) {
new BitmapDownloader(new OnImageLoadedListener() {
public void onImageLoaded(Bitmap image) {
if (image!=null) {
memCache.put(key, image);
diskCache.put(key, image);
}
post(key, image);
Log.d(LOG_TAG, "image "+key+", downloaded bitmap "+image.hashCode());
}
}).execute(imageUrl);
} else {
memCache.put(key, image);
Log.d(LOG_TAG, "image "+key+" found in diskcache");
post(key, image);
}
} else {
Log.d(LOG_TAG, "image "+key+" found in memcache");
post(key, image);
}
}
private void post(String key, Bitmap bitmap) {
//will now pass bitmap to all listeners
LinkedList<OnImageLoadedListener> imageListenersQueue=(LinkedList<OnImageLoadedListener>) listeners.get(key);
while (!imageListenersQueue.isEmpty()) {
imageListenersQueue.remove().onImageLoaded(bitmap);
}
}
//and the mysterious getKey()
private String getKey(String str) {
return Integer.toHexString(str.hashCode());
}
ImageCache のログ出力は次のとおりです。
最初に表示されるリスト項目:
12-05 12:30:06.036: D/ImageCache(21138): image 40bc8de0 for 1104135776: requesting...
12-05 12:30:06.036: D/ImageCache(21138): image 40bc8de0 for 1104135776: proceeding...
12-05 12:30:06.071: D/ImageCache(21138): image 6589e90 for 1103929688: requesting...
12-05 12:30:06.071: D/ImageCache(21138): image 6589e90 for 1103929688: proceeding...
12-05 12:30:07.251: D/ImageCache(21138): image 40bc8de0 for 1104135776 ready, setting bitmap 1104282656
12-05 12:30:07.255: D/ImageCache(21138): image 40bc8de0, downloaded bitmap 1104282656
12-05 12:30:08.407: D/ImageCache(21138): image 6589e90 for 1103929688 ready, setting bitmap 1104663160
12-05 12:30:08.411: D/ImageCache(21138): image 6589e90, downloaded bitmap 1104663160
次の項目にスクロールする場合:
12-05 12:30:44.669: D/ImageCache(21138): image 2e762e6c for 1104686624: requesting...
12-05 12:30:44.673: D/ImageCache(21138): image 2e762e6c for 1104686624: proceeding...
12-05 12:30:45.997: D/ImageCache(21138): image 2e762e6c for 1104686624 ready, setting bitmap 1104089312
12-05 12:30:45.997: D/ImageCache(21138): image 2e762e6c, downloaded bitmap 1104089312
load()
このアイテムを呼び出すだけで、正しいイメージ セットが request に正確に設定されるため、問題はキャッシュ メカニズムにあるようには見えませんImageView
。ダウンロードして設定するまでは、間違った画像が表示されるだけです。