android開発(java) トグルボタンの初期値設定で、OnCheckedChangedが呼び出されてしまう！！

Question

■実装したい内容

●ボタン付きのListView (トグルボタンのON/OFFはファイルから読み込み、初期値として設定する)

●リスト項目の削除機能

■問題点（詰まっている箇所）

リストの項目を削除すると、全てのON/OFFボタンが勝手に実行されてしまう。

Androidの ViewHolder を使用して実装を行っています。
ボタンの状態をViewHolderに保存してしまうと削除などの更新を行った後に画面に反映されないため、ボタンの状態(ON/OFF)の設定は下記の処理をViewHolderの外で行っています。
（正しい場所に設定できているか不安です）

holder.toggleButton.setChecked(Array.get(position).getOnOff());
//Arrayリストに保存してあるposition番目の項目のOn/Off状態をtrueまたはfalseで取得し、
//holder.toggleButton.setChecked()で初期値として設定する処理です。

この時、リスト項目は問題なく削除された結果が画面に反映されています。
ただし、トグルボタンに変更があった場合に呼び出される以下の処理

public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
    //処理
}

が呼び出されてしまい、全てのボタンが変更(実行)されてしまいます。

リスト項目を削除した結果を画面に反映させつつ、ON/OFFの状態を正しく表示させたいです。

onCheckedChanged が呼び出されないように、ボタンのON/OFFの初期値を設定できる方法などがあればいいのですが・・・

恐らく、アルゴリズムを正しく組めていれば実装できるものだと思いますが、どのように記述すればよいのか分からずお手上げ状態です。

ご教授頂けますと幸いです。

以下、処理部分のコードをまとめました。

 public View getView(final int position, View convertView, ViewGroup parent) {
        ViewHolder holder;
        // 最初だけ View を inflate して、それを再利用する
        if (convertView == null) {
            // activity_main.xml に list.xml を inflate して convertView とする
            convertView = inflater.inflate(itemLayoutId, parent, false);
            // ViewHolder を生成 (リストが更新されても変更されない)
            holder = new ViewHolder();
            holder.imageView = (ImageView) convertView.findViewById(R.id.imageView);
            holder.toggleButton = (ToggleButton) convertView.findViewById(R.id.toggleButton);


            // ボタンが変更された時の処理を設定
            holder.toggleButton.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener(){
            // トグルボタンがクリックされたと時のハンドラ
                @Override
                public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {

                    // トグルボタンの状態が変更された時の処理を記述
                    if(isChecked) {

                        //ONになった時の処理
                        Array.get(position).setOnOff(true);

                        //Arrayリストの内容をファイルに出力する処理★

                    }
                    else {

                        //OFFになった時の処理
                        Array.get(position).setOnOff(false);

                        //Arrayリストの内容をファイルに出力する処理★
                    }

                }
            });

            convertView.setTag(holder);

        }
        // holder を使って再利用
        else {
            holder = (ViewHolder) convertView.getTag();
        }



        ////////////////////////////////////////////////
        // リストが更新されると実行される。
        // リストが生成された時や、リストが画面外から出てきた時にも実行される。
        ////////////////////////////////////////////////

        // holder の imageView にセット
        holder.imageView.setImageResource(Array.get(position).getImage());
    holder.toggleButton.setChecked(Array.get(position).getOnOff());

        return convertView;
    }

Answer 1

トグルボタンにView作成中かどうかを判別できるようなタグをつけるのはどうでしょう。
getViewの処理中は「作成中」状態のタグをつけ、処理が終わるときに外す、ないしは「作成完了」などの状態のタグを付けます。
そしてリスナーメソッドで引数のbuttonViewからそのタグを取り出し、「作成中」タグなら何もせず終了するようにしてみてはいかがでしょうか？

Answer 2

setCheckedで値を設定する前にリスナーを外し、setCheckedで値を設定した後にリスナーをつけるようにすればいいですね。

Comments

[kt.tk.co]

ご回答ありがとうございます！
実装してみたところ、問題なく動くことを確認致しました。
いままでリスナーは付けるだけで「外す」という発想がありませんでした！

リストの画面をスクロールするたびに、リスナーを　外す→付ける　とするのは処理が多くなりそうなので気に掛かるので、今後時間があるときに他の方法を探してみる事に致します。

とても早く解決策をお教え頂きありがとうございました。

[yona]

そうですね。
ただ、匿名クラスではなくフィールド変数としてインターフェースを用意すればそんなに気にしなくてもいいと思いますよ。