Подтвердить что ты не робот

Firebase Android onAuthStateChanged() дважды срабатывает после signInWithEmailAndPassword()

Когда я использую signInWithEmailAndPassword() для входа в систему onAuthStateChanged(), всегда стреляйте дважды.

Я очень уверен, что прослушивание добавляется только один раз к firebaseAuth, and I have the code in onStop() `для удаления слушателя после этого.

Кто-нибудь знает, как это решить?

Мой код:

public class SignInActivity extends BaseActivity implements
        View.OnClickListener,
        GoogleApiClient.OnConnectionFailedListener{

    private static final String PREF_KEY_USER_EMAIL = "User_Email";
    private static final int RC_SIGN_IN = 1111;

    private FirebaseAuth firebaseAuth;
    private FirebaseAuth.AuthStateListener authStateListener;
    private DatabaseReference firebaseDbReference;

    private TextView fieldEmail;
    private TextView fieldPassword;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_sign_in);
        getSupportActionBar().hide();

        firebaseAuth = FirebaseAuth.getInstance();
        firebaseDbReference = FirebaseDatabase.getInstance().getReference();
        fieldEmail = (TextView) findViewById(R.id.field_email);
        fieldPassword = (TextView) findViewById(R.id.field_password);

        String userSavedEmail = getPreferences(MODE_PRIVATE).getString(PREF_KEY_USER_EMAIL, "");
        if(userSavedEmail != null) {
            fieldEmail.setText(userSavedEmail);
            fieldPassword.requestFocus();
        }

        TextView linkForgotPassword;
        Button buttonLogin;

        linkForgotPassword = (TextView) findViewById(R.id.link_forgotPassword);
        buttonLogin = (Button) findViewById(R.id.button_Login);
        buttonSignUp = (Button) findViewById(R.id.button_signUp);

        if (linkForgotPassword != null) {
            linkForgotPassword.setOnClickListener(this);
        }
        if (buttonLogin != null) {
            buttonLogin.setOnClickListener(this);
        }
        if (buttonSignUp != null) {
            buttonSignUp.setOnClickListener(this);
        }

        authStateListener = new FirebaseAuth.AuthStateListener() {
            @Override
            public void onAuthStateChanged(@NonNull FirebaseAuth firebaseAuth) {
                if(firebaseAuth.getCurrentUser() != null) {
                    onAuthSuccess(firebaseAuth.getCurrentUser());
                }
            }
        };
    }

    @Override
    protected void onStart() {
        super.onStart();
        firebaseAuth.addAuthStateListener(authStateListener);
    }

    @Override
    protected void onStop() {
        super.onStop();
        if (authStateListener != null) {
            firebaseAuth.removeAuthStateListener(authStateListener);
        }
    }

    @Override
    public void onClick(View v) {
        switch (v.getId()){
            case R.id.link_forgotPassword:
                forgotPassword();
                break;
            case R.id.button_Login:
                emailLogin();
                break;
            case R.id.button_signUp:
                emailSignUp();
                break;
        }
    }

    @Override
    public void onConnectionFailed(@NonNull ConnectionResult connectionResult) {
        Toast.makeText(this, "Google Play Services error.", Toast.LENGTH_SHORT).show();
    }

    private void forgotPassword(){
        FirebaseAuth auth = FirebaseAuth.getInstance();
        String emailAddress = fieldEmail.getText().toString();

        if(TextUtils.isEmpty(emailAddress)){
            Toast.makeText(SignInActivity.this, R.string.msg_EnterEmail,
                    Toast.LENGTH_SHORT).show();
        }
        else {
            showProgressDialog();
            auth.sendPasswordResetEmail(emailAddress)
                    .addOnCompleteListener(new OnCompleteListener<Void>() {
                        @Override
                        public void onComplete(@NonNull Task<Void> task) {
                            hideProgressDialog();
                            if (task.isSuccessful()) {
                                Toast.makeText(SignInActivity.this, R.string.msg_ResetPasswordEmailSent,
                                        Toast.LENGTH_LONG).show();
                            }
                        }
                    });
        }
    }

    private void emailLogin(){
        if (!validateForm()) {
            return;
        }

        showProgressDialog();
        String email = fieldEmail.getText().toString();
        String password = fieldPassword.getText().toString();

        firebaseAuth.signInWithEmailAndPassword(email, password)
                .addOnCompleteListener(new OnCompleteListener<AuthResult>() {
                    @Override
                    public void onComplete(@NonNull Task<AuthResult> task) {
                        hideProgressDialog();

                        if (!task.isSuccessful()) {
                            Toast.makeText(SignInActivity.this, R.string.msg_EmailLoginFailed,
                                    Toast.LENGTH_SHORT).show();
                        }
                        else {
                            // Save the email
                            getPreferences(MODE_PRIVATE).edit()
                                    .putString(PREF_KEY_USER_EMAIL, fieldEmail.getText().toString())
                                    .apply();
                        }
                    }
                });
    }

    private void emailSignUp(){
        if (!validateForm()) {
            return;
        }

        showProgressDialog();
        String email = fieldEmail.getText().toString();
        String password = fieldPassword.getText().toString();

        firebaseAuth.createUserWithEmailAndPassword(email, password)
                .addOnCompleteListener(new OnCompleteListener<AuthResult>() {
                    @Override
                    public void onComplete(@NonNull Task<AuthResult> task) {
                        hideProgressDialog();

                        if (task.isSuccessful()) {
                            FirebaseUser user = task.getResult().getUser();
                            String displayName = displayNameFromEmail(user.getEmail());

                            // Update profile display name.
                            UserProfileChangeRequest profileUpdates = new UserProfileChangeRequest.Builder()
                                    .setDisplayName("Jane Q. User")
                                    .build();
                            user.updateProfile(profileUpdates);
                        } else {
                            Toast.makeText(SignInActivity.this, R.string.msg_EmailSignUpFailed,
                                    Toast.LENGTH_SHORT).show();
                        }
                    }
                });
    }

    private void onAuthSuccess(FirebaseUser user) {
        // Write new user
        writeNewUser(user.getUid(),
                user.getDisplayName(),
                user.getEmail(),
                user.getPhotoUrl());

        // Go to MainActivity
        startActivity(new Intent(this.getApplicationContext(), MainActivity.class));
        finish();
    }

    private void writeNewUser(String userId, String displayName, String email, android.net.Uri photoUrl) {
        User user = new User(displayName, email);

        if(photoUrl != null){
            user.setPhotoUrl(photoUrl.toString());
        }

        firebaseDbReference.child("users").child(userId).setValue(user);
    }

    private String displayNameFromEmail(String email) {
        if (email.contains("@")) {
            return email.split("@")[0];
        } else {
            return email;
        }
    }

    private boolean validateForm() {
        boolean result = true;
        if (TextUtils.isEmpty(fieldEmail.getText().toString())) {
            fieldEmail.setError("Required");
            result = false;
        } else {
            fieldEmail.setError(null);
        }

        if (TextUtils.isEmpty(fieldPassword.getText().toString())) {
            fieldPassword.setError("Required");
            result = false;
        } else {
            fieldPassword.setError(null);
        }

        return result;
    }
}
4b9b3361

Ответ 1

Он срабатывает дважды, и я думаю, что это ошибка, которую ребята из Firebase должны исправить (глядя на вас, Фрэн Хехехе). Единственное, что я могу думать, что вы можете сделать прямо сейчас, - добавить флаг, подобный этому.

private boolean flag = true;
...
authStateListener = new FirebaseAuth.AuthStateListener() {
            @Override
            public void onAuthStateChanged(@NonNull FirebaseAuth firebaseAuth) {
                if(firebaseAuth.getCurrentUser() != null && flag) {
                    onAuthSuccess(firebaseAuth.getCurrentUser());
                    flag=false;
                }
            }
        };

Далек от идеала, но пока будет работать. Код по-прежнему срабатывает дважды, мы принимаем первый и отрицаем второй с нашим флагом, таким образом, если ребята из Firebase фиксируют его, и внезапно слушатель работает один раз, наш код все еще работает. Возможно, это предназначено для того, чтобы слушатель работал дважды, надеюсь, у нас будут некоторые ответы из перекрестного сообщения Фрэнка.

Ответ 2

Двойной вызов вызван регистрационным вызовом. Не только это, onAuthStateChanged будет вызываться много раз во многих разных состояниях, не имея возможности узнать, в каком состоянии оно находится.

Документация говорит:

onAuthStateChanged (FirebaseAuth auth)

Этот метод вызывается в потоке пользовательского интерфейса при изменении состояния аутентификации:

  • Сразу после регистрации слушателя

  • Когда пользователь подписывается

  • При выводе текущего пользователя
  • При изменении текущего пользователя
  • При изменении текущего токена пользователя

Вот несколько советов по обнаружению текущего состояния:

  • Регистрационный вызов: пропустите первый вызов с флагом.
  • Пользователь выполнил вход: пользователь из параметра:!= null.
  • Пользователь отключен: пользователь из параметра == null.
  • Текущие пользовательские изменения: пользователь из параметра is!= null, а последний идентификатор пользователя: = идентификатор пользователя из параметра
  • Обновление токена пользователя: пользователь из параметра: = null, а последний идентификатор пользователя - id пользователя из параметра

Этот слушатель является беспорядком и очень ошибкой. Команда Firebase должна изучить ее.

Ответ 3

Далеко от идеала, но более точное решение.

private String authFlag = "";

...

authStateListener = new FirebaseAuth.AuthStateListener() {
            @Override
            public void onAuthStateChanged(@NonNull FirebaseAuth firebaseAuth) {    
            if (firebaseAuth.getCurrentUser() == null) {
              if (authFlag != null) {
                authFlag = null;
                subscriber.onNext(null);
              }
            } else {
              String uid = firebaseAuth.getCurrentUser().getUid();
              if (authFlag == null || authFlag.isEmpty() || !authFlag.equals(uid)) {
                authFlag = uid;
                subscriber.onNext(firebaseAuth.getCurrentUser());
              }
            }
}