2

No consigo hacer andar este código. Se trata de ir listando los números primos y no primos definidos en un rango mostrando la tabla de multiplicar de los que son primos. A ver si alguien me aclara sobre como definir el código.

<?php


for($i=3;$i<=7;$i++)
{
    if(primo($i))
         echo '<div style="color:green;font-weight:bold;">' .  " ".$i." es un primo ". "<br/>"   ;

         echo "<table border=5 px  hight:200px width= 200px cellspacing=5  cellpadding=5 display: block   >";
         echo "<tr><th> Tabla del $tabla </th></tr>";
         {   
   // generamos la tabla 
     elseif(primo($i)) 
        for($i=1; $i<=10; $i++) 
     {

        echo "<tr><td align=center>$tabla por $i </td>
              <td  align=center style=background-color:green > ". ($tabla*$i) . "</td>
             </tr>";
     }

    else{
        echo '<div style="color:red"; font-weight:bold;>' . " ".$i." NO es número primo " . "<br/>";


}




function primo($num)

{
    $cont=0;

    // Funcion que recorre todos los numero desde el 2 hasta el valor recibido

    for($i=2;$i<=$num;$i++)
    {
        if($num%$i==0)
        {
            # Si se puede dividir por algun numero mas de una vez, no es primo
            if(++$cont>1)
                return false;
        }
    }
    return true;

}

?>
7
  • La variable $tabla de donde la sacas? que valor tiene? Commented el 7 oct. 2018 a las 16:28
  • ahí es principalmente donde no consigo ver solución porque para que me haga la tabla de multiplicar de los números que sí son primos tendré que asignarle a la variable tabla el valor de i, no?
    – rafael
    Commented el 7 oct. 2018 a las 16:34
  • NO entiendo muy bien, aparte de obtener cual es numero primo y cual no que más necesitas? saber de que tabla de multilplicar es? Commented el 7 oct. 2018 a las 16:57
  • No, necesito que me muestre la tabla de multiplicar de los números que sí son primos.
    – rafael
    Commented el 7 oct. 2018 a las 16:59
  • 1
    a ver si te entiendo bien, si 3 es numero primero tiene que decir 3 es primo y la tabla de multiplicar es de 3 no? Commented el 7 oct. 2018 a las 17:22

2 respuestas 2

2

OK, modifique tu código ya que tenias un desorden en cuento a la hora de armar la tabla y en cuanto a la lógica, logré optimizar la función de los números primos y la tabla, puedes ver el código funcionando aquí.

for($i=3;$i<=7;$i++){

  if(primo($i)){ 

    echo '<table border="1" width="200px" cellspacing="0"  cellpadding="0">
            <tr>
              <th style="color:green;font-weight:bold;" colspan="2"> '.$i.' es un primo.</th>
            </tr>';

    for($j=1; $j<10; $j++){
      echo '<tr>
              <td align=center>'.$i.' por '.$j.' </td>
              <td  align=center style="background-color:green" > '. ($i*$j) . '</td>
            </tr>';
    }

    echo '</table>';

  }else{

     echo '<table style="border=5px;hight:200px;" width="200px" cellspacing="5"  cellpadding="5">
             <tr>
               <td align=center>$i por $i </td>
               <td  align=center style="color:red; font-weight:bold;"> '.$i.' NO es número primo</td>
             </tr>
           </table>';

  }

}


function primo($num){

  $primo = 0;

  for ($b = 1; $b < $num; $b++) {
    if ($num % $b == 0) {
        $primo++;
    }
  }

  if ($primo >= 2) {
    return false;
  } else {
    return true;
  }

}


?>

Espero que sea lo que andas buscando.

2

Necesitas repasar php y html. De verdad.

Tu código:

<?php


for($i=3;$i<=7;$i++)
{
    if(primo($i))
         echo '<div style="color:green;font-weight:bold;">' .  " ".$i." es un primo ". "<br/>"   ;

         echo "<table border=5 px  hight:200px width= 200px cellspacing=5  cellpadding=5 display: block   >";
         echo "<tr><th> Tabla del $tabla </th></tr>";
         {   
   // generamos la tabla 
     elseif(primo($i)) 
        for($i=1; $i<=10; $i++) 
     {

        echo "<tr><td align=center>$tabla por $i </td>
              <td  align=center style=background-color:green > ". ($tabla*$i) . "</td>
             </tr>";
     }

    else{
        echo '<div style="color:red"; font-weight:bold;>' . " ".$i." NO es número primo " . "<br/>";


}




function primo($num)

{
    $cont=0;

    // Funcion que recorre todos los numero desde el 2 hasta el valor recibido

    for($i=2;$i<=$num;$i++)
    {
        if($num%$i==0)
        {
            # Si se puede dividir por algun numero mas de una vez, no es primo
            if(++$cont>1)
                return false;
        }
    }
    return true;

}

?>

Errores:

  1. Todas las líneas donde aparece $tabla porque es una variable que nunca la defines.
  2. Usar i en dos bucles for seguidos. El primer bucle incrementará i, pero el segundo también lo hará y al terminar la ejecución del segundo bucle, el valor de i no será el valor del primer bucle for sino del segundo.
  3. Etiquetas html con formato o estilo sin comillas: echo "<table border=5 px hight:200px width= 200px cellspacing=5 cellpadding=5 display: block >";y etiquetas no cerradas. Te falta </div> y </table> para cerrar el los div y table que creas.

No sé si hay más, pero cuando ya vi eso paré de contar. Tu código "arreglado" quedaría así:

<?php
    function primo($num){
        $cont=0;
        // Funcion que recorre todos los numero desde el 2 hasta el valor recibido
        for($i=2;$i<=$num;$i++){
            if(($num%$i)==0){
                // Si se puede dividir por algun numero mas de una vez, no es primo
                if(++$cont>1){
                    return false;
                }
            }
        }
        return true;
    }

    for($i=3;$i<=7;$i++){
        if(primo($i)){
            echo "<table>";
            echo '<tr><th style="color:green"> '.$i." es primo</th></tr>";
            echo "<table border='5'>";
            echo "<tr><th> Tabla del ".$i."</th></tr>";
            // Generamos la tabla 
            for($k=0; $k<=10; $k++){
                echo '<tr><td align="center">'.$i." por ".$k.' </td>
                    <td  align="center" style="background-color:green"> '.($k*$i). "</td></tr>";
             }
             echo "</table>";
        }
        else{
            echo "<table>";
            echo '<tr><th style="color:red"> '.$i." no es primo</th></tr>";
            echo "</table>";
        }
        echo "<br/>";
    }
?>

Creo las table para mostrar si un número es primo o no en vez de div, para no tener que estar asignándole posiciones y eso, y queden directamente uno debajo del otro.

2
  • Muchas gracias, es verdad. Una pregunta ¿ Para introducir por teclado los números que conforman el rango,sería con los GET y POST, no ?
    – rafael
    Commented el 7 oct. 2018 a las 18:21
  • 1
    Podrías hacer un formulario que pida un rango de números, con dos tipo input, el usuario ingresaría por ejemplo en el primer input 3 y en el segundo 7, y daría como resultado lo de tu ejemplo
    – Zeugirdor
    Commented el 7 oct. 2018 a las 20:54

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.