returnの場所は、一番下ですが、AlartDialogを作る関係上、clicklistenerが作動する前の値でreturnされてしまいます。

Question

//このreturnの値であるnullを、下の、Log出力時のように、etoと表示されるようにしたいです。どこでreturnをすれば、etoをusernameに代入しreturnできるのでしょうか？

06-10 05:50:02.567 17954-17954/com.example.android.sample.new3 I/System.out: return username;:null

06-10 05:50:07.603 17954-17954/com.example.android.sample.new3 I/System.out: select username from user where username = 'eto' and password = 'eto';
06-10 05:50:07.624 17954-17954/com.example.android.sample.new3 I/System.out: checkusername:eto
06-10 05:50:07.624 17954-17954/com.example.android.sample.new3 I/System.out: username:eto
06-10 05:50:07.624 17954-17954/com.example.android.sample.new3 I/System.out: ログイン完了eto

  public String newSet(LayoutInflater inflaters, final Context contexts,View layouts,InputMethodManager inputMethodManagers) {


        builder = new AlertDialog.Builder(context);
        builder.setView(layout);
        builder.setNegativeButton("ログイン", new DialogInterface.OnClickListener() {
            public void onClick(DialogInterface dialog, int which) {
                EditText getusername2 = (EditText) layout.findViewById(R.id.username);           
                String getusername = getusername2.getText().toString();
               
                //データベースから取得してあればログイン
                String sql = "select username from user where username = '" + getusername + "';";
                System.out.println(sql);/////////////////////ログ出力
                MyOpenHelper helper = new MyOpenHelper(context);
                SQLiteDatabase db = helper.getWritableDatabase();
                Cursor c = db.rawQuery(sql, null);
                Integer count = c.getCount();
                if (count != 0) {
                    c.moveToFirst();
                    String checkusername = c.getString(0);                 
                    Toast toast = Toast.makeText(context, "ログインしました", Toast.LENGTH_SHORT);
                    toast.show();                   
                    System.out.println("checkusername:"+checkusername);/////////////////////ログ出力
                    username = checkusername;
                    System.out.println("username:"+username);/////////////////////ログ出力
                    System.out.println("ログイン完了" + username);/////////////////////ログ出力
                }

            });
            alertDialog = builder.create();
        alertDialog.setOnShowListener(new DialogInterface.OnShowListener() {

                @Override
                public void onShow(DialogInterface arg0) {
                    getusername2 = (EditText) layout.findViewById(R.id.username);
                    //inputMethodManager = (InputMethodManager) getSystemService(Context.INPUT_METHOD_SERVICE);
                    inputMethodManager.showSoftInput(getusername2, 0);
                }

            });
        alertDialog.show();

        System.out.println("return usernmae;:"+username);/////////////////////ログ出力
        return username;

        }

Answer 1

usernameはDBを開いて読み込んで初めて確定しますよね？つまりsetNegativeButtonに渡したonClickメソッドが実行されるまでは確定しません。

つまりSystem.out.println("return usernmae;:"+username);より先にたまたま偶然setNegativeButtonに渡したonClickメソッドのusername = checkusername;が実行されない限りnullにしかなりません。
つまり言い換えるとその時点のlogはあまり意味がありません(nullであることを確認する意味合いしかない)

Comments

[edoooooo]

nullであることをわかりやすく伝えるために、ログを載せました。
質問の意図は、どうすれば、このコードを改善できますか？
でした。

[yumetodo]

単純にSystem.out.println("return usernmae;:"+username);を取り除けばいいのでは？

[edoooooo]

ありがとうございます。これで直接的な原因は解決しました。