[Kotlin]ListViewの要素を削除する方法

Question

前提・実現したいこと

一緒にプレイするユーザーを追加・削除できるシンプルな画面を作成しています。

発生している問題・エラーメッセージ

ユーザーを追加していくことはできたのですが、追加した後にユーザーを削除していく処理がうまくいきません。
まずはリストをタップして削除する機能を実装したいのですが、どのように修正すればうまくいくか教えていただけないでしょうか。
adapter.remove(item)を記載したのですが、消せませんでした。

該当のソースコード

package com.example.mollkyscorebook_ver11

import androidx.appcompat.app.AppCompatActivity
import android.os.Bundle
import android.view.View
import android.widget.*

class MainActivity : AppCompatActivity() {
    val playerList = mutableListOf("")
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_main)

        findViewById<Button>(R.id.addPlayerBtn).setOnClickListener(addPlayer())
        findViewById<ListView>(R.id.playerList).onItemClickListener = ListIemClickListener()
    }

private inner class addPlayer : View.OnClickListener{
    override fun onClick(view: View) {
        val displayPlayerList = findViewById<ListView>(R.id.playerList)
        val newPlayerName=findViewById<EditText>(R.id.newPlayerName)

        playerList.add(newPlayerName.text.toString())

        val adapter = ArrayAdapter(this@MainActivity, android.R.layout.simple_expandable_list_item_1,playerList)
        displayPlayerList.adapter=adapter
        newPlayerName.setText("")
        }
    }

private inner class ListIemClickListener : AdapterView.OnItemClickListener{
    override fun onItemClick(parent:AdapterView<*>, view:View, position: Int, id:Long) {
        val item = parent.getItemAtPosition(position) as String
        val adapter = ArrayAdapter(this@MainActivity, android.R.layout.simple_expandable_list_item_1,playerList)
        adapter.remove(item)
    }
}


}

<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    tools:context=".MainActivity">

<LinearLayout
    android:layout_width="wrap_content"
    android:layout_height="wrap_content"
    android:orientation="vertical">

    <LinearLayout
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:orientation="horizontal"
        android:layout_gravity="center">

        <EditText
            android:id="@+id/newPlayerName"
            android:layout_width="250dp"
            android:layout_height="wrap_content"
            android:hint="名前を入力してください"/>
        <Button
            android:id="@+id/addPlayerBtn"
            android:layout_width="wrap_content"
            android:layout_height="match_parent"
            android:text="追加"/>

    </LinearLayout>

    <ListView
        android:id="@+id/playerList"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"/>

    <Button
        android:id="@+id/startGame"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="スタート"
        android:layout_gravity="center"/>

</LinearLayout>

</androidx.constraintlayout.widget.ConstraintLayout>

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

Android Studio 4.1 Windows10

Comments

[hoshi-takanori]

追加や削除のたびに ArrayAdapter を作り直すのは良くないですね…。

[akira3213]

関数の外でadapterを定義してみたところ、今度はアプリが落ちるようになってしまいました…。adapterについて理解ができていないようです。初歩的な質問で申し訳ありませんが、一般的にはどう記述することが多いのでしょうか

Answer 1

とりあえずこんな感じでしょうか。

1. まず、adapter を MainActivity のプロパティにして、onCreate で初期化します。

また、playerList の初期値が mutableListOf("") ってことは、名前のないプレイヤーが一人いるってことになりますが、普通は空のリストにすべきでは。

 class MainActivity : AppCompatActivity() {
-    val playerList = mutableListOf("")
+    private val playerList = mutableListOf<String>()
+    private lateinit var adapter: ArrayAdapter<String>
 
     override fun onCreate(savedInstanceState: Bundle?) {
         // 略
 
+        adapter = ArrayAdapter(this, android.R.layout.simple_expandable_list_item_1, playerList)
+        findViewById<ListView>(R.id.playerList).adapter = adapter

1. 追加の際には、playerList に追加して adapter に変更を通知します。

         playerList.add(newPlayerName.text.toString())
+        adapter.notifyDataSetChanged()
 
-        val adapter = ArrayAdapter(this@MainActivity, android.R.layout.simple_expandable_list_item_1,playerList)
-        displayPlayerList.adapter=adapter
         newPlayerName.setText("")

1. 追加の際にも、playerList から削除して adapter に変更を通知します。

-        val item = parent.getItemAtPosition(position) as String
-        val adapter = ArrayAdapter(this@MainActivity, android.R.layout.simple_expandable_list_item_1,playerList)
-        adapter.remove(item)
+        playerList.removeAt(position)
+        adapter.notifyDataSetChanged()

---

ちなみに、自分だったら findViewById は onCreate でまとめて実行して (または view binding を使って)、OnClickListener などはラムダ式を使ってこんなふうに書きますね。

class MainActivity : AppCompatActivity() {
    private val playerList = mutableListOf<String>()

    private lateinit var newPlayerName: EditText
    private lateinit var addPlayerBtn: Button
    private lateinit var displayPlayerList: ListView
    private lateinit var adapter: ArrayAdapter<String>

    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_main)

        newPlayerName = findViewById<EditText>(R.id.newPlayerName)
        addPlayerBtn = findViewById<Button>(R.id.addPlayerBtn)
        displayPlayerList = findViewById<ListView>(R.id.playerList)

        addPlayerBtn.setOnClickListener { addPlayer() }
        displayPlayerList.setOnItemClickListener { _, _, position, _ -> removePlayer(position) }

        adapter = ArrayAdapter(this, android.R.layout.simple_expandable_list_item_1, playerList)
        displayPlayerList.adapter = adapter
    }

    fun addPlayer() {
        playerList.add(newPlayerName.text.toString())
        adapter.notifyDataSetChanged()

        newPlayerName.setText("")
    }

    fun removePlayer(position: Int) {
        playerList.removeAt(position)
        adapter.notifyDataSetChanged()
    }
}

---

それから、layout xml に関しては ListView の android:layout_width は "match_parent" にした方が良いのでは。
あと、ConstraintLayout の直下に LinearLayout を一つだけ配置してるのは謎ですね…。