0

Estuve preguntando en SO en inglés sin ningún éxito, así que me lanzo por aquí a ver si alguien me puede ayudar.

En mi método launchFirestore() intento obtener datos en dos instancias:

  1. Obtengo una referencia calRef, la cual, según la fecha del día me busca un campo que es del tipo referencia, o sea, ese campo lo que hace es apuntar a otro documento (es lo más cercano que encuentro a las llaves foráneas de las bases de datos). Digamos que en el campo 1 de calRef tengo esa referencia a otro documento, obteniéndolo aquí: DocumentReference dataRef=calSnapshot.getDocumentReference("1");

  2. Obtengo los datos que habría en dataRef en la otra parte del código if (calSnapshot != null && calSnapshot.exists()) { dataRef.get(). ...

Eso funciona bien cuando la ruta que apunta a dataRef existe. Pero cuando no existe la App se detiene, lanzando un error NPE.

Aquí pensé que controlaba el estado de nulidad de dataRef, pero no lo hace:

                if (e != null || dataRef==null) {
                    launchVolley();
                    return;
                }

Lo que debería ocurrir según mi lógica es que cuando la ruta a que apunta dataRef no exista, se lanzaría launchVolley() para buscar los datos en el servidor. Pero no lo hace. Hice una prueba poniendo una ruta de documento que no existe e imprimiendo en el Log el snapshot. Me muestra esto:

 D/OficioActivity: DocumentSnapshot{key=liturgia/lh/1/307040501,
 metadata=SnapshotMetadata{hasPendingWrites=false, isFromCache=false},
 doc=null}

Ahí se ve claramente que doc=null, supongo que será mi ruta liturgia/lh/1/307040501 que no existe. Lo que no sé es cómo poder verificar si esa ruta no existe, para entonces lanzar Volley.

Este es el código completo de mi método.

 public void launchFirestore() {
        final FirebaseFirestore db = FirebaseFirestore.getInstance();
        String fechaDD = strFechaHoy.substring(6, 8);
        String fechaMM = strFechaHoy.substring(4, 6);
        String fechaYYYY = strFechaHoy.substring(0, 4);
        String calendarPath = "calendar";

        DocumentReference calRef = db.collection(calendarPath).document(fechaYYYY).collection(fechaMM).document(fechaDD);
        calRef.addSnapshotListener(new EventListener<DocumentSnapshot>() {
            @Override
            public void onEvent(@Nullable DocumentSnapshot calSnapshot,
                                @Nullable FirebaseFirestoreException e) {
                DocumentReference dataRef=calSnapshot.getDocumentReference("1");
                if (e != null || dataRef==null) {
                    launchVolley();
                    return;
                }

                if (calSnapshot != null && calSnapshot.exists()) {
                    dataRef.get().addOnSuccessListener(new OnSuccessListener<DocumentSnapshot>() {
                        @Override
                        public void onSuccess(DocumentSnapshot dataSnapshot) {
                            mBreviario = dataSnapshot.toObject(Breviario.class);
                            Log.d(TAG,dataSnapshot.toString());
                            showData();
                        }
                    });
                } else {
                    launchVolley();
                }
            }
        });
    }

1 respuesta 1

0

En esta línea hay un posible NPE:

DocumentReference dataRef=calSnapshot.getDocumentReference("1");

Hay un if con el que lo estas controlando pero esa instrucción queda fuera, debería estar así:

    public void launchFirestore() {
            final FirebaseFirestore db = FirebaseFirestore.getInstance();
            String fechaDD = strFechaHoy.substring(6, 8);
            String fechaMM = strFechaHoy.substring(4, 6);
            String fechaYYYY = strFechaHoy.substring(0, 4);
            String calendarPath = "calendar";

            DocumentReference calRef = db.collection(calendarPath).document(fechaYYYY).collection(fechaMM).document(fechaDD);
            calRef.addSnapshotListener(new EventListener<DocumentSnapshot>() {
                @Override
                public void onEvent(@Nullable DocumentSnapshot calSnapshot,
                                    @Nullable FirebaseFirestoreException e) {
                    if (calSnapshot != null && calSnapshot.exists()) {
                        DocumentReference dataRef=calSnapshot.getDocumentReference("1");
                        if (e != null || dataRef==null) {
                            launchVolley();
                            return;
                        }
                        dataRef.get().addOnSuccessListener(new OnSuccessListener<DocumentSnapshot>() {
                            @Override
                            public void onSuccess(DocumentSnapshot dataSnapshot) {
                                mBreviario = dataSnapshot.toObject(Breviario.class);
                                Log.d(TAG,dataSnapshot.toString());
                                showData();
                            }
                        });
                    } else {
                        launchVolley();
                    }
                }
            });
        }
9
  • Esa es precisamente la línea del NPE y lo que pregunto es cómo controlarlo.
    – A. Cedano
    Commented el 5 abr. 2019 a las 15:40
  • Edito la respuesta con el código completo. Estabas accediendo a calSnapshot sin haber comprobado primero que fuera distinto de nulo, ahora dentro del if que lo comprueba ya no deberías tener problema.
    – Ydrojen
    Commented el 5 abr. 2019 a las 15:59
  • Esto da el mismo resultado. El problema no es calSnapshot, ese objeto no es nulo porque lo obtengo correctamente. Lo que no puedo llegar a controlar es que el campo de tipo reference que guardo en calSnapshot no sea nulo. Con tu código tengo igualmente un NPE. Y en el Log calSnashot muestra esto: DocumentSnapshot{key=liturgia/lh/1/03070407, metadata=SnapshotMetadata{hasPendingWrites=false, isFromCache=false}, doc=null} Aquí se ve el valor de mi campo de tipo reference, que es este: liturgia/lh/1/03070407 yo puse ahí un documento que no existe a propósito, pero no logro saber...
    – A. Cedano
    Commented el 5 abr. 2019 a las 20:36
  • ...algún modo de controlar que en esa referencia no hay nada. Yo veo esto en el Log: doc=null y me pregunto si no hay alguna forma de comprobar ese valor de doc ¿?
    – A. Cedano
    Commented el 5 abr. 2019 a las 20:37
  • @A.Cedano existe un método calSnapshot.contains("1"), creo que te puede valer ;)
    – Ydrojen
    Commented el 8 abr. 2019 a las 7:16

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.