Problema con PHP y MySQL

[drieraperez]

Buenas Tardes,

Necesito listar en dos columnas unos checkbox con sus nombres, esto sería las categorias de un sistema para poder seleccionar sus categorias y gardarlas en la base de datos.

Hasta aquí perfecto.

El problema es que cuando actualizo un articulo que se encuentra en la categoría con id 29, también se selecciona la categoría 9. (Pero no la 19).

Utilizo la función de PHP strstr(), dejo aquí el código para ver si me podéis ayudar.

Código:

 <?php $result = mysql_query("SELECT * FROM familias WHERE padre != '0'");  
                $i=0; 
        if ($row = mysql_fetch_array($result)){  
        echo "<table border = '0'> \n";  
               echo "<tr>"; 
               do { 
			  if (strstr($fami,$row['id'])) {
			   $marcar = "checked=\"checked\"";
			  }else{
			 $marcar = "";
			  }
                     echo "<td><p><input name='familia[]' ".$marcar." id='familia[]' type='checkbox' value='".$row['id']."' />".$row['nombre']."</p></td> \n";  
                     $i++; 
                 if($i%2==0){ 
                          echo "</tr><tr>"; 
                  } 
               } while ($row = mysql_fetch_array($result));  
           echo "</tr></table> \n";  
         } else {  
        echo "<center>No hay categorias que mostrar</center>";  
        }
        ?>

La variable $fami es el campo de familia en la tabla artículos de la base de datos por lo tanto contiene un 29.

Espero su ayuda muchas gracias.

[repara2]

y por qué no comparas así

if ($fami==$row ['id']) {

?

[drieraperez]

Porque la variable $fami contiene varios numero que hacen referencia a la categoría, ejemplo: 48,13,3.

Y el $row['id'] saca el id de cada categoría.

Gracias.

[Nemutagk]

Mmm y porque no usar por ejemplo...

if (in_array($row['id'], explode($fami,',')){}

vamos, no es lo mas recomendable, pero así como lo tienes es la forma mas "sencilla", lo mejor sería que desde un comienzo $fami sea un array para evitar la conversión en cada loop...

Por ultimo, no entiendo la manía (no lo digo por ti, si no en general) de usar doWhile cuando este no es lo correcto al recorrer registros de una base de datos...

[drieraperez]

Iniciado por Nemutagk

Mmm y porque no usar por ejemplo...

if (in_array($row['id'], explode($fami,',')){}

vamos, no es lo mas recomendable, pero así como lo tienes es la forma mas "sencilla", lo mejor sería que desde un comienzo $fami sea un array para evitar la conversión en cada loop...

Por ultimo, no entiendo la manía (no lo digo por ti, si no en general) de usar doWhile cuando este no es lo correcto al recorrer registros de una base de datos...

Hola Nemutagk, no me ha funcionado, lo he solucionado "reservando" los 10 primeros id, y por ahora con las categorías que tengo funciona bien.

Utilizo el DoWhile porque empecé así, ¿Me podrías decir que forma es la correcta para recorrer registros?

Muchas Gracias por tu ayuda

[repara2]

Entonces convierte $fami en un array y compara con in_array(), no se entiende dónde está la dificultad.

[Nemutagk]

Iniciado por drieraperez

Hola Nemutagk, no me ha funcionado, lo he solucionado "reservando" los 10 primeros id, y por ahora con las categorías que tengo funciona bien.

Utilizo el DoWhile porque empecé así, ¿Me podrías decir que forma es la correcta para recorrer registros?

Muchas Gracias por tu ayuda

De que no te funciona primero di el PORQUE no funciona, y luego muestra como lo has implementado...

Lo de doWhile es por una sencilla razón, pero es difícil entender sin un ejemplo, supongamos que quieres obtienes X registros de una base de datos pero aun no hay registros en la base de datos, por lo tanto tu sentencia SQL te devuelve 0 objetos, que pasa con while y doWhile en este caso:

a) while primero verifica si tiene registros que mostrar ANTES de hacer el primer loop, por lo tanto si no hay registros while detiene su funcionamiento

b) doWhile primero hace una pasada SUPONIENDO que existe al menos UN REGISTRO, pero como no hay registros es seguro que arrojará un alert como menos ya que intentarás acceder a datos que no existen, ya que termino de dar la primera vuelta es entonces que while verifica si hay mas datos que mostrar