时间戳

首页 > 解决方案 > 单个事件实时数据多次发出空值将给出正确的结果

android - 单个事件实时数据多次发出空值将给出正确的结果

问题描述

我在我的应用程序中进行了简单的登录，它在服务器端验证用户名和密码，并在此基础上显示一条 toast 消息。

登录片段

private fun onClickLogin(view: View) {

        view.loginButton.setOnClickListener {

            val emailAddress = view.emailTextInputEditText.text.toString()
            val password = view.passwordTextInputEditText.text.toString()

            viewmodel.generalLogin(emailAddress, password).observe(viewLifecycleOwner, Observer {

                if(it != null){

                    if (it.status) {
                        Toast.makeText(
                            context,
                            "Hi, " + it.data?.displayName,
                            Toast.LENGTH_SHORT
                        ).show()

                        val sharedPref = PreferenceManager
                            .getDefaultSharedPreferences(context)
                        val editor = sharedPref.edit()

                        editor.putString(getString(R.string.user_id), it.data?.email).apply()
                        editor.putString(getString(R.string.user_name), it.data?.email).apply()

                        activity?.finish()

                    } else {

                        Toast.makeText(
                            context,
                            "Error, " + it.message,
                            Toast.LENGTH_SHORT
                        ).show()
                    }
                }

            })

        }

    }

登录视图模型

fun generalLogin(email: String, password: String): LiveData<Resource<UserSession>> {

        val encryptedPassword = MCryptHelper.bytesToHex(MCryptHelper().encrypt(password))

        return Transformations.switchMap(loginRepository.generalLogin(email, encryptedPassword)) {
            it.getContentIfNotHandled().let{ resource ->
                val userSessionLiveData = MutableLiveData<Resource<UserSession>>()
                userSessionLiveData.value = resource
                return@switchMap userSessionLiveData
            }

        }
    }

登录存储库

fun generalLogin(email: String, encryptedPassword: String):MutableLiveData<SingleLiveEvent<Resource<UserSession>>>{

        val login = Global.network.login(email, encryptedPassword)

        login.enqueue(object : Callback<LoginResponse> {

            override fun onResponse(call: Call<LoginResponse>, response: Response<LoginResponse>) {

                if(response.body()?.status == 1){
                    val resource = Resource<UserSession>(true,"Success")
                    response.body().let {
                        if(it?.session != null){
                            resource.data = UserSession(it.session.userId!!,it.session.fullName!!)
                        }
                    }

                    loginMutableData.value = SingleLiveEvent(resource)

                }else{

                    val resource = Resource<UserSession>(false,response.body()?.msg ?: "Login failed. Try again")
                    loginMutableData.value  = SingleLiveEvent(resource)
                }
            }

            override fun onFailure(call: Call<LoginResponse>, t: Throwable) {
                loginMutableData.value = SingleLiveEvent(Resource(false, t.localizedMessage))
            }

        })

        return loginMutableData

    }

SingleLiveEvent

class SingleLiveEvent<out T>(private val content: T){

    var hasBeenHandled = false
        private set // Allow external read but not write

    /**
     * Returns the content and prevents its use again.
     */
    fun getContentIfNotHandled(): T? {
        return if (hasBeenHandled) {
            null
        } else {
            hasBeenHandled = true
            content
        }
    }

    /**
     * Returns the content, even if it's already been handled.
     */
    fun peekContent(): T = content
}

此示例适用于用户。但我注意到的是LoginViewModel generalLogin函数，它多次向片段发出空值，同时在其中给出写入值。这个应用程序只有在没有崩溃的情况下才能工作，因为我已经处理了 null 检查LoginFragment。当您使用不正确的登录凭据尝试越来越多时，似乎会增加 null 发射的数量。

有没有更好的方法来解决这个问题？如果有一种方法可以在结果为空的情况下以某种方式处理这个问题，那就太好了，因为根本getContentIfNotHandled()不发出任何东西，以便在 Fragment 中没有观察到任何东西。

提出你的建议。谢谢。

标签： androidandroid-architecture-componentsandroid-livedata

解决方案

每一层都有一些不同程度的错误。让我们一一介绍。


[1] 在点击侦听器内附加观察者的片段

每次用户单击按钮时，“LoginFragment”都会创建一个新的观察者，而之前的观察者仍然活着并在踢。这就是为什么每次尝试登录时，发射次数都会增加一的原因。此外，这种设计容易受到屏幕旋转和配置更改的影响。例如，假设用户在登录请求期间旋转屏幕。该登录请求的结果丢失了，因为没有观察到该请求，最糟糕的是视图将显示用户未登录，而存储库的观点是用户。

为了正确解决此问题，您需要分离观察逻辑和单击事件逻辑。还要记住始终观察onCreateView()或onActivityCreated()除非您传入自定义LifeCycleOwner对象或删除观察者。


[2] 视图模型中switchMap的错误使用

另一个问题是，每次viewModel.generalLogin()调用时，switchMap都会使用另一个，因此会创建一个全新LiveData的。LiveData不是您应该动态创建的东西。它们需要在视图模型初始化时创建一次并观察到视图模型被清除。


[3] 存储库

存储库代码大多是合理的，但我认为可以通过generalLogin不返回LiveData. 只是回去回调一种风格，或者根本不返回任何东西。另请注意，当前存储库具有loginMutableData. 虽然这没关系，但这是您需要手动跟踪的另一个变量。如果可能，您通常希望使存储库保持无状态。

所以完整的解决方案，存储库到片段：

存储库

// Create this new function
fun getLoginStatus(): LiveData<SingleLiveEvent<Resource<UserSession>>> {
    return loginMutableData
}

// Notice this function does not return anything.
fun generalLogin(email: String, encryptedPassword: String) {

    val login = Global.network.login(email, encryptedPassword)

    login.enqueue(object : Callback<LoginResponse> {

        override fun onResponse(call: Call<LoginResponse>, response: Response<LoginResponse>) {
            ...
        }

        override fun onFailure(call: Call<LoginResponse>, t: Throwable) {
            ...
        }

    })
}

视图模型

val userSessionLiveData: LiveData<SingleLiveEvent<Resource<UserSession>>>
// val userSessionLiveData: LiveData<Resource<UserSession>>

init {

    userSessionLiveData = loginRepository.getLoginStatus()

    // Use below if some mapping needs to be done
    // userSessionLiveData = Transformations.map(loginRepository.getLoginStatus()) {
    //     return it?.contentIfNotHandled?
    // }
}

fun generalLogin(email: String, password: String) {
    val encryptedPassword = MCryptHelper.bytesToHex(MCryptHelper().encrypt(password))
    loginRepository.generalLogin(email, encryptedPassword)
}

分段

void onCreateView(...): View {
    ...
    viewModel.userSEssionLiveData.observe(viewLifecycleOwner, Observer {

        val resource = it?.contentIfNotHandled?
        if (resource == null) return

        val session = resource.data?

        if (resource.status) {
            Toast.makeText(
                context,
                "Hi, " + session.displayName,
                Toast.LENGTH_SHORT
            ).show()

            val sharedPref = PreferenceManager
                .getDefaultSharedPreferences(context)
            val editor = sharedPref.edit()

            editor.putString(getString(R.string.user_id), session.email).apply()
            editor.putString(getString(R.string.user_name), session.email).apply()

            activity?.finish()

        } else {
            Toast.makeText(
                context,
                "Error, " + resource.message,
                Toast.LENGTH_SHORT
            ).show()
        }
    }
    ...
}

private fun onClickLogin(view: View) {

    view.loginButton.setOnClickListener {

        val emailAddress = view.emailTextInputEditText.text.toString()
        val password = view.passwordTextInputEditText.text.toString()

        viewmodel.generalLogin(emailAddress, password)
    }
}

推荐阅读