java - if 和 else 中的 Toast 在 else 条件为 true 时执行。代码首先给出 "signup successful"然后给出 "user already exists"

Question

创建新帐户时，此代码首先给出“注册成功”，然后给出“用户已存在”。其他一切都工作正常。

帮我解决这个问题。非常感谢您在此事上花费的时间和提供的帮助。

还尝试过:

dataSnapshot.child(edtRoll.getText().toString())!=null

但这让情况变得更糟，它总是显示“用户不存在”。

btnSignUp.setOnClickListener(new View.OnClickListener() {
    @Override
    public void onClick(View view) {

        if (edtRoll.getText().toString().trim().length()!= 9)
            Toast.makeText(SignUp.this, "Roll No. must have 9 characters!", Toast.LENGTH_SHORT).show();
        else if (edtName.getText().toString().trim().length() < 3)
            Toast.makeText(SignUp.this, "Name is too short!", Toast.LENGTH_SHORT).show();
        else if (edtPassword.getText().toString().trim().length() < 8)
            Toast.makeText(SignUp.this, "Password must have atleast 8 characters!", Toast.LENGTH_SHORT).show();
        else if(!(edtPassword.getText().toString()).equals(edtConfirmPassword.getText().toString()))
            Toast.makeText(SignUp.this, "Passwords do not match!", Toast.LENGTH_SHORT).show();
        else {

            final ProgressDialog mDialog = new ProgressDialog(SignUp.this);
            mDialog.setMessage("Please Wait...");
            mDialog.show();

            table_user.addValueEventListener(new ValueEventListener() {
                @Override
                public void onDataChange(DataSnapshot dataSnapshot) {
                    //Check if userRoll already exist
                    if (dataSnapshot.child(edtRoll.getText().toString()).exists()) {
                        mDialog.dismiss();
                        Toast.makeText(SignUp.this, "User already registered!", Toast.LENGTH_SHORT).show();
                    } else {
                        mDialog.dismiss();
                        User user = new User(edtName.getText().toString(), edtPassword.getText().toString());
                        table_user.child(edtRoll.getText().toString()).setValue(user);
                        Toast.makeText(SignUp.this, "Sign up Successfull!", Toast.LENGTH_SHORT).show();
                        finish();
                    }
                }

                @Override
                public void onCancelled(DatabaseError databaseError) {

                }
            });

        }
    }
});

Answer 1

可能会发生这种情况，因为每当单击按钮并且用户处于注册状态时，就会显示 “signup success” ，您将重新注册数据更改的监听器:

btnSignUp.setOnClickListener(new View.OnClickListener() {
   @Override
   public void onClick(View view) {
     if (...) {
      ...
     } else {
       table_user.addValueEventListener(new ValueEventListener() {
         ...
       }
     }
   }
});

因此，您会收到“注册成功”然后“用户已存在”消息。

您需要在单击按钮方法之外初始化数据更改监听器。像这样的事情:

final ProgressDialog mDialog;

private void initDataChangeListener() {

  table_user.addValueEventListener(new ValueEventListener() {
    @Override
    public void onDataChange(DataSnapshot dataSnapshot) {
      ...
    }

    @Override
    public void onCancelled(DatabaseError databaseError) {

    }
  });
}

然后在添加点击监听器之前调用initDataChangeListener:

initDataChangeListener();

btnSignUp.setOnClickListener(new View.OnClickListener() {
   @Override
   public void onClick(View view) {
     if (...) {
      ...
     } else {
       if(mDialog == null) mDialog = new ProgressDialog(SignUp.this);
       mDialog.setMessage("Please Wait...");
       mDialog.show();
       // no need to handle the data, because it's already handled by the data change listener.
     }
   }
});