ListViewで選択された項目を削除したい

Question

現状

全項目選択されていたら、特に不具合は起きず全削除される。
3つ項目を作って1つ目・3つ目にチェックすると、1つ目と2つ目が削除される
→多分上から順に削除されてしまう。

削除している部分

adapter.remove(MakeQuizActivity.titleList.get(i));

知りたいこと

解決策やヒントをください。お願いします！

コード

public class MainActivity extends AppCompatActivity {
    private ArrayList data = new ArrayList<String>();
    private static final int REQUEST_CODE = 1;
    private ArrayAdapter adapter;
    public static int count = 1;
    //ListView listView = (ListView) findViewById(R.id.listView);
    ListView listView;

    @Override
    protected void onActivityResult(int request_Code, int resultCode, Intent result){
        switch(request_Code){
            //MakeQuizActivityから戻ってきた場合
            case(REQUEST_CODE):
                //クイズを作るボタンを押して戻ってきたときの処理
                if(resultCode == RESULT_OK){
                        data.add(result.getStringExtra("INPUT_TITLE"));
                        adapter.notifyDataSetChanged();
                }
                //押されなかったときの処理
                else if(resultCode == RESULT_CANCELED){

                }
                break;
        }
    }
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        // リスト項目とListViewを対応付けるArrayAdapterを用意する
        adapter = new ArrayAdapter<>(this, android.R.layout.simple_list_item_multiple_choice, data);
        listView = findViewById(R.id.listView);
        // ListViewにArrayAdapterを設定する
        listView.setAdapter(adapter);
        SharedPreferences sp = getSharedPreferences("DataStore",MODE_PRIVATE);
        SharedPreferences.Editor editor = sp.edit();
        PreferenceManager.init(sp,editor);
        Button button = (Button) findViewById(R.id.makeQuiz_Button1);
        button.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                Intent intent = new Intent(MainActivity.this, com.test.quizmaker.MakeQuizActivity.class);
                startActivityForResult(intent, REQUEST_CODE);
            }
        });
        Button button1 = (Button)findViewById(R.id.update_Button);
        button1.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                //現時点のコード
                int updateInt = PreferenceManager.sp.getInt("count",10);
                for(int i = 1; i<=updateInt; i++){
                    String updateTitle = PreferenceManager.sp.getString(DataKeys.TITLE.getValue() + String.valueOf(i), "nothing");
                    data.add(updateTitle);
                    adapter.notifyDataSetChanged();
                }
            }
        });
    }
    //オプションメニューを作成
    public boolean onCreateOptionsMenu(Menu menu){
        //menuにcustom_menuレイアウトを適用
        getMenuInflater().inflate(R.menu.custom_menu, menu);
        //オプションメニューを表示する場合はtrue
        return true;
    }
    //メニュー選択時の処理
    @Override
    public boolean onOptionsItemSelected(MenuItem menuItem){
        //押されたメニューのIDで処理を振り分ける
        //if(menuItem.getItemId() == R.id.menuClear){
        switch (menuItem.getItemId()) {
            case R.id.menuSelect:
                //マップの情報を取得する
                SparseBooleanArray checked = listView.getCheckedItemPositions();

                for (int i = 0; i < checked.size(); i++) {
                    if (checked.valueAt(i)) {
                        for (Map.Entry<String, String> entry : MakeQuizActivity.mapTitle.entrySet()) {
                            if (entry.getValue().equals(MakeQuizActivity.titleList.get(i))) {
                                Log.v("Debug", "キーを取得" + entry.getKey());
                                adapter.remove(MakeQuizActivity.titleList.get(i));
                                adapter.notifyDataSetChanged();
                                PreferenceManager.remove(entry.getKey());
                            }
                        }
                    }
                    for (Map.Entry<String, String> entry : MakeQuizActivity.mapQuiz.entrySet()) {
                        if (entry.getValue().equals(MakeQuizActivity.quizList.get(i))) {
                            Log.v("Debug", "キーを取得" + entry.getKey());
                            PreferenceManager.remove(entry.getKey());
                        }
                    }
                    for (Map.Entry<String, String> entry : MakeQuizActivity.mapAnswer.entrySet()) {
                        if (entry.getValue().equals(MakeQuizActivity.answerList.get(i))) {
                            Log.v("Debug", "キーを取得" + entry.getKey());
                            PreferenceManager.remove(entry.getKey());
                        }
                    }
                }
                break;
            case R.id.menuClear:
                adapter.clear();
                adapter.notifyDataSetChanged();
                PreferenceManager.clear();
                count = 1;
                break;
        }
        return true;
    }
}

ublic class MakeQuizActivity extends AppCompatActivity {
    public static Map<String,String> mapTitle = new HashMap<>();
    public static Map<String,String> mapQuiz = new HashMap<>();
    public static Map<String,String> mapAnswer = new HashMap<>();
    public static ArrayList titleList = new ArrayList<String>();
    public static ArrayList quizList = new ArrayList<String>();
    public static ArrayList answerList = new ArrayList<String>();

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

        Button button2 = (Button)findViewById(R.id.makeQuiz_Button2);
        button2.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                Intent intent = new Intent(MakeQuizActivity.this,com.test.quizmaker.MainActivity.class);
                String strTitle = ((EditText)findViewById(R.id.editText)).getText().toString();
                String strQuiz = ((EditText)findViewById(R.id.editText2)).getText().toString();
                String strAnswer = ((EditText)findViewById(R.id.editText3)).getText().toString();
                PreferenceManager.saveInt(MainActivity.count);
                PreferenceManager.saveString(strTitle, DataKeys.TITLE, String.valueOf(MainActivity.count));
                titleList.add(strTitle);
                mapTitle.put(DataKeys.TITLE.getValue() + String.valueOf(MainActivity.count),strTitle);
                PreferenceManager.saveString(strQuiz, DataKeys.QUIZ, String.valueOf(MainActivity.count));
                quizList.add(strQuiz);
                mapQuiz.put(DataKeys.QUIZ.getValue() + String.valueOf(MainActivity.count),strQuiz);
                PreferenceManager.saveString(strAnswer, DataKeys.ANSWER, String.valueOf(MainActivity.count));
                answerList.add(strAnswer);
                mapAnswer.put(DataKeys.ANSWER.getValue() + String.valueOf(MainActivity.count),strAnswer);
                String title = PreferenceManager.sp.getString(DataKeys.TITLE.getValue() + String.valueOf(MainActivity.count), "Nothing");
                String quiz = PreferenceManager.sp.getString(DataKeys.QUIZ.getValue() + String.valueOf(MainActivity.count),"Nothing");
                String answer = PreferenceManager.sp.getString(DataKeys.ANSWER.getValue() + String.valueOf(MainActivity.count),"Nothing");
                intent.putExtra("INPUT_TITLE", title);
                intent.putExtra("INPUT_QUIZ",quiz);
                intent.putExtra("INPUT_ANSWER",answer);
                MainActivity.count ++;
                Log.v("Debug","countの値:" + String.valueOf(MainActivity.count));
                setResult(RESULT_OK, intent);
                finish();
            }
        });
    }
}

public class PreferenceManager{

    public static SharedPreferences sp;
    private static SharedPreferences.Editor spEditor;

    public static void init(SharedPreferences sp1, SharedPreferences.Editor editor1) {
        sp = sp1;
        spEditor = editor1;
    }

    public static void saveString(String s, DataKeys key, String c) {
        spEditor.putString(key.getValue() + c, s);
        spEditor.apply();
    }

    public static void saveInt(int c) {
        spEditor.putInt("count", c);
        spEditor.apply();
    }

    public static void remove(String key){
        spEditor.remove(key).commit();
    }

    public static void clear(){
        spEditor.clear().commit();
    }
}

Answer 1

次のようにすれば解決します。

titleList.get(checked.keyAt(i))

Comments

[j06110611]

同じようにやってみたのですが、3つ目の項目が削除されないです。

[Zuishin]

どうなりますか？

[j06110611]

さっきと同じように動きます。要は1つ目と3つ目の項目にチェックをつけて削除すると1つ目と2つ目が削除される感じです。

[Zuishin]

削除している場所はどこですか？
つまり、どの行をコメントアウトすると削除されなくなりますか？

[j06110611]

ListViewから項目を削除してるのはこの2行ですね
adapter.remove(MakeQuizActivity.titleList.get(i));
adapter.notifyDataSetChanged();

[Zuishin]

であれば、i を逆順にすることで逆から削除されるように思いますが。それでできないということは、titleList.get(i) で返ってくる数値が違うのではないかと思います。実際に何がどのような順番で返ってきているのか調べてみてください。

[j06110611]

デバッグして、1つめの項目と3つ目の項目をチェック（2個チェックされた状態）で削除すると、1回目はiに1が入り、2回目はiに0が入りました。なので、2つ目の項目と1つ目の項目が削除されてしまうということが分かったのですが、どうしたらいいと思いますか？本当は1つ目の項目と3つ目の項目を削除したいのですが。

[Zuishin]

titleList.get(i) で 0 と 2 を返すようにしてみたらどうでしょうか。

[j06110611]

必ずしも、1つ目の項目と3つ目の項目を削除すると決まっているわけではなくて、選択した項目だけが削除されるようにしたいんです。例えば4つ項目作って2つ目の項目と4つ目の項目を選択する可能性があってもしっかり機能させるようにしたいんです。

[Zuishin]

そうではなく、3 つ目の項目が選択されたときにきちんと 2 を返すようにしてみてはということです。

[Zuishin]

> 全項目選択されていたら、特に不具合は起きず全削除される。

リストに入っている項目を i に無関係に順に返している可能性がありますね。

[j06110611]

でも、選択せずに削除すると、何も起きないので違うんじゃないかなと思います。

[Zuishin]

選択した数だけ前から順に返しているんじゃないでしょうか。

[j06110611]

それは読んでみてもわかるのですが、どうすればいいかわからないです。

[Zuishin]

MakeQuizActivity.titleList.get(i) が正しく実装されていないんじゃないかということです。ここにそのコードは書かれていないので、それ以上はわかりません。

[j06110611]

titleListはtextViewで書かれた文字列を追加してるだけのListなんです。なのでListViewに表示される項目の数と一致しているはずです。

[Zuishin]

では、titleList.get(i) ではなく、titleList.get(checked.valueAt(i)) ではどうなりますか？

[j06110611]

checkedがboolean型？なので赤字になっちゃいますね。

[Zuishin]

間違えました。titleList.get(checked.keyAt(i)) ではどうなりますか？

[j06110611]

できました！ありがとうございます！すごい助かりました！1つ目の項目と3つ目の項目を選択して削除すると、2つ目の項目だけ残りました！

Answer 2

for (int i = 0; i < checked.size(); i++) {
                    if (checked.valueAt(i)) {
                        for (Map.Entry<String, String> entry : MakeQuizActivity.mapTitle.entrySet()){

ではチェックしたアイテムの数を取得してループを回しているのでおかしくなります。

1☑
2
3☑
4
5☑
6

と実際はなっているのにもかかわらず、そのコードだと

1☑
2☑
3☑

と認識されていて、実際のListから1,2,3が削除されているのでしょう。

MakeQuizActivity.mapTitleの実装が書かれていないので、それ以上は分かりかねますが、基本的にチェックボックスなどのユーザー入力を伴うListViewないしRecyclerViewはカスタムアダプターを作って、自分でレイアウトしたListを操作する方が圧倒的に楽です。