の内容に補足を付けて解説してます。
前回のエントリーで大きい画像を効果的に読む込む方法を解説しましたが、デコードするデータがディスクやネットワークにある場合、BitmapFactory の decode* メソッドは UI スレッドで行ってはいけません(というかメモリ上以外のデータを読み込む場合は全部だめ)。
これらの処理はディスクやネットワークのスピード、画像のサイズ、CPUのパワーなどさまざまな要因で完了までの時間が変わり、いつ完了するのかわかりません。 もし画像のデコード処理で UI スレッドをブロックしてしまうと、最悪 ANR が発生します。
そこで、AsyncTask を使ってバックグランドで Bitmap を読み込むようにします。
■ AsyncTask を使う
特に何も考えないで作ると、きっとこんな感じになると思います。
class BitmapWorkerTask extends AsyncTask<String, Void, Bitmap> {
ImageView mImageView;
int mWidth;
int mHeight;
String mFilePath;
public BitmapWorkerTask(ImageView imageView) {
mImageView = imageView;
mWidth = imageView.getWidth();
mHeight = imageView.getHeight();
}
// バックグラウンドで画像をデコード
@Override
protected Bitmap doInBackground(String... params) {
mFilePath = params[0];
return decodeSampledBitmapFromFile(mFilePath, mWidth, mHeight);
}
// ImageView に Bitmap をセット
@Override
protected void onPostExecute(Bitmap bitmap) {
if (bitmap != null) {
final ImageView imageView = mImageView;
imageView.setImageBitmap(bitmap);
}
}
}
この方法の問題は、AsyncTask のフィールドとして直接 ImageView のオブジェクトを持っていることです。 Activity からこの AsyncTask を起動したとして、Activity を終了しても(cancel() などを明示的に行わないなら)この AsyncTask は走り続けます。その際、この AsyncTask が ImageView への参照をもっているので、ImageView が GC の対象になりません。
これを防ぐために、次のように WeakReference を使って間接的に ImageView への参照を持つようにします。
class BitmapWorkerTask extends AsyncTask<String, Void, Bitmap> {
private final WeakReference<ImageView> mImageViewReference;
int mWidth;
int mHeight;
String mFilePath;
public BitmapWorkerTask(ImageView imageView) {
mImageViewReference = new WeakReference<ImageView>(imageView);
mWidth = imageView.getWidth();
mHeight = imageView.getHeight();
}
// バックグラウンドで画像をデコード
@Override
protected Bitmap doInBackground(String... params) {
mFilePath = params[0];
return decodeSampledBitmapFromFile(mFilePath, mWidth, mHeight);
}
// ImageView に Bitmap をセット
@Override
protected void onPostExecute(Bitmap bitmap) {
if (mImageViewReference != null && bitmap != null) {
final ImageView imageView = mImageViewReference.get();
if (imageView != null) {
imageView.setImageBitmap(bitmap);
}
}
}
}
WeakReference で ImageView への参照を持つようにすると、 AsyncTask は ImageView が GC されるのを妨げないようになります。
こうなると、onPostExecute() の時点で ImageView が存在していないことがあります。例えばバックキーで Activity が終了したり、画面回転などでコンフィグレーションが変わった場合などです。そこで ImageView の null チェックを行うようにします。
------------------------------
「AsyncTask は Activity の onPause() で cancel() すればいいじゃない」
というあなた。甘い、甘過ぎです。開発者としての視点ではそれでいいかもしれませんが、ユーザーのことを考えたら、必ずそうするのがいいとは限らないことがわかるはずです。
例えば、ユーザーがあるアプリAでネット上の画像を表示しようとして読み込み中になりました。
そこで、読み込みが終わるまでちょっと別のアプリ(例えばブラウザの記事を読んだり、twitter のタイムラインを見たり)に移動して、そろそろ読み込みが終わったかなーというころにアプリAに戻ってきました。 こういう使い方ってよくしますよね。
もし別のアプリに移動した時点(つまり、onPause() になったとき)で AsyncTask を cancel してしまったら、ユーザーが戻ってきたときに読み込みが完了できてないことになります。
これはユーザーとして嫌ですよね。
せめて onPause() で isFinishing() 判定くらいはしないとダメでしょう。
いやー、twicca よくできてるわ。
------------------------------
AsyncTask の実行はいつも通りです。
public void loadBitmap(String filePath, ImageView imageView) {
BitmapWorkerTask task = new BitmapWorkerTask(imageView);
task.execute(filePath);
}
■ 平行処理をあつかう
上記の方法は ListView や GridView で使うにはまだ問題が残っています。
ListView や GridView ではご存知の通り、スクロール時に子ビューを再利用しています。1行のビューを取得する getView() の中で上記の AsyncTask を走らせた場合、タスクが完了するまえにビューが再利用されてしまうと、別の行に別の画像が表示されるという、大変残念なことになってしまいます。
そこで、ImageView にセットされる Drawable にタスクへの参照を持たせておいて、タスクが完了したときに同じかどうかチェックするようにします。
Drawable として BitmapDrawable を使うようにすれば、タスクが完了するまでの間 BitmapDrawable の画像が ImageView に表示されるので、読み込み中画像を表示したい場合などに便利です。
static class AsyncDrawable extends BitmapDrawable {
private final WeakReference<BitmapWorkerTask> bitmapWorkerTaskReference;
public AsyncDrawable(Resources res, Bitmap bitmap, BitmapWorkerTask bitmapWorkerTask) {
super(res, bitmap);
bitmapWorkerTaskReference = new WeakReference<BitmapWorkerTask>(bitmapWorkerTask);
}
public BitmapWorkerTask getBitmapWorkerTask() {
return bitmapWorkerTaskReference.get();
}
}
タスクを走らせる前この BitmapDrawable を作成し、ImageView にセットしておきます。
public void loadBitmap(Context context, String filePath, ImageView imageView, Bitmap loadingBitmap) {
final BitmapWorkerTask task = new BitmapWorkerTask(imageView);
final AsyncDrawable asyncDrawable = new AsyncDrawable(context.getResources(), loadingBitmap, task);
imageView.setImageDrawable(asyncDrawable);
task.execute(filePath);
}
読み込みが終わったら、タスクが WeakReference として参照を持っている ImageView から Drawable をとりだし、その Drawable からタスクを取り出し、そのタスクがこのタスクと同じかどうかをチェックします。同じなら再利用されてないということです。
まず、ImageView からタスクを取り出すメソッドを用意しておきます。
private static BitmapWorkerTask getBitmapWorkerTask(ImageView imageView) {
if (imageView != null) {
final Drawable drawable = imageView.getDrawable();
if (drawable instanceof AsyncDrawable) {
final AsyncDrawable asyncDrawable = (AsyncDrawable) drawable;
return asyncDrawable.getBitmapWorkerTask();
}
}
return null;
}
onPostExecute() では上記のメソッドを呼び出して ImageView からタスクを取り出し、現在のタスクと同じか比較して同じなら ImageView に Bitmap をセットします。
class BitmapWorkerTask extends AsyncTask<String, Void, Bitmap> {
...
@Override
protected void onPostExecute(Bitmap bitmap) {
if (mImageViewReference != null && bitmap != null) {
final ImageView imageView = mImageViewReference.get();
if (imageView != null) {
// ImageView からタスクを取り出す
final BitmapWorkerTask bitmapWorkerTask = getBitmapWorkerTask(imageView);
if (this == bitmapWorkerTask && imageView != null) {
// 同じタスクなら ImageView に Bitmap をセット
imageView.setImageBitmap(bitmap);
}
}
}
}
}
(原文ではなぜが imageView の null チェックがなくなってますが必要です)
これで別の行に別の画像が表示されるという問題はなくなりました。
しかし他の問題が残っています。タスクが開始されたビューが一旦画面外にスクロールアウトされて、また画面にスクロールで戻ってきたとき、同じタスクが重複して走ってしまいます。
また、ビューが再利用された場合、以前のタスクはもはや必要ないのでキャンセルしたほうがいいでしょう。
そこで、タスクを走らせる前に ImageView にすでにタスクがセットされていないかチェックし、タスクがセットされている場合は、そのタスクが読み込んでいる画像の識別子(例えば、ファイルパス、URL、リソースIDなど)とこれから読もうとしている画像の識別子を比較します。
同じ識別子なら、すでに走っているのとまったく同じタスクを走らせようとしてるので、タスクを走らせないようにします。
違う識別子なら、ImageView が再利用されたということなので、以前のタスクはキャンセルし、新しいタスクを走らせます。
public static boolean cancelPotentialWork(String filePath, ImageView imageView) {
final BitmapWorkerTask bitmapWorkerTask = getBitmapWorkerTask(imageView);
if (bitmapWorkerTask != null) {
final String bitmapData = bitmapWorkerTask.mFilePath;
if (!bitmapData.equals(filePath)) {
// 以前のタスクをキャンセル
bitmapWorkerTask.cancel(true);
} else {
// 同じタスクがすでに走っているので、このタスクは実行しない
return false;
}
}
// この ImageView に関連する新しいタスクを実行する
return true;
}
public void loadBitmap(Context context, String filePath, ImageView imageView, Bitmap loadingBitmap) {
// 同じタスクが走っていないか、同じ ImageView で古いタスクが走っていないかチェック
if (cancelPotentialWork(filePath, imageView)) {
final BitmapWorkerTask task = new BitmapWorkerTask(imageView);
final AsyncDrawable asyncDrawable = new AsyncDrawable(context.getResources(), loadingBitmap, task);
imageView.setImageDrawable(asyncDrawable);
task.execute(filePath);
}
}
最後に、onPostExecute() にこのタスクがキャンセルされていないかチェックする部分を追加します。
class BitmapWorkerTask extends AsyncTask<String, Void, Bitmap> {
...
@Override
protected void onPostExecute(Bitmap bitmap) {
// キャンセルされていたらなにもしない
if (isCancelled()) {
bitmap = null;
}
if (mImageViewReference != null && bitmap != null) {
final ImageView imageView = mImageViewReference.get();
if (imageView != null) {
// ImageView からタスクを取り出す
final BitmapWorkerTask bitmapWorkerTask = getBitmapWorkerTask(imageView);
if (this == bitmapWorkerTask && imageView != null) {
// 同じタスクなら ImageView に Bitmap をセット
imageView.setImageBitmap(bitmap);
}
}
}
}
}
これで、ListView でも重複の心配なく、非同期読み込みできます!
わーい。
(でもキャッシュは、、、?)
最終的な全体のコードも載せておきます。
public void loadBitmap(Context context, String filePath, ImageView imageView, Bitmap loadingBitmap) {
// 同じタスクが走っていないか、同じ ImageView で古いタスクが走っていないかチェック
if (cancelPotentialWork(filePath, imageView)) {
final BitmapWorkerTask task = new BitmapWorkerTask(imageView);
final AsyncDrawable asyncDrawable = new AsyncDrawable(context.getResources(), loadingBitmap, task);
imageView.setImageDrawable(asyncDrawable);
task.execute(filePath);
}
}
public static boolean cancelPotentialWork(String filePath, ImageView imageView) {
final BitmapWorkerTask bitmapWorkerTask = getBitmapWorkerTask(imageView);
if (bitmapWorkerTask != null) {
final String bitmapData = bitmapWorkerTask.mFilePath;
if (!bitmapData.equals(filePath)) {
// 以前のタスクをキャンセル
bitmapWorkerTask.cancel(true);
} else {
// 同じタスクがすでに走っているので、このタスクは実行しない
return false;
}
}
// この ImageView に関連する新しいタスクを実行する
return true;
}
private static BitmapWorkerTask getBitmapWorkerTask(ImageView imageView) {
if (imageView != null) {
final Drawable drawable = imageView.getDrawable();
if (drawable instanceof AsyncDrawable) {
final AsyncDrawable asyncDrawable = (AsyncDrawable) drawable;
return asyncDrawable.getBitmapWorkerTask();
}
}
return null;
}
static class AsyncDrawable extends BitmapDrawable {
private final WeakReference<BitmapWorkerTask> bitmapWorkerTaskReference;
public AsyncDrawable(Resources res, Bitmap bitmap, BitmapWorkerTask bitmapWorkerTask) {
super(res, bitmap);
bitmapWorkerTaskReference = new WeakReference<BitmapWorkerTask>(bitmapWorkerTask);
}
public BitmapWorkerTask getBitmapWorkerTask() {
return bitmapWorkerTaskReference.get();
}
}
class BitmapWorkerTask extends AsyncTask<String, Void, Bitmap> {
private final WeakReference<ImageView> mImageViewReference;
int mWidth;
int mHeight;
String mFilePath;
public BitmapWorkerTask(ImageView imageView) {
mImageViewReference = new WeakReference<ImageView>(imageView);
mWidth = imageView.getWidth();
mHeight = imageView.getHeight();
}
// バックグラウンドで画像をデコード
@Override
protected Bitmap doInBackground(String... params) {
mFilePath = params[0];
return decodeSampledBitmapFromFile(mFilePath, mWidth, mHeight);
}
@Override
protected void onPostExecute(Bitmap bitmap) {
// キャンセルされていたらなにもしない
if (isCancelled()) {
bitmap = null;
}
if (mImageViewReference != null && bitmap != null) {
final ImageView imageView = mImageViewReference.get();
if (imageView != null) {
// ImageView からタスクを取り出す
final BitmapWorkerTask bitmapWorkerTask = getBitmapWorkerTask(imageView);
if (this == bitmapWorkerTask && imageView != null) {
// 同じタスクなら ImageView に Bitmap をセット
imageView.setImageBitmap(bitmap);
}
}
}
}
}
0 件のコメント:
コメントを投稿