Home / Questions / 444384
Accepted
Daniel Rivas
Asked: 2022-04-15 18:37:22 +0800 CST 2022-04-15 18:37:22 +0800 CST

"Variable" is undefined JavaScript

  • 772

I am calling some data from a Firebase database , the code:

db.collection("citas_registradas").where("fecha", "==", fecha).where("hora", "==", hora)
        .get()
        .then((querySnapshot) => {
            querySnapshot.forEach((doc) => {

                var fechaValidacion = fecha;
                var horaValidacion = hora;

            });
        })
        .catch((error) => {
            console.log("Error getting documents: ", error);
        });

And my validations :

if (fecha == fechaValidacion && hora == horaValidacion) {
        Swal.fire("Oops!", "La cita ya está llena, por favor, busque una hora libre y asigne la cita.", "error");
} 
else if (fecha != fechaValidacion && hora != horaValidacion || fecha == fechaValidacion && hora != horaValidacion) {
Swal.fire("¡Genial!", "La cita está disponible.", "success");
}

But when testing my app, and entering a date and time that is already registered, it gives me the following error:

"dateValidation" is undefined

Is there a way to send those variables that are used only inside the db.collection to a "global" variable to store the data , or a way to define it ? , From already thank you very much.

javascript

2 Answers

  1. Best Answer

    Your variable is there undefinedbecause it is within a local scope, which in this case would be thenif you want to access that variable from a global scope, you won't be able to because it is only available within that scope and the more nested scopes, I don't know if I can explain myself. Put your conditional inside thenand that's it

    //CODIGO PARA LLAMAR FECHA Y HORA DE LA BASE DE DATOS
db.collection("citas_registradas").where("fecha", "==", fecha).where("hora", "==", hora)
    .get()
    .then((querySnapshot) => {
        var fechaV;
        var horaV;
        querySnapshot.forEach((doc) => {

            fechaV = fecha;
            horaV = hora;

        });

        //validacion si la fecha ingresada es igual a la fecha que retorna la base de datos y la hora es igual a la hora
        //que retorna la base de datos, es por que la cita está llena.
        if (fecha == fechaV && hora == horaV) {
            Swal.fire("Oops!", "La cita ya está llena, por favor, busque una hora libre y asigne la cita.", "error");

            //si la fecha es diferente a la que retorna la base de datos y la hora igual, o la fecha es igual a la de la base
            //de datos pero la hora es diferente, registra la cita.
        } else if (fecha != fechaV && hora != horaV || fecha == fechaV && hora != horaV) {
Swal.fire("¡Genial!", "La cita está libre.", "success");

    }

    • 3 
  2. var fechaValidacion, horaValidacion;

db.collection("citas_registradas").where("fecha", "==", fecha).where("hora", "==", hora)
        .get()
        .then((querySnapshot) => {
            querySnapshot.forEach((doc) => {

                fechaValidacion = fecha;
                horaValidacion = hora;

            });
        })
        .catch((error) => {
            console.log("Error getting documents: ", error);
        });

    If you declare them like this, they would be global, accessible to the entire field. If you want to keep them private, you could put the validation inside a function and pass the variables as parameters.

    • 1