Esto es un problema con el alcance de las variables. Si te fijas, utilizas el mismo iterador x
en dos bucles diferentes. Pero como no lo estás declarando (con var
o let
) te encuentras con que ¡se está declarando como una variable global! y los cambios en un bucle también afectan al otro.
Como nota aparte, deberías limitar el alcance de tus variables y funciones para evitar problemas de este tipo. Tal y como tienes tu código ahora mismo, hay varias variables globales que podrían dar problemas. Lee esta otra pregunta sobre variables globales y por qué deberías evitarlas siempre que sea posible.
¿Qué ocurre entonces? que el primer bucle (el de CantidadPrimos
) se ve afectado por los cambios que se producen en el segundo bucle (el de NumerosPrimos
) y se ejecuta menos veces de las que esperas (en concreto para los números 3, 5 y 7, se ejecutará sólo una vez, para el 3, y por eso recibes un 1 como resultado).
La solución rápida: añade var
o let
cuando uses la variable en los bucles. De ese modo se estará limitando su alcance a la función o al bucle respectivamente y no te encontrarás este problema:
for (var x = 0; x < list.length; x++) {
y
for (var x = 2; x < numero; x++) {
Aquí puedes ver tu código con ese cambio y ya funciona correctamente:
<!doctype html>
<html>
<head>
<title>numeros</title>
</head>
<body>
<h2>Números</h2>
<!--Título de la página-->
<p>Números primos: <input type="text" id="primos" name="primos"></p>
<input type="button" value="Mostrar" onclick="ejecutarFuncion();">
<!--Comienza código javascript-->
<script>
//Creo un array vacío llamado "numeros" y posteriormente le voy añadiendo los números introducidos por pantalla
var numeros = [];
var cont = 0;
var i = 0;
var x = prompt("Dime un n\u00FAmero. Pulsa -1 para salir");
while (x != -1) {
numeros.push(x);
x = prompt("Dime otro número. Pulsa -1 para salir");
}
//Función que recibe como parámetro una lista de números y muestra la cantidad de números primos
function CantidadPrimos(list) {
var contador = 0;
var num = 0;
for (var x = 0; x < list.length; x++) {
num = parseInt(list[x]);
NumerosPrimos(num);
if (cont == 0) {
contador = contador + 1;
}
}
document.getElementById('primos').value = contador;
}
//Función que recibe como parámetro un número y nos permite conocer si éste es primo o no
function NumerosPrimos(numero) {
if (numero > 1) {
for (var x = 2; x < numero; x++) {
if (numero % x === 0) {
cont = cont + 1;
}
}
} else {
cont = 1;
}
}
//Función que ejecuta el resto de funciones anteriores, y que será llamada por evento onclick del botón "Mostrar" de nuestra página
function ejecutarFuncion() {
CantidadPrimos(numeros);
}
</script>
</body>
</html>