Sign In y Sign Out Correo y Contraseña en Platzigram

Clase 69 de 96 • Curso Definitivo de Android 2016

Gerardo Hernández

student•

Hola Ann, aquí considero que existe una mala práctica, ya que estás implementando la lógica (o parte de ella) de Firebase en la vista, si los principios de SOLID y Clean Architecture dictan que esta lógica debería estar en el modelo (repositorio en nuestro caso), la inicialización de Firebase debería estar implementada en el repositorio y desde la vista llamar a un método en el presenter que a su vez pase por el interactor y luego por el repositorio y es allí donde se inicia, el mismo principio aplica para en onStart y onStop de la vista, de este modo, toda la lógica referente a Firebase queda completamente encapsulada

Ann Code

teacher•

Tienes toda la razón! gracias por tu observación entendiste perfecto el proceso MVP + Clean Arquitecture 😄

Jair Israel Avilés Eusebio

student•

Concordando con el comentario del compañero ghernagon y externando una sugerencia para el team Platzi, por favor sean conscitentes con el uso de convenciones en la programación durante las clases. Entiendase desde nomenclatura, patrones de diseño y metodologias.

Uno como estudiante de la plataforma esperaria eso, ya que en el mundo real las mejores prácticas se complementan con el conocimiento integrado al producto final.

Carlos Jesús Fernández Salazar

student•

Intenté aplicar todos los procesos de firebase con MVP + Clean Architecture.

En el LoginActivity incluí los métodos:

    @Override
    public void startFirebaseListener() {
        presenter.startFirebaseListener();
    }

    @Override
    public void onStart() {
        super.onStart();
        presenter.onStart();
    }

    @Override
    public void onStop() {
        presenter.onStop();
        super.onStop();
    }

Estos métodos siguieron su camino a través del presenter e interactor hasta llegar al repository (también en sus interfaces) donde introduje lo siguiente:

...
public class LoginRepositoryImpl implements LoginRepository{

    LoginPresenter presenter;

    private static final String TAG = "LoginRepositoryImpl";
    private FirebaseAuth firebaseAuth;
    private FirebaseAuth.AuthStateListener firebaseAuthListener;

    public LoginRepositoryImpl(LoginPresenter presenter) {
        this.presenter = presenter;
    }

    @Override
    public void signIn(String username, String password, Activity activity) {

        firebaseAuth.signInWithEmailAndPassword(username, password)
                .addOnCompleteListener(activity, new OnCompleteListener<AuthResult>() {
                    @Override
                    public void onComplete(@NonNull Task<AuthResult> task) {
                        Log.d(TAG, "signInWithEmail:onComplete:" + task.isSuccessful());

                        // If sign in fails, display a message to the user. If sign in succeeds
                        // the auth state listener will be notified and logic to handle the
                        // signed in user can be handled in the listener.
                        if (!task.isSuccessful()) {
                            Log.w(TAG, "signInWithEmail:failed", task.getException());
                            presenter.loginError("No se ha podido iniciar");
                        } else {
                            presenter.loginSuccess();
                        }

                        // ...
                    }
                });
    }

    @Override
    public void startFirebaseListener() {
        firebaseAuth = FirebaseAuth.getInstance();
        firebaseAuthListener = new FirebaseAuth.AuthStateListener() {
            @Override
            public void onAuthStateChanged(@NonNull FirebaseAuth firebaseAuth) {
                FirebaseUser user = firebaseAuth.getCurrentUser();
                if (user != null) {
                    // User is signed in
                    Log.d(TAG, "onAuthStateChanged:signed_in:" + user.getUid());
                } else {
                    // User is signed out
                    Log.d(TAG, "onAuthStateChanged:signed_out");
                }

            }
        };
    }

    @Override
    public void onStart() {
        firebaseAuth.addAuthStateListener(firebaseAuthListener);
    }

    @Override
    public void onStop() {
        if (firebaseAuthListener != null) {
            firebaseAuth.removeAuthStateListener(firebaseAuthListener);
        }
    }
}

Corre adecuadamente en mi dispositivo. Me gustaría saber si ¿de esta forma está correcto el proceso o debería hacer alguna modificación? (En cuanto a MVP + Clean Architecture)

Luis Manuel Huaman Salvatierra

student•

yo lo hice de la misma forma espero, la profe se pueda pronunciar

Didier Zúñiga

student•

Implementé el signup en Firebase y envío un correo de validación con un enlace a la persona que se registra, pero como hago para saber si esta persona da click en el enlace o no?, ya que actualmente me esta permitiendo loguear sin necesidad de clickear en el enlace, y no debería ser así.
PD: He buscado en la doc de Firebase pero no dice nada mas allá de como enviar el correo con el enlace.
Gracias 😃

Christian Medina

student•

Encontraste alguna forma de solucionar eso?
estoy en con el mismo dilema

Ann Code

teacher•

Aquí tienes que hacer un par de validaciones en tu enlace y en tu backend.

Cuando el usuario se registre debes colocar algún tipo de bandera de que el usuario se encuentra inactivo y posteriormente cuando envíes tu enlace este debe tener la bandera en true dando la indicación de que al darle click se activará la cuenta. Interiormente valida que siempre cuando un usuario quiera acceder a la sesión su cuenta esté activa es decir en true.

Si tienes más dudas me comentas 😃

Juan Antonio Sopale

student•

He usado tanto el signInWithEmailAndPassword […] como createUserWithEmailAndPassword […] de firebase sin necesidad de pasar el Activity y funciona igual, (por si le sirve a alguien):

    @Override
    public void signIn(String username, String password) {

        firebaseAuth = FirebaseAuth.getInstance();

        firebaseAuth.signInWithEmailAndPassword(username, password).addOnCompleteListener(new OnCompleteListener<AuthResult>() {
            @Override
            public void onComplete(@NonNull Task<AuthResult> task) {
                if (task.isSuccessful()) {
                    loginPresenter.loginSuccess();
                }
                else {
                    loginPresenter.loginError("Error");
                }
            }
        });

    }

    @Override
    public void signUp(String email, String password, String name, String username) {

        firebaseAuth.createUserWithEmailAndPassword(email, password)
                .addOnCompleteListener(new OnCompleteListener<AuthResult>() {
                    @Override
                    public void onComplete(@NonNull Task<AuthResult> task) {
                        if (task.isSuccessful()) {
                            signUpPresenter.createAccountSuccess();
                        }
                        else {
                            signUpPresenter.createAccountError();
                        }
                    }
                });

    }

Juan Antonio Sopale

student•

Mejor enlazarlo a una Activity para que el listener solo este activo mientras la Activity enlazada esté “con vida”. Por tanto, en lugar pasar la Actvity como parámetro, en la interfaz del Presenter declaro un método que invoca al contexto: getContext(). De este modo accedo a la Activity que necesito para los listener de firebaseAuth.

@Override
public Context getContext() {
   return ((Activity) loginActivityView);
}

Esteban Suárez

student•

El video se corta super abrupto en el min 11:36 aprox, además que quedó faltando el tem de sign out

Esteban Suárez

student•

Buenas Tardes, quisiera pedirles el favor de colaborarme con una inquietud. Resulta que en esta clase como en la anterior veo que los elementos de Firebase se implementan en el activity (como se ejemplifica en la documentación de Firebase), o se sugiere por algunos compañeros que sean implementados en el repository siguiendo las buenas prácticas de SOLID+CLA, sugerencia con la que concuerdo. Sin embargo en clases anteriores, vimos e inclusive creamos una clase que extiende de Application y hace las veces de entry point hacía la aplicación. En la lección relacionada con dicha clase se mencionaba que allí se crearía todo lo relacionado con Firebase, pero hasta este punto la clase se encuentra vacía. Quisiera saber entonces si en la clase entry point es posible y/o recomendable inicializar los elementos de Firebase para este escenario del Auth, y de no ser así, para cuales servicios de Firebase o escenarios utilizando este backend se utilizaría entonces dicha clase?

Juan Guillermo Perez Cardozo

student•

Mis package si estan segun la logica del negocio: login, facturas, envios pero no inclui la metodologia mvp hasta saber mas. por el momento me funciona bien todo sin dar tanta vuelta y estar declarando varias veces lo mismo.

Johan Manuel Perez Soto

student•

En el caso del TAG para utilizar el metodo Log.i(); yo lo que hago es un getClass().getSimpleName() en lo personal me gusta mas de este modo

Esnor Noel Enrique Vaca Moreno

student•

Tengo este error, por favor ayuda.

12-23 21:13:08.012 7971-7971/com.platzi.platzigram E/AndroidRuntime: FATAL EXCEPTION: main
                                                                     Process: com.platzi.platzigram, PID: 7971
                                                                     java.lang.NullPointerException: Attempt to invoke virtual method 'com.google.android.gms.tasks.Task com.google.android.gms.common.api.GoogleApi.zzb(com.google.android.gms.common.api.internal.zzdd)' on a null object reference
                                                                         at com.google.android.gms.internal.zzdvv.zzb(Unknown Source)
                                                                         at com.google.android.gms.internal.zzdwc.zza(Unknown Source)
                                                                         at com.google.firebase.auth.FirebaseAuth.createUserWithEmailAndPassword(Unknown Source)
                                                                         at com.platzi.platzigram.login.view.CreateAccountActivity.createAccount(CreateAccountActivity.java:65)
                                                                         at com.platzi.platzigram.login.view.CreateAccountActivity.access$000(CreateAccountActivity.java:21)
                                                                         at com.platzi.platzigram.login.view.CreateAccountActivity$2.onClick(CreateAccountActivity.java:56)
                                                                         at android.view.View.performClick(View.java:5637)
                                                                         at android.view.View$PerformClick.run(View.java:22429)
                                                                         at android.os.Handler.handleCallback(Handler.java:751)
                                                                         at android.os.Handler.dispatchMessage(Handler.java:95)
                                                                         at android.os.Looper.loop(Looper.java:154)
                                                                         at android.app.ActivityThread.main(ActivityThread.java:6119)
                                                                         at java.lang.reflect.Method.invoke(Native Method)
                                                                         at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:886)
                                                                         at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:776)

Ann Code

teacher•

Mira parece que estás enviando un valor nullo en este punto:

CreateAccountActivity.java línea 21

Ann Code

teacher•

Muestrame esta parte 😃

José Tuzinkievicz

student•

El sing out viene después?

    @Override
    public void startFirebaseListener() {
        presenter.startFirebaseListener();
    }

    @Override
    public void onStart() {
        super.onStart();
        presenter.onStart();
    }

    @Override
    public void onStop() {
        presenter.onStop();
        super.onStop();
    }

...
public class LoginRepositoryImpl implements LoginRepository{

    LoginPresenter presenter;

    private static final String TAG = "LoginRepositoryImpl";
    private FirebaseAuth firebaseAuth;
    private FirebaseAuth.AuthStateListener firebaseAuthListener;

    public LoginRepositoryImpl(LoginPresenter presenter) {
        this.presenter = presenter;
    }

    @Override
    public void signIn(String username, String password, Activity activity) {

        firebaseAuth.signInWithEmailAndPassword(username, password)
                .addOnCompleteListener(activity, new OnCompleteListener<AuthResult>() {
                    @Override
                    public void onComplete(@NonNull Task<AuthResult> task) {
                        Log.d(TAG, "signInWithEmail:onComplete:" + task.isSuccessful());

                        // If sign in fails, display a message to the user. If sign in succeeds
                        // the auth state listener will be notified and logic to handle the
                        // signed in user can be handled in the listener.
                        if (!task.isSuccessful()) {
                            Log.w(TAG, "signInWithEmail:failed", task.getException());
                            presenter.loginError("No se ha podido iniciar");
                        } else {
                            presenter.loginSuccess();
                        }

                        // ...
                    }
                });
    }

    @Override
    public void startFirebaseListener() {
        firebaseAuth = FirebaseAuth.getInstance();
        firebaseAuthListener = new FirebaseAuth.AuthStateListener() {
            @Override
            public void onAuthStateChanged(@NonNull FirebaseAuth firebaseAuth) {
                FirebaseUser user = firebaseAuth.getCurrentUser();
                if (user != null) {
                    // User is signed in
                    Log.d(TAG, "onAuthStateChanged:signed_in:" + user.getUid());
                } else {
                    // User is signed out
                    Log.d(TAG, "onAuthStateChanged:signed_out");
                }

            }
        };
    }

    @Override
    public void onStart() {
        firebaseAuth.addAuthStateListener(firebaseAuthListener);
    }

    @Override
    public void onStop() {
        if (firebaseAuthListener != null) {
            firebaseAuth.removeAuthStateListener(firebaseAuthListener);
        }
    }
}

    @Override
    public void signIn(String username, String password) {

        firebaseAuth = FirebaseAuth.getInstance();

        firebaseAuth.signInWithEmailAndPassword(username, password).addOnCompleteListener(new OnCompleteListener<AuthResult>() {
            @Override
            public void onComplete(@NonNull Task<AuthResult> task) {
                if (task.isSuccessful()) {
                    loginPresenter.loginSuccess();
                }
                else {
                    loginPresenter.loginError("Error");
                }
            }
        });

    }

    @Override
    public void signUp(String email, String password, String name, String username) {

        firebaseAuth.createUserWithEmailAndPassword(email, password)
                .addOnCompleteListener(new OnCompleteListener<AuthResult>() {
                    @Override
                    public void onComplete(@NonNull Task<AuthResult> task) {
                        if (task.isSuccessful()) {
                            signUpPresenter.createAccountSuccess();
                        }
                        else {
                            signUpPresenter.createAccountError();
                        }
                    }
                });

    }

12-23 21:13:08.012 7971-7971/com.platzi.platzigram E/AndroidRuntime: FATAL EXCEPTION: main
                                                                     Process: com.platzi.platzigram, PID: 7971
                                                                     java.lang.NullPointerException: Attempt to invoke virtual method 'com.google.android.gms.tasks.Task com.google.android.gms.common.api.GoogleApi.zzb(com.google.android.gms.common.api.internal.zzdd)' on a null object reference
                                                                         at com.google.android.gms.internal.zzdvv.zzb(Unknown Source)
                                                                         at com.google.android.gms.internal.zzdwc.zza(Unknown Source)
                                                                         at com.google.firebase.auth.FirebaseAuth.createUserWithEmailAndPassword(Unknown Source)
                                                                         at com.platzi.platzigram.login.view.CreateAccountActivity.createAccount(CreateAccountActivity.java:65)
                                                                         at com.platzi.platzigram.login.view.CreateAccountActivity.access$000(CreateAccountActivity.java:21)
                                                                         at com.platzi.platzigram.login.view.CreateAccountActivity$2.onClick(CreateAccountActivity.java:56)
                                                                         at android.view.View.performClick(View.java:5637)
                                                                         at android.view.View$PerformClick.run(View.java:22429)
                                                                         at android.os.Handler.handleCallback(Handler.java:751)
                                                                         at android.os.Handler.dispatchMessage(Handler.java:95)
                                                                         at android.os.Looper.loop(Looper.java:154)
                                                                         at android.app.ActivityThread.main(ActivityThread.java:6119)
                                                                         at java.lang.reflect.Method.invoke(Native Method)
                                                                         at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:886)
                                                                         at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:776)