質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.04%

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

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 61

score 1

前提・実現したいこと

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

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

ユーザーを追加していくことはできたのですが、追加した後にユーザーを削除していく処理がうまくいきません。
まずはリストをタップして削除する機能を実装したいのですが、どのように修正すればうまくいくか教えていただけないでしょうか。
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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • hoshi-takanori

    2021/05/04 16:22

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

    キャンセル

  • akira3213

    2021/05/04 18:55 編集

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

    キャンセル

回答 1

checkベストアンサー

0

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

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

2) 追加の際には、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("")

3) 追加の際にも、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 を一つだけ配置してるのは謎ですね…。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.04%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る