Funcion para recuperar datos de una bbdd

[rafamadrile]

Hola.

He realizado una funcion que realiza dos operaciones. una leer los datos de una bbdd y otra verificar si existe una imagen.

El tema es que creo que la variable contador que he insertado es un poco "chapuza".

¿Alguien sabe como podría omitir dicha variable?

Adjunto el codigo

Código PHP:

  function mostrar_productos($id_cat){ 
    $mostrar = array(); 
    $busca = @mysql_query("SELECT * FROM productos WHERE id_categoria LIKE '$id_cat'"); 
    $resultados = mysql_num_rows($busca); 
    if ($resultados != 0){ 
        $contador = 0; 
        while ($arrbusca = mysql_fetch_assoc ($busca)){ 
            $mostrar[] = $arrbusca; 
 
            if (file_exists('img_productos/producto'.$arrbusca['id'].'_1.jpg')){ 
                $mostrar[$contador]['imagen'] = 'existe imagen'; 
            }else{ 
                $mostrar[$contador]['imagen'] = 'no existe imagen'; 
            } 
            $contador++; 
        } 
    }else{ 
        $mostrar[] = 'No hay productos'; 
    }     
    return $mostrar; 
} 
//-------------------------------- 
foreach (mostrar_productos(1) as $value) { 
    echo '<strong>'.$value['imagen'].'</strong><br />'; 
    echo $value['nombre_producto'].'<br />'; 
    echo $value['unidades_producto'].'<br />'; 
    echo $value['descripcion_producto'].'<br />'; 
}  
   

[maycolalvarez]

no hace falta el contador, puedes dejar los corchetes vacíos, automáticamente php asignará el indice incrementalmente

[Ribon]

maycolalvarez, eso mismo pensaba yo, pero es fundamental saber el índice para poner el otro índice "imagen" en la posición correcta.

No veo que problema te hace el uso de ese índice.

Pero si realmente te estorba prueba con este código

Código PHP:

if ($resultados != 0){ 
  while ($arrbusca = mysql_fetch_assoc ($busca)){ 
 
         if (file_exists('img_productos/producto'.$arrbusca['id'].'_1.jpg')){ 
             $arrbusca['imagen'] = 'existe imagen'; 
         }else{ 
             $arrbusca['imagen'] = 'no existe imagen'; 
         }
         $mostrar[] = $arrbusca;
 
     } 
}else{ 
     $mostrar[] = 'No hay productos'; 
}
 

Saludos.

[Henkka]

Iniciado por Ribon

maycolalvarez, eso mismo pensaba yo, pero es fundamental saber el índice para poner el otro índice "imagen" en la posición correcta.

Por que? No hay tal cosa como posicion correcta. $contador va desde 0 hasta n, siendo n la cantidad de productos obtenido. $contador no corresponde a producto.id, cosa bastante inintuitiva.

Tu solucion cambia la logica del programa del OP, al corregir lo que mencione al final.

[Ribon]

No entendí tu crítica xD!
yo nunca confundí producto.id con $contador o.o
yo hablaba de índices de array.

Saludos.

y mi solución lo que hace es agregarle el índice 'imagen' al array que contiene el fetch result para luego asignarlo al array mostrar que contendrá todo al final.

Saludos.

[maycolalvarez]

existen funciones para devolver el indice como array_keys , además con el mismo foreach te lo da:

Código PHP:

  foreach (mostrar_productos(1) as $key => $value) {
    echo '<strong>'.$value['imagen'].'</strong><br />';
    echo $value['nombre_producto'].'<br />';
    echo $value['unidades_producto'].'<br />';
    echo $value['descripcion_producto'].'<br />';
}  
   

$key => $value


además la línea:

$mostrar[] = $arrbusca;

deberías asignarla al final y ser $arrbusca la que modifiques, así:

Código PHP:

    //...
            if (file_exists('img_productos/producto'.$arrbusca['id'].'_1.jpg')){ 
                $arrbusca['imagen'] = 'existe imagen'; 
            }else{ 
                $arrbusca['imagen'] = 'no existe imagen'; 
            } 

            $mostrar[] = $arrbusca;  
   

recuerda que no es necesario el contador, porque el índice siempre inicia de 0, exactamente lo que hace tu variable contadora, realmente NO lo necesitas, estás creando más código innecesario

[Ribon]

Lo de modificar arrbusca, eso fue lo que hice en mi ejemplo

Saludos

[rafamadrile]

Muchas gracias a todos

dejo el codigo como ha quedado al final...

Código PHP:

  function mostrar_productos($id_cat){ 
    $mostrar = array(); 
    $busca = @mysql_query("SELECT * FROM productos WHERE id_categoria LIKE '$id_cat'"); 
    $resultados = mysql_num_rows($busca); 
    if ($resultados != 0){ 
        while ($arrbusca = mysql_fetch_assoc ($busca)){ 
            if (file_exists('img_productos/producto'.$arrbusca['id'].'_1.jpg')){ 
                $arrbusca['imagen'] = 'existe imagen'; 
            }else{ 
                $arrbusca['imagen'] = 'no existe imagen'; 
            } 
            $mostrar[] = $arrbusca; 
        } 
    }else{ 
        $mostrar[] = 'No hay productos'; 
    }     
    return $mostrar; 
} 
//-------------------------------- 
foreach (mostrar_productos(1) as $value){ 
    if (is_array($value)){ 
        echo '<pre>'; 
        print_r($value); 
        echo '</pre>'; 
    }else{ 
        echo $value; 
    } 
}  
   

[maycolalvarez]

@Ribon, cierto, no lo vi!!! +karma