AsyncTaskクラスの終了時にMainActivityの処理に戻る

Question

AsyncTaskClass task = new AsyncTaskClass();
task.execute("");
SharedPreferences.Editor editor = preferences.edit();
        editor.putInt("count",1);
        editor.commit();

このような処理がMainActivityにあったとして、3行目以降をAsyncTaskクラスの処理が終わった後に実行したい場合はどのようにすればよいでしょうか。

public class AsyncTaskClass extends AsyncTask<String, String, String> {

    @Override
    protected String doInBackground (String... params) {
        return "";
    }

    @Override
    protected void onPostExecute(String str) {
		// ここ
    }
}

上記のコードのコメントがある部分に記載をすればよいと思うのですがここからMainActivityの処理を呼び出す方法がわかりません。

質問が複数になって申し訳ありませんが、以下のコードは正しいですか。特にContext(★)の部分が心配です。またほかに良い方法はありますでしょうか。

//MainActivity
AsyncTaskClass task = new AsyncTaskClass(this);
task.execute("");

public class AsyncTaskClass extends AsyncTask<String, String, String> {

    Context context; //★

    public AsyncTaskClass(Context context) {
        this.context = context;
    }

    @Override
    protected String doInBackground (String... params) {
        scanFile();
        return "";
    }
    private void scanFile(String fileName){
        InputStream is = null;
        BufferedReader br = null;
        int times = 1;
        try {
            try {
                is = context.getAssets().open(fileName+".txt"); //★
                br = new BufferedReader(new InputStreamReader(is));
                String str;

                while ((str = br.readLine()) != null) {
                    //省略
                }
            } finally {
                if (is != null) is.close();
                if (br != null) br.close();
            }
        } catch (Exception e){
        }
    }

    @Override
    protected void onPostExecute(String str) {
        // UIスレッドに反映する処理
    }
}

Answer 1

最初にAsyncTaskは数秒程度の作業にとどめておくのがGoogleの推奨です
AsyncTask

WeakReferenceを使う方法もありますが、メモリリークが心配です
You don’t have to use WeakReference to avoid memory leaks

If you are an Android developer and you do not use WeakReferences, you have a problem.

Listnerを使うのが自然なやり方ではないでしょうか
非同期処理 AsyncTaskの使い方

Answer 2

MainActivity の特定のメソッドを呼び出す形で宜しければ, AsyncTaskClass のコンストラクタ等で MainActivity の参照を渡しておき, onPostExecute でメソッドを呼び出す形にすれば良いかと思います.

public MainActivity extends Activity {
    public void method() {
        //AsyncTaskClass の onPostExecute から呼ばれる処理
        SharedPreferences.Editor editor = preferences.edit();
        editor.putInt("count",1);
        editor.commit();
    }
}

public class AsyncTaskClass extends AsyncTask<String, String, String> {
    private MainActivity activity;
    public AsyncTaskClass(MainActivity activity) {
        this.activity = activity;
    } 
    @Override
    protected String doInBackground (String... params) {
        return "";
    }

    @Override
    protected void onPostExecute(String str) {
        // ここ
        activity.method();
    }
}

---

質問が複数になって申し訳ありませんが、

先のご質問と関係ないのでしたら, 分けたほうが良いかと思います.
こちらのサイトは(一応?)問題を共有し解決方法を蓄積・検索できるようにすることを目的としているはずです. タイトルから問題内容が分からないのでは意味がありません.

以下のコードは正しいですか

何をもって「正しい」と判断するのでしょうか.
動作するという点では, ご提示のコードで Context を使用しているのは getAssets() の呼び出しですので, それが uhsi さんの想定通りの結果を出したのであれば「正しい」といえるかもしれません.
「動作すれば正しい, しなければ間違い」程度であれば実際に動かしてみればよいでしょうし, それ以外はご自分で判断して頂かなくてはなりません.

Comments

[uhsi]

ありがとうございます。「MainActivity の参照を渡しておき」とありますが以下のコードでよいのでしょうか。
AsyncTaskClass task = new AsyncTaskClass(this);
task.execute("");

[uhsi]

また、教えていただいた方法と同じようなものだったのですが質問の下の部分のコードは正しいでしょうか。

[jimbe]

> 以下のコードでよいのでしょうか
ご心配なのだろうことは分かりますが, パラメータの渡し方までお聞きになるのでは, あまりに程度が低く思われます.
MainActivity 型として宣言している引数に対し, MainActivity の実体を(指す this を)渡せるのは java として当然です.

> 質問の下の部分のコード...
につきましては, 回答に追記致しました.

[shogiOtakku]

気になったのですが、`AsyncTaskClass `内の`MainActivity`はWeakReferenceで持った方がいいのではと思いました。
`doInBackground `実行中に端末を回転するなどでActivityの再生成されると`activity.method()`でNullPointerExceptionでアプリが落ちた記憶があるのですが、間違っていたらすみません。

[jimbe]

万全を期すならそういうことになるかもしれません.
ありがとうございます.

[kakajika]

@shogiOtakku ちょっと勘違いがあるみたいです。この場合問題になるのはActivityへの参照がnullになることではなく、Activityが破棄された後にメソッドを呼べてしまうことです。落ちるとしたらそのメソッドの中でActivityの提供する機能にアクセスしようとして落ちると思います。
対策としては、WeakReferenceを使ったりListenerを定義してActivityに実装するのは意味がなく、Activityの破棄時にAsyncTaskからの参照をnullにしてメソッドを呼ばれなくするのが重要です。（もしくは、Android Architecture Componentなどが提供するLifecycleへのバインドを利用するという手もあります）

[shogiOtakku]

@kakajika ありがとうございます。勘違いしていました。