onClickListnerの使い方

Question

以下のようなコードを書いてみたのですが、電卓のようなものを想定していて0～9のボタンをタップするとTextViewにそれが表示される想定です。
しかしTextViewに何も表示されずに困っています。おかしな所教えていただけるとありがたいです。

import android.content.Intent;
import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.view.View;
import android.widget.Button;
import android.widget.TextView;

public class ActivityInput extends AppCompatActivity {

    Button[] buttons;
    String inputMoney;
    TextView textView;

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

        textView = (TextView) findViewById(R.id.inputText);
        click();
    }

    public void click() {
        buttons = new Button[]{findViewById(R.id.btn0), findViewById(R.id.btn1), findViewById(R.id.btn2), findViewById(R.id.btn3), findViewById(R.id.btn4), findViewById(R.id.btn5), findViewById(R.id.btn6), findViewById(R.id.btn7), findViewById(R.id.btn8), findViewById(R.id.btn9)};
        for (int a = 0; a < buttons.length; a++) {
            buttons[a] = new Button(this);
        }
        for (int i = 0; i < buttons.length; i++) {
            final String s = String.valueOf(i);
            buttons[i].setOnClickListener(new View.OnClickListener() {
                @Override
                public void onClick(View v) {
                    inputMoney = inputMoney + s;
                    textView.setText(inputMoney);
                }
            });
        }
    }
}

コード

Answer 1

デザイナーで定義したボタンをそのまま生かそうとしているのか、デザイナーではなく実行時に動的にボタンを配置しようとしているのかはっきりしない中途半端なコードになっているのではないでしょうか？

実行してもエラーが出ないのだとしたら

buttons = new Button[] { findViewById(R.id.btn0), ... };

この行でbuttonsにはデザイナーで定義したボタンの参照を要素とする配列が設定されていると思います。それ以上何もする必要はないと思いますが、その次のfor文を実行してしまうと「新たに生成したボタン」で配列要素が置き換わってしまいますね。

• デザイナーでビューを定義したなら素直にそれをfindViewByIdで求めてそのまま使うのが順当と思います。
• そうせずに動的にビューを生成するならするで、しかるべきLayoutインスタンスを用意しそこへボタンを配置すべきと思います。

そうしないとビューとしての機能を果たさないと思います。つまり新たに生成したボタンはただメモリー上に存在するだけのものであり、画面上に見えることもありませんし、ユーザー操作に反応することもありません。

Comments

[sena14]

無事できました！
ありがとうございます！

Answer 2

デバッグをしましょう。
onClickの行にブレークポイントを設置してデバッグ実行してください。その後にボタンをクリックし停止させて、各変数の値を確認してください。

また、Androidのタグを質問に付けてください。

Comments

[sena14]

ブレークポイントを付けてデバッグ実行しても停止しません。