[Kotlin] ラジオボタンで選択した項目をデータベース（SQLite）に保存する。

Question

現在、簡単なメモアプリを作成しております。
その際に、Mainとは別のアクティビティで選択したラジオボタンの項目をDB（SQLite）に保存し、画面遷移しても選択された項目をそのままにしたいと思っています。
下記のように実装していますがうまく動作しないのが現状です。

該当箇所に★マークを付けています。

package to.msn.wings.onsen_memo_2

import android.app.DatePickerDialog
import android.app.DatePickerDialog.OnDateSetListener
import android.content.ContentValues
import android.media.Rating
import android.os.Bundle
import android.view.Menu
import android.view.MenuItem
import android.view.View
import android.widget.*
import androidx.appcompat.app.AppCompatActivity
import java.text.FieldPosition
import java.util.*

class MemoActivity:AppCompatActivity() {


    companion object{
        private const val TABLE_NAME = "memos"
    }



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

        supportActionBar?.setDisplayHomeAsUpEnabled(true)

        val helper = DBHelper(this)
        val memoId: Long = intent.getLongExtra("id",0)
        val textTitle = findViewById<EditText>(R.id.text_title)
        val textContent = findViewById<EditText>(R.id.text_content)
        val rate = findViewById<RatingBar>(R.id.rating)
        val textDate = findViewById<EditText>(R.id.text_date)
        val textonsen = findViewById<EditText>(R.id.text_onsen)
        val buttononsen = findViewById<Button>(R.id.btn_onsen)
        val buttonsento = findViewById<Button>(R.id.btn_sento)
        val rgroup = findViewById<RadioGroup>(R.id.rgroup) ★


  
        if (memoId != 0L) {
            helper.readableDatabase.use { db ->
                db.query(
                    TABLE_NAME,
                    arrayOf("id", "title", "content","rating","date","onsen","radio"),
                    "id = ?",
                    arrayOf(memoId.toString()),
                    null,
                    null,
                    null,
                    "1"
                )
                    .use { cursor ->
                        if (cursor.moveToFirst()) {
                            textTitle.setText(cursor.getString(1))
                            textContent.setText(cursor.getString(2))
                            rate.setRating(cursor.getFloat(3))
                            textDate.setText(cursor.getString(4))
                            textonsen.setText(cursor.getString(5))
                            rgroup.setId(cursor.getInt(6))　★
                        }
                    }

            }
        }

        findViewById<Button>(R.id.save_button).setOnClickListener{
            helper.writableDatabase.use {
                    db ->
                val values = ContentValues().apply {
                    put("title", textTitle.text.toString())
                    put("Content", textContent.text.toString())
                    put("rating",rate.rating)
                    put("date",textDate.text.toString())
                    put("onsen",textonsen.text.toString())
                    put("radio",rgroup.checkedRadioButtonId)　★
                }
                if (memoId != 0L) {
                    db.update(TABLE_NAME, values,"id = ?", arrayOf(memoId.toString()))
                } else {
                    db.insert(TABLE_NAME,null, values)
                }
            }
            finish()
        }
    }

package to.msn.wings.onsen_memo_2

import android.content.Context
import android.database.sqlite.SQLiteDatabase
import android.database.sqlite.SQLiteOpenHelper

class DBHelper(context: Context?): SQLiteOpenHelper(context,DBNAME,null,version) {
    companion object{
        private const val DBNAME = "DBSample.sqlite"
        private const val version = 1
    }

    override fun onCreate(db: SQLiteDatabase?) {
        db?.let {
            it.execSQL("create table memos(id integer primary key, title text, content text,rating real,date text,onsen text,radio integer)")
        }
    }

    override fun onUpgrade(p0: SQLiteDatabase?, p1: Int, p2: Int) {

    }

    override fun onOpen(db: SQLiteDatabase?) {
        super.onOpen(db)
    }
}

そもそもSQLiteにはラジオボタンから取得したIDを登録できないのか、
また、SQLite書き込みの際に、<RadioGoupe>ではなく、<RadioButton>にID値を投入するのかわからない状況です。
もし何かアドバイス等ありましたらご教授していただけると幸いです。

Answer 1

レイアウト xml に書いた id をコードで使えるようにする R.id.xxx というのは int の定数です。
ですので SQLite の integer に入れることが出来ると思います。
が、 R.id.xxx の値は AndroidStudio が自動的にテキトウに決めた値ですので、何かの拍子に値が変わってしまう可能性があります。
以前は 100 だったのに今は 200 になっているとかがあり得るということで、もしそうなってしまったら、データベースに 100 と入っているのに該当するボタンが無い! なんてことになります。
ですので一般的にはビューの id をデータベースに入れることはしないでしょう。

ラジオボタンのそれぞれに番号を付けたつもりになって、1 つ目が選択なら 1 、 2 つ目が選択なら 2 をデータベースに書き込み、読み込みは 1 が入っていたら 1 つ目を選択状態にし、 2 が入っていたら 2 つ目を選択状態に・・・というのが一つの方法です。(これもまだ危なっかしいですが。)

もう少しちゃんとするなら、選択されたもの自体の情報をデータベースに入れることです。
例えば "java" か "kotlin" かを選ぶラジオボタンだったとして、番号(並び順)をデータベースに入れる方法だと java だったら 1 、 kotlin だったら 2 という感じになります。が、もし後から『これからは kotlin だから kotlin を先にしよう』とか『いやいや xamarin こそ一番目に追加だ』とかいう様々な理由で並びが変わる可能性があり、変わってしまったらデータベースの 1 が ラジオボタンの 1 番目では無くなるかもしれません。
そこで、実際に選ぶのは "java" か "kotlin" か ("xamarin" か) なのですから、データベースにも java が選ばれていたら "java"、 kotlin が選ばれていたら "kotlin" ( xamarin が選ばれ(ry ) という文字列そのものを入れてしまえば、データベースを直接見た場合も 1 とか 2 とかより分かり易いと思います。
(この考えでいけば、先の番号を保存する方法も、順番の番号では無く=順番に関係無く java は 1 、 kotlin は 2 と割り当てて １や 2 を保存するということでも同じになりますが、データベースを直接見て「これ 1 って何だっけ?」というよりは文字列で書いてあるほうが分かり易いです。)

Comments

[android_app_tw]

回答ありがとうございます。
試しにですが、下記のように文字列を保存して、その文字列から再度画面を開いたときに、チェックされてるかされてないかの判断をしようとしました。
R.id.rb1にテキスト（Java）
R.id.rb2にテキスト（Kotlin）を入れています。

val rgroup = findViewById<RadioGroup>(R.id.rgroup)
val radioId = rgroup.checkedRadioButtonId
val radioText = findViewById<RadioButton>(radioId)

.use { cursor ->
if (cursor.moveToFirst()) {
textTitle.setText(cursor.getString(1))
textContent.setText(cursor.getString(2))
rate.setRating(cursor.getFloat(3))
textDate.setText(cursor.getString(4))
textonsen.setText(cursor.getString(5))
if (cursor.getString(6) == "Java"){　★
rgroup.check(R.id.rb1)
}else{
rgroup.check(R.id.rb2)
}
}
}

val values = ContentValues().apply {
put("title", textTitle.text.toString())
put("Content", textContent.text.toString())
put("rating",rate.rating)
put("date",textDate.text.toString())
put("onsen",textonsen.text.toString())
put("radio",radioText.text.toString()) ★
}

ただ、上記設定を行ったときに
java.lang.NullPointerException: Attempt to invoke virtual method 'java.lang.CharSequence android.widget.RadioButton.getText()' on a null object reference
のようなエラーが出てしまいます。

これに対する対処法等はなにか考えられますでしょうか。

[jimbe]

それぞれのコードブロックが質問本文の★の付いているブロックに対応するのだと思いますが、だとすると、
val radioId = rgroup.checkedRadioButtonId
がそこでは、まだラジオボタンが選択されていませんので 0 になっていると思います。
val radioId = rgroup.checkedRadioButtonId
val radioText = findViewById<RadioButton>(radioId)
の 2 行はラジオボタンが選択された状態で実行しなければ意味がありませんので、素直に
val values = ContentValues().apply {
の直前で行っては如何でしょうか。

ついでにですが、
if (cursor.getString(6) == "Java"){　★
rgroup.check(R.id.rb1)
}else{
rgroup.check(R.id.rb2)
}
というのは、ラジオボタンが増減したら問題になる書き方です。
データベースへ書き込む文字列をラジオボタンのテキストにするのでしたら、この部分も『ラジオグループに含まれる各ラジオボタンの中でテキストが cursor.getString(6) と一致するものを チェックする』ようにしたほうが良いと思います。
そうすれば、ラジオボタンが幾つあってもどんなテキストでも(同じテキストのボタンが複数で無いかぎり)選んだテキストがデータベースに入り、読み込んだ時にそのボタンが(存在すれば)選択した状態に出来るでしょう。

[android_app_tw]

回答ありがとうございます！
上記試した結果、やりたいことができました。
また、色々と勉強になります。対応ありがとうございました！