El problema de utilizar typeof
es que un valor NaN
es del tipo number
. Si esto te suena raro, puedes leer más sobre eso.
Por ejemplo, si el usuario no ingresa nada en el prompt. La primera condición será válida.
Para evitar, esto podemos expandir la primera condición:
if (typeof cambioLimite === "number" && !Number.isNaN(cambioLimite)) {
}
Number.isNaN()
devuelve true
si el valor es NaN
(Not a Number) y false
si el no lo es. Por eso lo estamos negando !
en el ejemplo de arriba. Básicamente estamos diciendo:
"Si cambioLimite
es del tipo number
y ademas no es NaN
"
Sin embargo, como Number.isNaN()
ya nos dice si el valor es NaN
, podemos simplificar el primer condicional:
if (!Number.isNaN(cambioLimite)) {
}
También es recomendable revisar que el valor ingresado sea mayor a cero. Ver ejemplo completo:
let limiteExtraccion = 10;
function cambiarLimiteDeExtraccion() {
let cambioLimite = parseInt(prompt("Ingrese nuevo limite de extracción"));
if (!Number.isNaN(cambioLimite) && cambioLimite > 0) {
limiteExtraccion = cambioLimite;
alert(`Su nuevo limite de extracción es: ${limiteExtraccion}`);
} else if (cambioLimite <= 0 ) {
alert("ingrese un valor mayor a cero.");
} else {
alert("Tiene que ingresar un valor númerico.");
}
}
cambiarLimiteDeExtraccion();
Acerca de Number.isNaN()
vs. isNaN()
Unos de los problemas de la versión global isNaN()
es la coerción de cadenas de texto a números. Por eso se recomienda el uso de Number.isNaN()
.