AsyncTaskのonPostExecuteが呼ばれない

Question

ボタンを押したらHTTPリクエストしてその結果をTextViewに表示するようにしたいのですが
doInBackgroundが実行されたあと、onPostExecuteが呼ばれません。
MainActivityのonClickからAsyncTaskのサブクラスのexecuteをよんでいるだけなのですが、、、

単に別スレッドでリクエスト投げて結果をコールバックで受け取れればよいだけのですが
AsyncTaskは使う必要はないのしょうか？

コードです

    class RequestTask extends AsyncTask<Void, Void, String> {
        @Override
        protected void onPreExecute() {
        }
        @Override
        protected String doInBackground(Void... arg0) {
            return response_data;
        }
        @Override
        protected void onPostExecute(String result) {
            showResponseView(result);
        }
    }
これをMainActivityのonClickからよんでるだけです
    public void onClick(View v) {
        if (v == null) {
            return;
        }
        switch (v.getId()) {
            case R.id.button:
                new RequestTask().execute();

Comments

[hoshi-takanori]

AsyncTask は非推奨になりましたが、使えることは使えるはずです。
onPostExecute が呼ばれない件は、ソースを貼ってくれないと分かりません。

[jimbe]

> AsyncTaskは使う必要はないのしょうか

必要性を判断できるのはご本人だけです。

[a.‫com]

なるほど非推奨なのですね！ありがとうございます。

[jimbe]

コードがご提示されましたが、大分省略されているようで、 doInBackground が終了しているのをどう確認されたのかも分かりません。
doInBackground が終了していること、そして onPostExecute が呼ばれていないことをどう確認されたのでしょうか。

[a.‫com]

デバッグでdoInBackgroundのreturnまで確認しました

[a.‫com]

onPostExecuteにブレークポイント貼っていたのですが呼ばれませんでした。

[jimbe]

> onPostExecuteにブレークポイント貼っていたのですが呼ばれませんでした

つまり「 onPostExecute で止まらなかった」ので呼ばれていないと判断されたということでしょうか。

[a.‫com]

もちろん、textviewにテキストを設定するようなメソッド呼び出しもしていましたが何も反応なかったので

[a.‫com]

hoshi-takanori様
jimbe様

原因が判明しました。実は例外が発生していました。
大変申し訳ございません。

[jimbe]

解決すればとりあえずは OK ですが、お時間ありましたら、どのようにブレークポイントを設定して実行した場合に、どこで例外が発生すると doInBackground の return まで行っても onPostExecute が呼ばれないのか、同じように onPostExecute が実行されないことで teratail を見に来るかもしれない後日の人の為に、情報を回答として載せて置いて頂けませんでしょうか。

Answer 1

原因としてはdoInBackground内で捕捉できない例外が発生していたです。ソケットの読み取り方法に不具合があり、ワーカースレッドが落ちたのかと思います。

Answer 2

こちらでは AndroidStudio の EmptyActivity から新しくプロジェクトを作って以下のコードで実験しても、 RequestTask.onPostExecute が実行されない様子は無く、 "response_data" が表示されました。

MainActivity.java

import androidx.appcompat.app.AppCompatActivity;

import android.os.*;
import android.widget.*;

public class MainActivity extends AppCompatActivity {
  private TextView textView;

  @Override
  protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);

    Button button = findViewById(R.id.button);
    button.setOnClickListener(view -> new RequestTask().execute());

    textView = findViewById(R.id.textView);
  }

  class RequestTask extends AsyncTask<Void, Void, String> {
    @Override
    protected void onPreExecute() {
    }
    @Override
    protected String doInBackground(Void... arg0) {
      return "response_data";
    }
    @Override
    protected void onPostExecute(String result) {
      textView.setText(result);
    }
  }
}

activity_main.xml

<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout
    xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    tools:context=".MainActivity">

  <Button
      android:id="@+id/button"
      android:layout_width="wrap_content"
      android:layout_height="wrap_content"
      android:text="START"
      app:layout_constraintBottom_toTopOf="@id/textView"
      app:layout_constraintLeft_toLeftOf="parent"
      app:layout_constraintRight_toRightOf="parent"
      app:layout_constraintTop_toTopOf="parent" />
  <TextView
      android:id="@+id/textView"
      android:layout_width="wrap_content"
      android:layout_height="wrap_content"
      android:text="Hello World!"
      app:layout_constraintBottom_toBottomOf="parent"
      app:layout_constraintLeft_toLeftOf="parent"
      app:layout_constraintRight_toRightOf="parent"
      app:layout_constraintTop_toBottomOf="@id/button" />

</androidx.constraintlayout.widget.ConstraintLayout>