returnしたいusernameに値を入れたのに、 returnの値がnullとなってしまいます

Question

returnしたいusernameに値を入れたのに、
returnの値がnullとなってしまいます。どうすればいいのでしょうか？

  String username;
    
        public String newSet(LayoutInflater inflater, final Context context,View layout,InputMethodManager inputMethodManager) {
            
            builder = new AlertDialog.Builder(context);
            builder.setView(layout);
            builder.setPositiveButton("新規登録", new DialogInterface.OnClickListener() {

                public void onClick(DialogInterface dialog, int which) {
                    EditText getusername2 = (EditText) layout.findViewById(R.id.username);
                   username = getusername2.getText().toString();
                }
            });
            alertDialog = builder.create();
            alertDialog.setOnShowListener(new DialogInterface.OnShowListener() {

                @Override
                public void onShow(DialogInterface arg0) {
                    getusername2 = (EditText) layout.findViewById(R.id.username);
                     inputMethodManager.showSoftInput(getusername2, 0);
                }
            });
            alertDialog.show();
            return username;
        }

Answer 1

OnClick() 内に書かれている、

String username = getusername2.getText().toString();

で、String があるから新しい変数を定義して、そこに値を入れてます。
一方、return しているのは、先頭の String username; で定義した変数なので、空のままになっています。

上の行から String を外せば解決するのではないですかね？

Comments

[edoooooo]

ありがとうございます。消したのですが、
実行してもnullとなってしまいました。

[edoooooo]

System.out.println(""); で出力すると、わかるのですが、
新規登録ボタンを押して、public void onClick(DialogInterface dialog, int which) {
となる前に、returnが行われてしまい値が代入されていないのです。

[edoooooo]

そこで、public void onClick(DialogInterface dialog, int which) {の中で、
Intent intent = new Intent(context, LocationActivity.class);
intent.putExtra("username", username);
startActivity(intent);
として、ActivityのonStartで、値を取得しようとしたのですが、
java.lang.NullPointerException: Attempt to invoke virtual method 'android.app.ActivityThread$ApplicationThread android.app.ActivityThread.getApplicationThread()' on a null object reference

が出てしまいました。やはり、呼び出したメソッドから、startActivityで、呼び出し元に戻ることはできないのでしょうか？

[Tetsu.S]

ボタンを押すまでは処理が止まっていますか？

止まっていないとしたら、
----
リスナーを登録しても、ボタンが押されるのを待つわけではないので、処理が進んでしまって null が return されてしまいます。

リスナーの登録処理は、このメソッドではなく、コンストラクタでやるか、それ用のメソッドを新設して、このメソッドを呼び出す前にリスナー登録処理を完了しておいてはどうでしょう？

ボタンが押されてからusernameを取得するわけですが、OnClick は void なので、username は属性にしておいて、それを参照することになりますね。
----

ボタンを押すまで処理が止まっているとしたら、、、
難しい問題になりそうです。

[edoooooo]

ボタンを押さなくても、AlartDialogは、.show()されます。
そして、呼び出し元のActivitiyに戻ります
つまり、ボタンを押さなくとも、自動で、builderをcreateし、.show()して、
OnShowListener()が作動します。
新規登録ボタンのonClickListenerは、ボタンを押すまで、起動しません。

[edoooooo]

つまり、止まっているということでしょうか？

[edoooooo]

Alart alart=new Alart();
alart.setUsername(username);
のようにして、LocationActivityで、
username=alart.getUsername();
をしようと思ったのですが、
一連の流れをまとめますと、
LocationActivityからnewSet()を呼び出す。
alartBuilder=builder.create()
alartBuilder.show();
を終えたところで、ボタンが押されることを待たずに、LocationActivityの呼び出し元に戻る。
画面に表示されているAlartDialogの新規登録ボタンを押すと、onClickListenerが作動して、Alartクラスの、onClickListener()の処理を行い、 alart.setUsername(username)) する。
ここで、終わってしまいます。
ここから、LocationActivityに戻るには、どうすればいいのでしょうか？

[Tetsu.S]

AlertDialog が show されるということは、
止まっていないのではないですかね。

少し遠まわりになるかもしれませんが、
この newSet というメソッドの役割（やりたいこと）は何でしょうか？

それを明確にして、このクラスのメソッドとして実装し、それを OnClick から呼び出す、という構造にすると、すっきりするように思います。

このメソッドで、リスナーをセットして、更に入力した値のリターンも行う、という仕様には無理があり、それが問題を複雑にしているように感じます。

[Tetsu.S]

14:45 を読む前に送信してしまいました。

[Tetsu.S]

> alartBuilder.show();
> を終えたところで、ボタンが押されることを待たずに、LocationActivityの呼び出し元に戻る。

というのは、処理が進んで、return username; で null を返しているようですね。

14:49 で書いたように、やりたいことを明確にして、構造を見直すことを検討してみてください。

[edoooooo]

ありがとうございます。
このメソッドで、リスナーをセットして、更に入力した値のリターンも行う、という仕様には無理があり、それが問題を複雑にしているように感じます。>とのことで、メソッドを細分化しようと思います。

やりたいことは 、menuItemのtitleの文字によって、違った、AlartDialogを表示することでした。
そして、表示したAlartDialogにくっついているボタンを、押した時には、新規登録であったら、usernameにEidtTextから値を代入して、Activityにusernameを戻し、そこで、Activityの中にあるメソッドを呼び出すときの引数にusernameを使おうとしていました。

しかし、複雑すぎることから、色々検索していたところ、
http://d.hatena.ne.jp/poyonshot/20120223/1330007695
このようなサイトを見つけました。これの通りにmethodをわかりやすくしてみます。

とても助かりました。先に進めず、立ち往生していたところでした。ありがとうございました。

[Tetsu.S]

ヒントになるサイトが見つかって良かったですね。
問題は必ず解決できると思いますので、頑張って下さい。

[edoooooo]

はい、コールバックインターフェースを使ってみます。
つまずきましたが、毎回こういう機会にいろんな知識が身につくので、辛抱強く調べてみます。
ありがとうございました。感謝します。