Skip to main content
Agregar opción para saber si el usuario presionó el botón cancelar
Origen Enlace
Triby
  • 25.2k
  • 3
  • 18
  • 36

El problema es que estás enviando la alerta sin detener la ejecución de la función. Podrías agregar un return; después de cada alerta o, mejor aún, tener el flujo adecuado, ejecutando la petición AJAX solo si se introdujo un valor válido:.

Cuando se presiona el botón cancelar. prompt() devuelve null y es lo que se puede usar para evitar mostrar alertas.

function calificar(idpd, vc1) {
    // Definir una variable adicional para obtener la respuesta
    let cantresp = parseInt(prompt("Calificar del 1 al 5",1);
    // Comparación por tipo y valor (estricta, 3 signos =)
    if(resp === nul) {
        // El usuario canceló, salir de función
        return;
    }
    let cant = parseInt(resp);
    if (Number.isInteger(cant)){
        // Es un entero, analizar si se puede continuar
        if(cant >= 1 && cant <= 5) {
            // El valor está entre 1 y 5, se puede ejecutar la petición
            var paramet ={ 
                "idpd" : idpd,
                "cant" : cant,
                "vc1"    : vc1
            };
            $.ajax({
                data: paramet,
                url: './calificar.php',
                type: 'post',
                success: function (){
                    alert("Guardado");
                }
            });
        } else {
            // El valor está fuera de los límites
            alert("El valor es incorrecto, reintentar");
        }
    } else {
        // El valor no es entero
        alert("Recuerda: Sólo números");
    };
}

El problema es que estás enviando la alerta sin detener la ejecución de la función. Podrías agregar un return; después de cada alerta o, mejor aún, tener el flujo adecuado, ejecutando la petición AJAX solo si se introdujo un valor válido:

function calificar(idpd,vc1){
    let cant = parseInt(prompt("Calificar del 1 al 5",1));
    if (Number.isInteger(cant)){
        // Es un entero, analizar si se puede continuar
        if(cant >= 1 && cant <= 5) {
            // El valor está entre 1 y 5, se puede ejecutar la petición
            var paramet ={ 
                "idpd" : idpd,
                "cant" : cant,
                "vc1"    : vc1
            };
            $.ajax({
                data: paramet,
                url: './calificar.php',
                type: 'post',
                success: function (){
                    alert("Guardado");
                }
            });
        } else {
            // El valor está fuera de los límites
            alert("El valor es incorrecto, reintentar");
        }
    } else {
        // El valor no es entero
        alert("Recuerda: Sólo números");
    };
}

El problema es que estás enviando la alerta sin detener la ejecución de la función. Podrías agregar un return; después de cada alerta o, mejor aún, tener el flujo adecuado, ejecutando la petición AJAX solo si se introdujo un valor válido.

Cuando se presiona el botón cancelar. prompt() devuelve null y es lo que se puede usar para evitar mostrar alertas.

function calificar(idpd, vc1) {
    // Definir una variable adicional para obtener la respuesta
    let resp = prompt("Calificar del 1 al 5",1);
    // Comparación por tipo y valor (estricta, 3 signos =)
    if(resp === nul) {
        // El usuario canceló, salir de función
        return;
    }
    let cant = parseInt(resp);
    if (Number.isInteger(cant)){
        // Es un entero, analizar si se puede continuar
        if(cant >= 1 && cant <= 5) {
            // El valor está entre 1 y 5, se puede ejecutar la petición
            var paramet ={ 
                "idpd" : idpd,
                "cant" : cant,
                "vc1"    : vc1
            };
            $.ajax({
                data: paramet,
                url: './calificar.php',
                type: 'post',
                success: function (){
                    alert("Guardado");
                }
            });
        } else {
            // El valor está fuera de los límites
            alert("El valor es incorrecto, reintentar");
        }
    } else {
        // El valor no es entero
        alert("Recuerda: Sólo números");
    };
}
Origen Enlace
Triby
  • 25.2k
  • 3
  • 18
  • 36

El problema es que estás enviando la alerta sin detener la ejecución de la función. Podrías agregar un return; después de cada alerta o, mejor aún, tener el flujo adecuado, ejecutando la petición AJAX solo si se introdujo un valor válido:

function calificar(idpd,vc1){
    let cant = parseInt(prompt("Calificar del 1 al 5",1));
    if (Number.isInteger(cant)){
        // Es un entero, analizar si se puede continuar
        if(cant >= 1 && cant <= 5) {
            // El valor está entre 1 y 5, se puede ejecutar la petición
            var paramet ={ 
                "idpd" : idpd,
                "cant" : cant,
                "vc1"    : vc1
            };
            $.ajax({
                data: paramet,
                url: './calificar.php',
                type: 'post',
                success: function (){
                    alert("Guardado");
                }
            });
        } else {
            // El valor está fuera de los límites
            alert("El valor es incorrecto, reintentar");
        }
    } else {
        // El valor no es entero
        alert("Recuerda: Sólo números");
    };
}