Skip to main content
se añadieron 23 caracteres en el cuerpo
Origen Enlace
Alvaro Montoro
  • 49.8k
  • 28
  • 109
  • 190
<!doctype html>
<html>

<head>
  <title>numeros</title>
</head>

<body>
  <h2>N&uacute;meros</h2>
  <!--Título de la página-->

  <p>N&uacute;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]);
        cont = 0;
        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>
<!doctype html>
<html>

<head>
  <title>numeros</title>
</head>

<body>
  <h2>N&uacute;meros</h2>
  <!--Título de la página-->

  <p>N&uacute;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>
<!doctype html>
<html>

<head>
  <title>numeros</title>
</head>

<body>
  <h2>N&uacute;meros</h2>
  <!--Título de la página-->

  <p>N&uacute;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]);
        cont = 0;
        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>
cuerpo editado
Origen Enlace
Alvaro Montoro
  • 49.8k
  • 28
  • 109
  • 190

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á declarandoutilizando como una variable global! y los cambios en un bucle también afectan al otro.

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.

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á utilizando como una variable global! y los cambios en un bucle también afectan al otro.

Origen Enlace
Alvaro Montoro
  • 49.8k
  • 28
  • 109
  • 190

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&uacute;meros</h2>
  <!--Título de la página-->

  <p>N&uacute;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>