ダイアログフラグメントからAsyncTaskを起動し、ダイアログのレイアウトにsetTextする方法。

Question

ダイアログフラグメントから非同期でネット接続して
ネット上から数値取得した後、ダイアログフラグメントのViewに反映させたいのですが、
エラーが出てアプリが落ちます。

java.lang.NullPointerException: Attempt to invoke virtual method 'android.view.View android.app.Activity.findViewById(int)' on a null object reference
        at com.example.myapplication.ui.home.AsyncHttpRequest.onPostExecute(AsyncHttpRequest.java:62)

@Override
    public Dialog onCreateDialog(Bundle savedInstanceState) {
        AlertDialog.Builder builder = new AlertDialog.Builder(getActivity());
        LayoutInflater inflater = requireActivity().getLayoutInflater();
        View view = inflater.inflate (R.layout.custom_layout, null);
        textView = (TextView)view.findViewById(R.id.textView5);
        textView2 = (TextView)view.findViewById(R.id.textView6);
        button = (Button)view.findViewById(R.id.button2);
        button.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                PW = rand.nextInt(100000);
                if (new AsyncHttpRequest(getActivity())
                        .execute("https://xxxxxxxxxxxxxx.jp" + PW) != null) {
                    textView2.setText(String.valueOf(PW));
                }
            }
        });
builder.setView(view)
                .setTitle("Pairring")
                .setPositiveButton("OK", new DialogInterface.OnClickListener() {
                    public void onClick(DialogInterface dialog, int id) {

                    }
                })
                .setNegativeButton(R.string.cancel, new DialogInterface.OnClickListener() {
                    public void onClick(DialogInterface dialog, int id) {
                        // User cancelled the dialog
                    }
                });

        return builder.create();
}

public AsyncHttpRequest(Activity activity) {
        mActivity = activity;
    }

    @Override
    protected void onPreExecute() {
        super.onPreExecute();
    }

    @Override
    protected void onPreExecute() {
        super.onPreExecute();
    }

    @Override
    protected String doInBackground(String... params) {

        HttpURLConnection connection = null;
        StringBuilder sb = new StringBuilder();
        try {
            URL url = new URL(params[0]);
            connection = (HttpURLConnection) url.openConnection();
            InputStream is = connection.getInputStream();
            BufferedReader reader = new BufferedReader(new InputStreamReader(is, "UTF-8"));
            String line = "";
            while ((line = reader.readLine()) != null)
                sb.append(line);
            is.close();
        } catch (IOException e) {
            e.printStackTrace();
        } finally {
            connection.disconnect();
        }
        if (sb.toString() == "NO") {
            return null;
        } else {
            return (sb.toString());
        }
    }

    public void onPostExecute(String string) {
        if (string == null) {
            ((TextView)mActivity.findViewById(R.id.textView9)).setText("通信に失敗しました。再度接続をお試しください。");
        } else {
            try {
                int i = Integer.parseInt(string);
                ((TextView)mActivity.findViewById(R.id.textView5)).setText(String.valueOf(i));
            } catch (NumberFormatException e) {
            }
        }


    }

原因はおそらくmActivityの部分だと思うのですが、フラグメントに対して
どのような処理を行えばいいのかわかりません。

ネットで調べてみましたが、いまいち仕組みがわかりません。

ご教示いただければ幸いです。

よろしくお願いいたします。

Comments

[hoshi-takanori]

new AsyncHttpRequest の引数に渡している getActivity() が null のようですが、コードが断片的すぎてどうしてそうなってるのかは分かりません。たぶん、Fragment のライフサイクルを無視して Activity に接続されてない状態でそれを実行してるんだとは思いますが…。

[ToaZI]

ありがとうございます。
コードを追加しましたがこちらのコードでよかったでしょうか？

getActivityを一度何かの変数に入れればよいのでしょうか？

[hoshi-takanori]

DialogFragment のソースありがとうございます。こちらは問題なさそうですね。
そして、AsyncHttpRequest で mActivity.findViewById wぽ呼び出してるのはコンストラクタの中ではないってことでしょうか？

[ToaZI]

ありがとうございます。
AsyncTaskのほうのコードも追加させていただきました。
呼び出しを行っているのはコンストラクタの中ではないと思います。
一度処理を行った後のonPostExecuteで呼び出しています。

[hoshi-takanori]

mActivity は AsyncHttpRequest クラスのフィールドなら問題ないはずですが…。
デバッガか Logcat で mActivity の値を確認してみると良いかもしれません。

あと、onPostExecute の R.id.textView9 や R.id.textView5 は Activity に表示されてるってことですよね？ (Dialog のものなら findViewById のエラーではなく setText のエラーになるはずですけど…。)

[ToaZI]

ありがとうございます。

onPostExecuteのものはDialog上のカスタムレイアウトに反映させたいです。

[hoshi-takanori]

Dialog 上の TextView に表示させたいなら、new AsyncHttpRequest の引数を Activity ではなくダイアログそのものか、表示したい textView を渡せば良いかと。

[ToaZI]

ありがとうございます。

public AsyncHttpRequest(Activity activity) {
mActivity = activity;
}
の部分のActivityをDialogに変えればよいのでしょうか？
何度か試してみたのですが、うまくできませんでした。
ご教示お願いいたします。

[hoshi-takanori]

渡すとしたら Dialog ではなく、DialogFragment のサブクラス (または TextView) でしょうね。
DialogFragment のサブクラスの定義や、使ってるフィールドの宣言が省略されてるので DialogFragment を渡す例は書きづらいので、回答欄に TextView を渡す例を書いておきます。

Answer 1

結果を R.id.textView5 に表示したいとして、その TextView を渡す方法だとこんな感じでしょうか。

                if (new AsyncHttpRequest(textView)

public class AsyncHttpRequest extends AsyncTask<String, String, String> {
    private TextView mTextView;

    public AsyncHttpRequest(TextView textView) {
        mTextView = textView;
    }

    // 省略

    @Override
    public void onPostExecute(String string) {
        if (string == null) {
            mTextView.setText("通信に失敗しました。再度接続をお試しください。");
        } else {
            try {
                int i = Integer.parseInt(string);
                mTextView.setText(String.valueOf(i));
            } catch (NumberFormatException e) {
            }
        }
    }
}

Comments

[ToaZI]

いつもありがとうございます。
処理を反映させることができました。
今後ともよろしくお願いいたします。