Mejora de codigo

[Reco-X]

Tengo una funcion que lo que hace es pedir los elementos de una base de datos y los guarda en un array entonces despues ese array lo desordeno y guardo la primer posicion en una variable, hago esto ultimo 2 veces mas y luego las 3 variables que tengo las guardo en un array de 3 posiciones y retorno este ultimo array

Código PHP:

      public function obtenerItems(){
            $arrayItems=array();
            $link= new Conexion();
            $link->open();
            $resultado = $link->query("SELECT * from items");
            while ($row = mysql_fetch_array($resultado)){
                $item = new Item ($row["nombre"],$row["valor"]);
                $arrayItems[]=$item;
                shuffle ($arrayItems);
                $valor1 = $arrayItems[0];
                shuffle ($arrayItems);
                $valor2 = $arrayItems[0];
                shuffle ($arrayItems);
                $valor3 = $arrayItems[0];
                $arrayValores = array ($valor1, $valor2, $valor3);
            }
            $link->close();
            return $arrayValores;    
        }  
   

[Triby]

Y qué es lo que pretendes hacer con tanto shuffle?

Supongo que si en la consulta agregas un ORDER BY RAND() LIMIT 3 después sólo necesitas cargar los 3 registros en un array y devolverlo.

- ORDER BY RAND() seleccionará los registros al azar, ya no necesitarías el shuffle, además de que en tu código podrías enviar lo mismo en las 3 posiciones.
- LIMIT 3 hará que la consulta te devuelva sólo 3 registros.
- UPS! No me había fijado que ese proceso lo repites en cada iteración del while!

[rigobcastro]

Como dice Triby con la función RAND() de SQL se puede ejecutar un orden aleatorio, sin embargo, no es recomendable ya que consume muchos recursos. Claro, siempre y cuando no sean mas de 10 datos dentro de la tabla no habrá problema...

http://systemadmin.es/2010/11/order-...-es-buena-idea

El Shuffle lo puedes usar una vez obteniendo los datos de la consulta.

Código PHP:

$row = mysql_fetch_array($resultado);
shuffle($row);
 

Sería algo mucho más fácil si necesitas un random rapido... Personalmente lo uso para los "recomendados" de las secciones.