Problema al convertir un Select a php

[kidinshell]

Buenas, tengo un pequeño problema y la verdad es que no entiendo que es lo que está mal, yo tenía el siguiente Select.

Código PHP:

                         <?php
echo'
<select name="taric6" style="width: 108px;">
   <option selected value="'.$taric6.'"> '.$nombre_taric6.' </option>  
       <optgroup label="codigo">
       <option value="0">vacio</option> 
       <option value="1">Azulejos</option> 
       <option value="2">Cementos</option> 
       <option value="3">Expositores de aluminio</option>
       <option value="4">Paneles Aglomerados</option>
   </optgroup> 
</select>';
?>   

lo puse así porque en principio solo iban a ser 4 codigos los necesarios, pero como ahora hay mas de 20, decidí recorrer la tabla de los códigos de esta manera.

Código PHP:

  <?php
echo'<select name="taric" style="width: 108px;">
   <option selected value="'.$taric.'"> '.$nombre_taric.' </option>
   <optgroup label="codigo">';
    $codigos1 = mysql_query("SELECT * FROM codigo ORDER BY id_taric ASC") or die("Error de MySQL calidad: ". mysql_error());
            while ($valor1 = mysql_fetch_array($codigos1)) {
                
                $nombre1 = $valor1[1];
                $id1 = $valor1[0];
        
echo '
       <option value="'.$id1.'">'.$nombre1.'</option> ';
        }
echo'
   </optgroup> 
</select>';
?>   

[Triby2]

Moredadores, seria genial una seccion de obligada lectura a todos los miembros para configurar php para el desarrollo activando error_reporting y display_errors y consultar antes el manual de php en español antes de realizar cualquier consulta.

Si tuvieses eso configurado para reportar errores de programacion, estarias solucionando problemas mucho mas rapido.

Te aconsejo esta entrada: http://php.net/manual/es/mysql.examples-basic.php

Aunque tambien te recomiendo que no utilizes mysql y uses mysqli, solo vas a tener que añadir una "i" a mysql y cambiar el orden de los parametros para la conexion y las consultas.

[Triby]

Cuál es el problema?, hay diferencia en el HTML generado o qué es lo que no funciona?

[kidinshell]

Iniciado por Triby

Cuál es el problema?, hay diferencia en el HTML generado o qué es lo que no funciona?

el problema es que antes si yo seleccionaba el valor 1 azulejos por ejemplo, por post me pasaba el 1 y se guardaba bien.

al cambiar esto seleccione el que seleccione, pasa el valor 0

[Triby2]

Iniciado por kidinshell

el problema es que antes si yo seleccionaba el valor 1 azulejos por ejemplo, por post me pasaba el 1 y se guardaba bien.

al cambiar esto seleccione el que seleccione, pasa el valor 0

Y ya le has hecho un var_dump() al array que te devuelve la consulta para saber porque siempre te imprime 0 en $id1?

[KATHYU]

Yo lo que te puedo aconsejar es que no unas código HTML con PHP, separalos.

Código PHP:

<select class="form-control" style="height: 40px;" name="contrato" id="contrato" >
  <!-- Estas Variables vienen de tu selecte maestro -->
  <option value="<?php echo $contrato; ?>"><?php echo $contrato; ?></option>
  <?php
    $contrato = $mysqli->query("SELECT id,nombre FROM tipo_contrato ORDER BY nombre");
    while ($row = $contrato->fetch_array(MYSQLI_ASSOC)) {
  ?>
    <option value="<?php echo $row['id']; ?>"><?php echo $row['nombre']; ?></option>
  <?php
    }
  ?>
</select>
 

Incluso en el código anterior podemos meter los $row['id']; en variables $id = row['id']; y seria bastante bueno.

Y claro que habilites los errores, veas el código HTML generado y hagas var_dump()

[jhonnyalexander_91]

Iniciado por KATHYU

Yo lo que te puedo aconsejar es que no unas código HTML con PHP, separalos.

Código PHTML:

<select class="form-control" style="height: 40px;" name="contrato" id="contrato" >
  <!-- Estas Variables vienen de tu selecte maestro -->
  <option value="<?php echo $contrato; ?>"><?php echo $contrato; ?></option>
  <?php
    $contrato = $mysqli->query("SELECT id,nombre FROM tipo_contrato ORDER BY nombre");
    while ($row = $contrato->fetch_array(MYSQLI_ASSOC)) {
  ?>
    <option value="<?php echo $row['id']; ?>"><?php echo $row['nombre']; ?></option>
  <?php
    }
  ?>
</select>
 

Incluso en el código anterior podemos meter los $row['id']; en variables $id = row['id']; y seria bastante bueno.

Y claro que habilites los errores, veas el código HTML generado y hagas var_dump()

Metiendo un poco la cucharada y sin dar solución al error, quiero añadir una sugerencia que quizá permita mejorar la legibilidad del código. Por supuesto, estoy de acuerdo en no mezclar HTML con PHP, sin embargo, en el caso de que el programador desee hacerlo, sugiero usar [URL="https://secure.php.net/manual/en/control-structures.alternative-syntax.php"]la sintaxis alternativa para estructuras[/URL]:

Código PHP:

<select class="form-control" style="height: 40px;" name="contrato" id="contrato"
    <!-- Estas Variables vienen de tu selecte maestro -->
    <option value="<?= $contrato ?>"><?= $contrato ?></option>
    <?php $contrato = $mysqli->query("SELECT id,nombre FROM tipo_contrato ORDER BY nombre") ?>
    <?php while (list ($id, $nombre) = $contrato->fetch_array(MYSQLI_ASSOC)) : ?>
        <option value="<?= $id ?>"><?= $nombre ?></option>
    <?php endwhile; ?>
</select>
 

Con respecto al problema concreto, me gustaría entender el contexto, ya que es muy extraño que use una agrupación de opciones en un desplegable que no parece estar muy "categorizado" y que además tiene una opción por fuera de "las categorías", casi parecería que no tiene mucho que ver con el desplegable.

[KATHYU]

Iniciado por jhonnyalexander_91

Metiendo un poco la cucharada y sin dar solución al error, quiero añadir una sugerencia que quizá permita mejorar la legibilidad del código. Por supuesto, estoy de acuerdo en no mezclar HTML con PHP, sin embargo, en el caso de que el programador desee hacerlo, sugiero usar [URL="https://secure.php.net/manual/en/control-structures.alternative-syntax.php"]la sintaxis alternativa para estructuras[/URL]:

Muy bien, créame que lo tomare muy muy en cuenta

[Triby]

La única diferencia que veo entre los dos códigos es que el primer select es taric6 y el segundo es solo taric... podría ser eso?