ListViewの要素削除

Question

リストの中のボタンタップ時にリストの要素を削除したいです。
下記のようなコードを書いてみたのですがリストが最後の一行になるまでは消えていってくれます。最後の一行のとき配列の要素の削除。まではできていることを確認できたのですがリストに表示が残ったままになってしまっています。
その状態でリストをタップするとIndexOutOfBoundsExceptionで強制終了してしまいます。

どのように修正すればよいでしょうか？
よろしくお願いします

public class Activity extends AppCompatActivity {

    String[] strings =  {"a","b","c","d"};

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

    public void listView(){
        ListView listView = findViewById(R.id.lv);
        ArrayList<ListItem> listItems = new ArrayList<>();
        MyAdapteradapter = new MyAdapteradapter (this,0,listItems){
            @Override
            public View getView(final int pos, View convertView, ViewGroup parent) {
                LinearLayout linearLayout = (LinearLayout)super.getView(pos, convertView, parent);
                Button button= linearLayout.findViewById(R.id.button1);
                button.setOnClickListener(new View.OnClickListener() {
                    @Override
                    public void onClick(View v) {
                        //押された配列の要素を削除。（省略）
                        listItems.remove(pos);
                        listView();
                    }
                });

                return linearLayout;

            }
        };

        for (int c = 0; c < strings.length;c++){
            ListItem listItem = new ListItem();
            listItem.setText(strings[c]);
            listView.setAdapter(adapter);
            listItems.add(listItem);
        }
    }
}

Answer 1

最後の1行を消すと、stringsのサイズは0になりますから、for文内に入ることなく、
listView.setAdapterが呼ばれません。
そして更新されないままタップしても、配列のサイズは0なのに0番目の要素にアクセスしようとして
IndexOutOfBoundsExceptionしています。

---

そもそも、ListViewの更新をするときには一々setAdapterし直すのではなく、
notifyDataSetChanged()を呼びます。
それから、ListAdapterを継承したクラスにリストを渡しているなら、
そちらをremoveするのが良いかと思います。

Comments

[sena14]

なるほど。

public void onClick(View v) {
//押された配列の要素を削除。（省略）
listItems.remove(pos);
listView();
adapter.notifyDataSetChanged();
}

ということですか？
ここでやるとadapterが参照できなくて、、、、
どのタイミングでやればいいのでしょうか？

それから、ListAdapterを継承したクラスにリストを渡しているなら、
そちらをremoveするのが良いかと思います。

一応上記のlistItems.remove(pos);のところでremoveしているつもりなのですがこれとは違うのでしょうか？

[退会済みユーザー]

とりあえずMyAdapteradapterというクラスを作っているなら、
そのクラス内で処理したほうがいいと思います。

---
public class Activity extends AppCompatActivity {

String[] strings = {"a","b","c","d"};

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

ListView listView = findViewById(R.id.lv);
MyAdapter adapter = new MyAdapter(this, strings);
listView.setAdapter(adapter);
}
}
---

Activity側ではリストビューにアダプターをセットするにとどめて、
「タップしたら要素を削除する」のはMyAdapterの中で実装するのがいいと思います。

[sena14]

その方法の方がいいのかもしれませんが今回Activityの方で処理したいです。
質問のコードをどのようにすれば適切なのでしょうか？

[退会済みユーザー]

その前にMyAdapterというのも載せて頂けますか？
その中で何をしているかも知りたいので。

[sena14]

すみません。Activityで記載にこだわる必要がなくなってしまったので解決とさせていただきます。
ありがとうございました。