androidアプリの非同期処理で得たデータの受け渡しについて

変数データの取り回し方でアドバイスをいただけないでしょうか？

現在、android studio で用意されたログイン用のテンプレートを元に、外部APIと連動したログイン処理を作りたいと思っています。
その中で、一部、データをどのように渡すべきか悩んでおります。

kotlinのログインテンプレート、LoginViewModel内から呼ばれる、loginRepositoryのloginメゾットから先で

class LoginViewModel(private val loginRepository: LoginRepository) : ViewModel() {

    private val _loginResult = MutableLiveData<LoginResult>()
    val loginResult: LiveData<LoginResult> = _loginResult

    fun login(email: String, password: String) {
	
        // can be launched in a separate asynchronous job
        val result = loginRepository.login(email, password)
        if (result is Result.Success) {
            _loginResult.value = LoginResult(success = LoggedInUser(userToken = result.data.displayName))
        } else {
            _loginResult.value = LoginResult(error = R.string.login_failed)
        }
	}
....

次のように非同期のHTTP通信で得たデータ（user）を戻す最適解のアドバイスをいただけないでしょうか？

    fun login(username: String, password: String): Result<LoggedInUser> {
        try {
            // TODO: handle loggedInUser authentication

            "http://hogehoge.jp/api/userdata".httpPost().responseObject<LoggedInUser>{ req, res, result ->
                val(user,err) = result
            }
	
            return Result.Success(user)
        } catch (e: Throwable) {
            return Result.Error(IOException("Error logging in", e))
        }
    }

代案として同期処理での実装も試みたのですが、まず、メインスレッドでのHTTP通信が禁止との事。
ならばと、次のようにThreadで処理を切り離して、HTTP通信部分を同期処理としてみたのですが、

        Thread {
            val result = loginRepository.login(email, password)
            if (result is Result.Success) {
                _loginResult.value =
                    LoginResult(success = LoggedInUser(userToken = result.data.userToken))
            } else {
                _loginResult.value = LoginResult(error = R.string.login_failed)
            }
        }

こちらは LiveData の更新がうまく拾えず稼働しませんでした。
アプローチの方法だけでもアドバイスいただけましたら、非常に助かります。

どうぞよろしくお願いいたします。

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

回答2件

ベストアンサー

ログインに限らず、非同期処理を開始するイベントが発生したら、
a) バックグラウンドで HTTP 処理を開始
b) 二重ログインにならないように、フラグを立てたり、ボタンを無効化
して、処理を戻して HTTP の結果を待ちます。結果を待ってる間もメインスレッドでは GUI の処理が行われるので、b が重要です。

そして、HTTP の結果が返ってきたら、結果を保存して、GUI を更新するためにメインスレッドに通知する、というのが基本的な考え方になります。

具体的なコードの書き方はいろいろあります。(ありすぎて困る…。) 今までは生のスレッドよりも AsyncTask クラスを使うのが一般的でしたが、次期 OS では廃止されるようなので、Kotlin であればコルーチンに移行するのがいいようです。(私もまだ勉強中ですが…。)
参考: AsyncTask is Deprecated, Now What?

投稿2020/04/23 19:24

編集2020/04/23 19:27