Roomを用いてDB接続を行い、adapterにセットし、ListViewに表示できない

Question

Roomを用いてDB接続を行い、取得結果をArrayAdapterにセットし、ListViewに表示しようとしていますができません。
ソースコードは以下になります。

class ResultActivity : AppCompatActivity() {
    companion object {
        lateinit var database: ResultRoomDatabase
    }
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.result)

        database = Room.databaseBuilder(this, ResultRoomDatabase::class.java, "kotlin_room_sumple.db").build()

        //戻るオブジェクト取得
        val btBack = findViewById<Button>(R.id.btBack)

        //エラー箇所---
        GlobalScope.launch(Dispatchers.Unconfined){
            val arrayAapter: ArrayAdapter<Result> = ArrayAdapter(this, android.R.layout.simple_list_item_1, create())
            result.adapter = arrayAapter
        }
        //---ここまでエラー
        result.adapter = adapter

    }

    fun create(): List<Result> {
        val dao = database.resultDao()
        dao.insertAll(Result(0, "09:00", "17:30", 8, "2020/01/23", 1))
        var resultList: List<Result> = dao.getAll()
        return resultList
    }

    fun onBack(view: View){
        finish()
    }

    override fun onDestroy() {
        super.onDestroy()
    }
}

GlobalScope内のadapterでエラーが以下のエラーが出ます。
None of the following functions can be called with the arguments suplied
GlobalScope内ではArrayAdapterをセットする方法を教えてください。

Comments

[kakajika]

いくつかツッコミどころがありそうなコードですが、何を参考にコードを書かれているのでしょうか？

[negoto]

Roomに関しては、Android developerのRoomを用いてDB接続を行うと、個人サイトのRoomでDB管理をするを参考にしました。

[kakajika]

なるほど。。ご質問のコードで気になったのは主にcoroutines関連のところだったのですが、参考にされた個人サイトをちらっと見た感じだとRoomの実装にも問題がありそうですね。そのサイトの情報はあまり参考にしない方が良いかもしれません。

Answer 1

スマホからの操作でミス

Answer 2

• まず、エラーの原因について

ArrayAdapterの第1引数に渡しているthisがCoroutineScopeを指してしまっているので、Activityを指すように修正してみてください。

val arrayAapter = ArrayAdapter(this@ResultActivity, ...)

おそらくこれでエラーは消えると思います。
ただし、他の箇所にいくつか問題があるので以下参考にしてください。

---

• Roomの実行スレッドについて

ご提示のコードを拝見する限りでは、Daoの各メソッドをメインスレッド上で行うようになっていて、アプリを実行すると落ちるはずです。Roomはデフォルトではメインスレッド上での処理を許可しないので、非同期で実行する必要があります。coroutinesをお使いなら、Roomのcoroutinesサポート androidx.room:room-ktx を利用して各メソッドをsuspend関数にするのが一番シンプルで効率的な方法です。

もしくは、次の例のように自前で非同期処理を行ってその中で各メソッドを呼ぶようにします。

suspend fun create(): List<Result> {
    return withContext(Dispatchers.IO) {
        val dao = database.resultDao()
        dao.insertAll(Result(0, "09:00", "17:30", 8, "2020/01/23", 1))
        dao.getAll()
    }
}

LiveDataを使う方法もありますが、ここでは割愛します。

---

• CoroutineScopeとCoroutineContextについて

Activity内でコルーチンを立ち上げる場合、GlobalScopeは使うべきではありません。ドキュメントにも書かれている通り、Activityのライフサイクルに準じたスコープを利用するべきです。ご質問のコードでは、コルーチン内でActivityやViewを参照してしまっているので、最悪Activityがリークします。

また、CoroutineContextにDispatchers.Uncofinedを指定していますが、これは挙動が少しややこしいdispatcherなので、Dispatchers.Mainなどを指定した方が動作が明確になります。

MainScopeを利用するとこれら2つの問題を一緒に解決できるのでおすすめです。androidx.lifecycleを導入済みなら、LifecycleScopeを使うとさらにシンプルに書けます。

Comments

[negoto]

拙いコードを読み取りアドバイスありがとうございます。
今__イタリックテキスト__出先で試せないので帰宅したら試してみます。

以下疑問点になります。

Roomはデフォルトではメインスレッドで行えないのはコルーチン内ならメインでも行えるということですか？
そうでないなら、Dispatchers.MainもMainScopeもとちらもメインスレッド？で行うのでできないのではと思っています。

まだまだ始めたばかりでわからないことが多いので参考になるサイトなどを教えてもらえませんか？

[negoto]

ArrayAdapterの引数のcontextをthis@ResultActivityにしたらエラーはなくなりました。動かしたら、データ取得する際に今度はUserRecoverableAuthIOExcaptなるエラーが生じエラー落ちしました。（今回の問題とは別なので置いておきます…）
今後はDAOの各メソッドをsuspend修飾子を付け、GlobalScopeをMainScopeに変えて試していきます。
最後にもう一つ質問なのですが、ArrayAdapterのcontext引数をthis@ResultActivityにしたのですか？Contextを渡すものだと認識しています。
回答ありがとうございました、とても助かりました。

[kakajika]

> Roomはデフォルトではメインスレッドで行えないのはコルーチン内ならメインでも行えるということですか？

いえ、コルーチン内でもメインスレッドからアクセスした場合は落ちます。Daoのメソッドをsuspend関数にした場合は、アクセス時に自動的にCoroutineContextが切り替わり別スレッドからのアクセスとなるので落ちません。回答に書いた方法のように自前でCoroutineContextを切り替えた場合も同様です。

> そうでないなら、Dispatchers.MainもMainScopeもとちらもメインスレッド？で行うのでできないのではと思っています。

Dispatchers.Mainを指定するのはコルーチンをlaunchする際のCoroutineContextです。Roomへアクセスする際に別のDispatcherを指定したCoroutineContextに切り替わっていれば問題ありません。

> 最後にもう一つ質問なのですが、ArrayAdapterのcontext引数をthis@ResultActivityにしたのですか？Contextを渡すものだと認識しています。

ActivityはContextを継承しています。
もちろん、Activityを直接参照せずにViewなどからContextを取得して渡すようにしてもOKです。

[kakajika]

> まだまだ始めたばかりでわからないことが多いので参考になるサイトなどを教えてもらえませんか？

うーん、すみません。。入門者向けのサイトについてはあまり詳しくなくてわかりません。個人サイトよりは公式ドキュメントの方を参考にすべきと思います。
Androidアプリでのcoroutinesの利用についてはCodelabsの日本語訳を見つけたので参考にどうぞ。 https://droidkaigi.github.io/codelabs-kotlin-coroutines-ja/

[negoto]

スレッドを変えているのではなくCoroutineContextを変えることが重要と理解しました。
参考サイトもありがとうございます。
大変助かりました。