android:ListViewの項目の削除

Question

ListViewにチェックボックスを付け（multiple_choiceの形式）、複数ある項目のうち
チェックがつけられた項目を削除できるようにしたいです。
今組んでいるコードでは、SparseBooleanArrayのvalueAtメソッドを使い
チェックのついているものを取得しようとしています。
しかし実際に動かしてみると、どの項目をチェックしたかに関わらず、
1回目のループでif(checked.valueAt(i))がtrueとなり、
一番上の項目が削除されます。どうすればよろしいでしょうか。
よろしくお願いいたします。
※掲載したコードは、削除ボタンをクリックした際に行われる処理です。

//完了画面へ
    View.OnClickListener buttonDelete = new View.OnClickListener() {
        @Override
        public void onClick(View view) {
            EmpView = findViewById(R.id.EmpView);
            SparseBooleanArray checked = EmpView.getCheckedItemPositions();
            dbOpenHelper = new DBOpenHelper(getApplicationContext());
            db = dbOpenHelper.getWritableDatabase();
            //チェック項目を消す
            try {
                for(int i = 0; i<checked.size(); i++) {
                    if(checked.valueAt(i)) {
                        db.delete("m_employee","_id = ?",new String[]{codes.get(i)});
                    }
                }
            } catch (SQLiteException e) {
                e.printStackTrace();
                Intent intent = new Intent(getApplicationContext(),Error.class);
                startActivity(intent);
            } finally {
                db.close();
            }
            Intent intent = new Intent(getApplicationContext(),Complete.class);
            startActivity(intent);
        }
    };

Comments

[yona]

EmpViewの中でSparseBooleanArrayを操作している箇所を質問に追記してください。

[aiue]

すみません。コードを確認したのですがEmpViewの中でSparseBooleanArrayを操作はしていませんでした。そこが問題なのでしょうか？

[hiramekun]

checked周りに問題があるならば、`SparseBooleanArray checked = EmpView.getCheckedItemPositions();`の部分で、EmpViewがどのような関数を実行しているのかをみる必要があります。

[aiue]

デバッグで確認するのでしょうか。すみません。

Answer 1

以下のコードに問題があります。

                for(int i = 0; i<checked.size(); i++) {
                    if(checked.valueAt(i)) {
                        db.delete("m_employee","_id = ?",new String[]{codes.get(i)});
                    }
                }

SparseBooleanArray#size()のドキュメントを読むと、次のように書いてあります。

Returns the number of key-value mappings that this SparseBooleanArray currently stores.

arrayが保持しているkey-valueのペアの個数を返すということです。
デバッガを見てもわかるように、SparseBooleanArrayは通常の配列とは異なり、indexのkeyとそのvalueを保持するものです。

よって、今回の場合ですと codes.get(i) -> codes.get(checked.keyAt(i))として

                for(int i = 0; i<checked.size(); i++) {
                    if(checked.valueAt(i)) {
                        db.delete("m_employee","_id = ?",new String[]{codes.get(checked.keyAt(i))});
                    }
                }

と書くと期待した挙動になると思います。

Comments

[aiue]

その通りに記述したところ期待通り動きました。
ありがとうございました！