Question

因此，我正在使用架构对我的应用进行模块化，现在，在我看来，我得到了 fullName ，密码1（pw1）， password2（ pw2）和电子邮件。

我有一个调用signUp()方法的按钮动作

注册

override fun signUp() {
        val fullName:String = etxt_name.text.trim().toString()
        val email:String = etxt_email.text.trim().toString()
        val pw1:String = etxt_pw1.text.trim().toString()
        val pw2:String = etxt_pw2.text.trim().toString()

        if(presenter.isEmailValid(email)){

            if(presenter.passwordsMatch(pw1,pw2)){

                if(presenter.isPasswordEmpty(pw1,pw2)){

                    etxt_pw1.setError("Empty field")
                    etxt_pw2.setError("Empty field")
                    return

                }else{
                    if(presenter.isNameEmpty(fullName)){

                        etxt_name.setError("Empty name")
                        return

                    }else{
                        presenter.signUp(fullName,email,pw1)
                    }

                }

            }else{
                etxt_pw1.setError("Passwords does not match")
                etxt_pw2.setError("Passwords does not match")
                return
            }

        }else{

            etxt_email.setError("Invalid E-mail")
            return
        }

    }

这将调用我的演示者中的方法来验证每个字段

演示者

   override fun passwordsMatch(pw1: String, pw2: String): Boolean {
        return pw1.equals(pw2)
    }

    override fun isEmailValid(email: String): Boolean {
        return Pattern.matches(
            "(?:[a-z0-9!#$%&'*+/=?^_`{|}~-]+(?:\\.[a-z0-9!#$%&'*+/=?^_`{|}~-]+)*|\"(?:[\\x01-\\x08\\x0b\\x0c\\x0e-\\x1f\\x21\\x23-\\x5b\\x5d-\\x7f]|\\\\[\\x01-\\x09\\x0b\\x0c\\x0e-\\x7f])*\")@(?:(?:[a-z0-9](?:[a-z0-9-]*[a-z0-9])?\\.)+[a-z0-9](?:[a-z0-9-]*[a-z0-9])?|\\[(?:(?:(2(5[0-5]|[0-4][0-9])|1[0-9][0-9]|[1-9]?[0-9]))\\.){3}(?:(2(5[0-5]|[0-4][0-9])|1[0-9][0-9]|[1-9]?[0-9])|[a-z0-9-]*[a-z0-9]:(?:[\\x01-\\x08\\x0b\\x0c\\x0e-\\x1f\\x21-\\x5a\\x53-\\x7f]|\\\\[\\x01-\\x09\\x0b\\x0c\\x0e-\\x7f])+)\\])",
            email)
    }

    override fun isPasswordEmpty(pw1: String, pw2: String): Boolean {
        return !(pw1.isEmpty() || pw2.isEmpty())
    }

    override fun isNameEmpty(fullName: String): Boolean {
        return fullName.isEmpty()
    }

我的问题

我正以这种方式实施正确的方法吗？还是我应该做得更好？如果是这样，那么任何提示都会被采用，如果语句在注册方法中降低了性能，也会给这些提示吗？

谢谢

Answer 1

您的方法很好，但是可以进行修改，以使其他人更容易理解。例如，可以像这样重新创建功能signUp()：

override fun signUp() {
    val fullName:String = etxt_name.text.trim().toString()
    val email:String = etxt_email.text.trim().toString()
    val pw1:String = etxt_pw1.text.trim().toString()
    val pw2:String = etxt_pw2.text.trim().toString()

    if (!presenter.isEmailValid(email)) {
        etxt_email.setError("Invalid E-mail")
        return
    }

    if (!presenter.passwordsMatch(pw1,pw2)) {
        etxt_pw1.setError("Passwords does not match")
        etxt_pw2.setError("Passwords does not match")
        return
    }

    if (presenter.isPasswordEmpty(pw1,pw2)) {
        etxt_pw1.setError("Empty field")
        etxt_pw2.setError("Empty field")
        return
    }

    if (presenter.isNameEmpty(fullName)) {
        etxt_name.setError("Empty name")
        return
    }

    presenter.signUp(fullName, email, pw1)
}

据我所知，if语句不会降低性能，这是创建验证器的常用方法。至于您的其他文件，似乎是正确的。希望对您有所帮助：）

这些if陈述会降低绩效吗？

注册

演示者

我的问题

1 个答案: