[Android][Kotlin]DBから取得したデータにアクセスできない

実現したいこと

DBから取得したデータにアクセスしたい

前提

Roomを使用して簡単なtodoアプリを作成しています。

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

MainActivity#onCreate()内で、DBから取得した値が保存されているviewmodel#List<Todo>にアクセスするようにしていますが、このタイミングでIndexOutOfBoundsExceptionが発生しています。
順序保障されていないことが原因だと思いますが、この場合どういう実装にすればよいのかよくわからなかったので、教えていただきたいです。

java.lang.IndexOutOfBoundsException: Empty list doesn't contain element at index 0.

該当のソースコード

kotlin
1class MainActivity : ComponentActivity() {
2    companion object {
3        const val TAG = "MainActivity"
4    }
5
6    private val viewModel: TodoViewModel by viewModels {TodoViewModel.factory}
7
8    override fun onCreate(savedInstanceState: Bundle?) {
9        super.onCreate(savedInstanceState)
10
11        Log.d(TAG, "onCreate")
12
13        val dummyData = Todo(title = "dummy data", status = TodoStatus.NOT_COMPLETED)
14        viewModel.createTodo(dummyData) //ここでDaoのAPIを呼び出してviewmodel.todoListに値を保存する
15        Log.d(TAG, "onCreate todoList=${viewModel.todoList}")
16        val text = viewModel.todoList[0].title //ここでIndexOutOfBoundsException
17
18        setContent {
19            TodoAppRoomJetpackComposeTheme {
20                Surface(
21                    modifier = Modifier.fillMaxSize(),
22                    color = MaterialTheme.colorScheme.background
23                ) {
24                    Greeting(text)
25                }
26            }
27        }
28    }
29}
30
31@Composable
32fun Greeting(name: String, modifier: Modifier = Modifier) {
33    Text(
34        text = "Hello $name!",
35        modifier = modifier
36    )
37}
38
39@Preview(showBackground = true)
40@Composable
41fun GreetingPreview() {
42    TodoAppRoomJetpackComposeTheme {
43        Greeting("Android")
44    }
45}

kotlin
1class TodoViewModel(
2    private val repository: TodoRepository
3    ): ViewModel() {
4    companion object {
5        const val TAG = "TodoViewModel"
6        val factory: ViewModelProvider.Factory = object : ViewModelProvider.Factory {
7            override fun <T : ViewModel> create(
8                modelClass: Class<T>,
9                extras: CreationExtras
10            ): T {
11                val context = checkNotNull(extras[APPLICATION_KEY])
12                val dao = TodoDatabase.createDb(context).todoDao()
13                val repository = TodoRepository(dao)
14                return TodoViewModel(repository) as T
15            }
16        }
17    }
18
19    var todoList = listOf<Todo>()
20
21    fun getTodoList() {
22        viewModelScope.launch {
23            todoList = repository.getTodoList()
24            Log.d(TAG, "getTodoList todoList=$todoList")
25        }
26    }
27
28    fun createTodo(todo: Todo) {
29        Log.d(TAG, "createTodo")
30        viewModelScope.launch {
31            repository.createTodo(todo)
32        }
33        getTodoList()
34    }
35}

kotlin
1class TodoRepository(private val dao: TodoDao) {
2    companion object {
3        const val TAG = "TodoRepository"
4    }
5
6    suspend fun getTodoList(): List<Todo> {
7        Log.d(TAG, "getTodoList")
8        return dao.observeAll()
9    }
10
11    suspend fun createTodo(todo: Todo) {
12        Log.d(TAG, "createTodo")
13        dao.insert(todo)
14    }
15
16    suspend fun delete(todo: Todo) {
17        dao.delete(todo)
18    }
19}

kotlin
1@Dao
2interface TodoDao {
3    @Query("SELECT * FROM todo")
4    suspend fun observeAll(): List<Todo>
5    @Insert(onConflict = OnConflictStrategy.REPLACE)
6    suspend fun insert(todo: Todo)
7    @Delete
8    suspend fun delete(todo: Todo)
9}

kotlin
1@Database(entities = [Todo::class], version = 1, exportSchema = false)
2abstract class TodoDatabase(): RoomDatabase() {
3    abstract fun todoDao(): TodoDao
4
5    companion object {
6        private const val TAG = "TodoDatabase"
7        private var INSTANCE: TodoDatabase? = null
8
9        fun createDb(context: Context): TodoDatabase {
10            Log.d(TAG, "createDb")
11            val instance = INSTANCE ?: synchronized(this) {
12                Room.databaseBuilder(
13                    context,
14                    TodoDatabase::class.java,
15                    "todo_database"
16                ).fallbackToDestructiveMigration().build()
17            }
18            INSTANCE = instance
19            return instance
20        }
21    }
22}

kotlin
1@Entity(tableName = "todo")
2data class Todo(
3    @PrimaryKey(autoGenerate = true) val id: Int = 0,
4    val title: String,
5    val status: TodoStatus
6)
7
8enum class TodoStatus {
9    COMPLETED,
10    NOT_COMPLETED
11}

行動規範の内容に同意します

回答1件

ベストアンサー

この回答はGPT-4による自動生成回答です

問題は、createTodoメソッド内で非同期にデータベース操作を行っているため、その結果が反映される前にgetTodoListメソッドが呼び出されてしまっていることです。そのため、getTodoListメソッドが呼び出された時点ではまだデータベースにデータが存在しないため、空のリストが返され、その結果IndexOutOfBoundsExceptionが発生しています。

この問題を解決するためには、createTodoメソッド内でデータベース操作が完了するのを待ってからgetTodoListメソッドを呼び出すようにする必要があります。以下にそのための修正例を示します。

kotlin
1class TodoViewModel(
2    private val repository: TodoRepository
3    ): ViewModel() {
4    // ...
5
6    fun createTodo(todo: Todo) {
7        Log.d(TAG, "createTodo")
8        viewModelScope.launch {
9            repository.createTodo(todo)
10            getTodoList()
11        }
12    }
13}

この修正により、createTodoメソッド内でデータベース操作が完了するまで待つようになり、その後でgetTodoListメソッドが呼び出されるようになります。これにより、getTodoListメソッドが呼び出された時点で必ずデータベースにデータが存在することが保証され、IndexOutOfBoundsExceptionが発生しなくなります。

投稿2023/06/28 12:28