質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.01%

[Android] AsyncTaskの中断について

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 2,037

morizoo-

score 48

AsyncTask#cancelが上手く作動しなくて困っています。

AsyncTaskを保持するActivityを終了した時にAsyncTaskのBackground処理を中断させようと思い、AsyncTask#cancel(true)を使ってみたのですが、ログを見るとどうにも中断できていないようです。。。

Activity

public class ProfessorListActivity extends Activity {
    Context context = this;
    /** ListView内要素を検索するSearchView */
    SearchView searchView;
    /** Professorのカテゴリを要素として保持するListView */
    ListView listView;
    /**Professorの属性を保持するProfessorDataを扱うList*/
    public ArrayList<ProfessorData> professorDataList = new ArrayList<>();
    /**ProfessorDataを扱うAdapter*/
    ProfessorListAdapter adapter;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        //searchViewとListViewの実装、ProfessorDataListの生成
        ----------処理------------------------
        adapter = new ProfessorListAdapter(context, R.layout.custom_cell, professorDataList);
        listView.setAdapter(adapter);
        ----------処理------------------------
    }

    @Override
    protected void onDestroy() {
        //if (adapter.task != null) {
        if (!adapter.task.isCancelled()) {
            Log.d(CLASS_TAG, "onDestroy()!!!!");
            adapter.task.cancel(true);
        }
        super.onDestroy();
    }
}


Adapter

public class ProfessorListAdapter extends ArrayAdapter<ProfessorData> {
    /** 呼び出し元であるActivityのContext */
    private Context _context;
    /** 取得したListViewCellに要素を挿入するためのLayoutInflater */
    private LayoutInflater _inflater;
    /** 指定されたListViewCellId */
    private int _textViewResourceId;
    /** ListView生成時の初期要素を保持するList */
    private List<ProfessorData> initialEntry;
    /** 検索Filter */
    private MenuFilter _filter;
    /** 呼び出し元のActivityに渡すため*/
    public DownloadTask task;

    public ProfessorListAdapter(Context context, int resource, ArrayList<ProfessorData> objects) {
        super(context, resource, objects);
        _context = context;
        _inflater = (LayoutInflater) _context.getSystemService(Context.LAYOUT_INFLATER_SERVICE);
        _textViewResourceId = resource;
    }

----------------------------処理-------------------------

    //ここで非同期処理
    @Override
    public View getView(int position, View convertView, ViewGroup parent) {
        ItemLayout view; //ViewHolderの代わり
        final ProfessorData item = (ProfessorData) getItem(position);
        if (convertView == null) {
            view = (ItemLayout) _inflater.inflate(_textViewResourceId, null);
        } else {
            view = (ItemLayout) convertView;
        }
        view.mThumbNailView.setVisibility(View.GONE); //ListViewCell画像
        view.mThumbNailView.setTag(item.imageString);
        task = new DownloadTask(_context, view.mThumbNailView, item);
        task.execute(item.imageString);
        view.mNameView.setText(item.name); //ListViewCellText
        return view;
    }
}


AsyncTask

public class DownloadTask extends AsyncTask<String, Void, Bitmap> {

    private Context mContext;
    private ImageView mImageView;
    private String mTag;
    private ProfessorData item;

    public DownloadTask(Context context, ImageView imageView, ProfessorData item) {
        this.mContext = context;
        this.mImageView = imageView;
        this.mTag = imageView.getTag().toString();
        this.item = item;
    }

    @Override
    protected Bitmap doInBackground(String... imgStrs) {
        synchronized (mContext) {
            if (isCancelled()) {
                Log.d("task#doInBackground", "Cancelled!!");
                return null;
            } else {
                if (item.imageString.equals(imgStrs[0]))
                    Log.d("task#doInBackground", "name -> " + String.valueOf(item.name) + ", imageStr" + String.valueOf(item.imageString));
                try {
                    NCMBFile file = new NCMBFile((String) imgStrs[0]);
                    byte[] bytes;
                    try {
                        bytes = file.fetch();
                    } catch (Exception e) {
                        bytes = null;
                    }
                    Bitmap image = BitmapFactory.decodeByteArray(bytes, 0, bytes.length);
                    if (image == null) {
                        try {
                            image = UoASetting.loadBitmapFromAsset("res/img/TopActivity/info_button.png");
                        } catch (Exception e) {
                            Log.d("Exception type", String.valueOf(e.getClass()));
                            Log.d("task#doInBackground", "UoASetting#loatBitmapFromAsset: " + String.valueOf(e));
                        }
                    }
                    return image;
                } catch (Exception e) {
                Log.d("Exception type", String.valueOf(e.getClass()));
                Log.d("task#doInBackground", "task#doInBackground: " + String.valueOf(e));
                }
                return null;
            }
        }
    }

    @Override
    protected void onPostExecute(Bitmap img) {
        if (this.mTag.equals(this.mImageView.getTag())) {
            if (img != null) {
                this.mImageView.setImageBitmap(img);
                this.mImageView.setVisibility(View.VISIBLE);
            }
        }
    }
}


一応無駄な部分はできるだけ消しましたが、長くなってしまい申し訳ないです。AsyncTaskのコードだけはほぼそのままにしておきました。
自分的にはDownloadTaskのオブジェクトを親に渡している所が原因な気がするのですが、どうすれば良いのかが思いつきません。
皆様宜しくお願い致します。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+1

ProfessorListAdapter.task フィールドは、 getView が呼ばれる度に入れ替わりますが、 getView は、行が表示される毎に呼ばれます

つまり、onDestroy で adapter.task.cancel(true) しているのは、「最後に表示された行の AsyncTask だけ」 になっているのかなと思います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/09/19 13:42 編集

    なるほど、確かにTaskは複数のオブジェクトになりますね!
    お陰様で無事実装出来ました。
    ご回答誠にありがとうございました!

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.01%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる