[SOLUCIONADO] PHP OO Tomar valores de una base de datos y colocarlos en un formulario

[matiD]

Como están?

A ver si pueden darme una ayuda para algo creo que sencillo.

Necesito tomar los valores de una tabla y colocarlos en un formulario en un select

Este sería el formulario a llenar

Código HTML:

<select name="Locales" size="1" simple>
<option value="valor 1">Texto de opción 1</option>
<option value="valor 2" selected>Texto de opción 2</option>
<option value="valor 3">Texto de opción 3</option>
<option value="valor 4">Texto de opción 4</option>
</select>
 

En valor 1 debería colocar el idlocal que tomo de la BD y en Texto de opción 1 el valor que tomo de nombre de la BD y así sucesivamente con los 4 locales.

Estoy creando una clase con las operaciones a realizar en MySQL puedo tomar bien los valores, pero no se como pasarlos al archivo registro.php para poder utilizarlo.

Si imprimo desde el metodo creado en la clase me funciona bien me muestra los locales pero realmente me gustaría crear el bucle en el archivo que contiene el formulario... Creo que es mejor o no???

esta es la clase

Código PHP:

class OpMySQL
{
    public $tabla;
    
    //Tomamos los locales para colocarlos en el formulario
    public function tomarLocalesParaForm()
    {
        //Preparamos la conexion a la BD
        $con = new mysqli(DB_HOST, DB_USUARIO, DB_PASSWORD, DB_DATABASE);
        //Creamos la consulta
        $sql = 'SELECT id_local, nombre FROM locales';
        
        //Preparamos la sentencia
        $pre = $con->prepare($sql);
        //Ejecutamos la sentencia
        $pre->execute();
        //Definimos las variables para guardar los valores devueltos por la consulta
        $pre->bind_result($idlocal, $nombrelocal);
        
        
    }
 

Lo único que se me ha ocurrido es colocar un bucle while y utilizar $pre->fetch para que mientras va realizando el bucle vaya guardando los datos en un array asociativo, y luego hago un return de dicho array, y creo otro bucle while que vaya recorriendo el array en la página del formulario y lo vaya llenando.

Pero tal vez hay alguna solución más sencilla.

Desde ya muchas gracias!

[NSD]

Código PHP:

class OpMySQL
{
    public $tabla;
    
    private $_con;
    private $_stmt;
    
    public function __construct()
    {
        //Preparamos la conexion a la BD
        $this->_con = new mysqli(DB_HOST, DB_USUARIO, DB_PASSWORD, DB_DATABASE);
    }
    
    //Tomamos los locales para colocarlos en el formulario
    public function tomarLocalesParaForm()
    {
        //Creamos la consulta
        $sql = 'SELECT id_local id, nombre descripcion FROM locales';        
        //Preparamos la sentencia
        $this->_stmt = $this->_con->prepare($sql);
        //Ejecutamos la sentencia
        $this->_stmt->execute();
    }
    
    public function lastQueryToCombo($params = array())
    {
        $combo = '<select id="'.$params['id'].'" name="'.$params['name'].'" '.($params['multiple'] ? 'multiple' : '').' '.($params['disabled'] ? 'disabled' : '').'>'
        $resultado = $this->_stmt->get_result();
        while($fila = $resultado->fetch_assoc())
        {
            $combo .= '<option value="'.$fila['id'].'" '.($params['default']==$fila['id'] ? 'selected' : '').'>'.$fila['descripcion'].'</option>';
        }   
        $combo .= '</select>';
        return $combo;
    }
}
 

y lo llamas asi:

Código PHP:

$ob = new OpMySQL();
$ob->tomarLocalesParaForm();
echo($ob->lastQueryToCombo());
 

si lo emprolijas un poco, este metodo te servira para cualquier otra cosa que quieras hacer, se puede hacer de 10 maneras diferentes mas, solo es una idea

[matiD]

Si así es como lo había pensado en primer momento, lo que no quería era hacer les impresiones desde la función en el archivo opmysql.

Mi idea era dejar el archivo de operaciones mysql solo para operaciones con la BD y no con impresiones.

Pero me resulta funcional si se me ocurre alguna otra idea lo cambiaré luego...

Además de eso muchas gracias por una aclaración que me has hecho sin yo haber preguntado, se me estaba complicando un poco saber que objetos declarar en la clase opmysql, por lo que veo declaras prácticamente todas las variables uqe se van a utilizar más de 1 vez. yo no sabía si estaba bien declarar el objeto con y el stmt y aparte muy logico de crear el metodo de conexión para no tener que escribirlo en cada metodo que cree luego para otras consultas.

Muchas gracias!

[Heli0s]

Esta muy bien planteado no querer tener código HTML en tus librerías, las hace muchísimo más reusables.

La solución es muy parecida a la que te ha dado el usuario NSD, pero en vez de hacer el bucle en la clase, lo que haría seria que el método que recoge la información de la base de datos, la devuelva sin ningun tipo de HTML

Código PHP:

  $resultado = $this->_stmt->get_result();
return $resultado;  
   

Y después en la vista donde quieras implementar eso, ya sería hacer el bucle, pero en la vista, no en la clase

Código PHP:

  $datos = $conection->lastQueryToCombo();
while($fila = $datos->>fetch_assoc())
        {
            //se imprimen los option
        }  
   

El código es solo un ejemplo, ya tu lo implementas con los nombres de variables/metodos correctos etc, espero que se haya entendido.

Un saludo

[matiD]

Iniciado por Heli0s

Esta muy bien planteado no querer tener código HTML en tus librerías, las hace muchísimo más reusables.

La solución es muy parecida a la que te ha dado el usuario NSD, pero en vez de hacer el bucle en la clase, lo que haría seria que el método que recoge la información de la base de datos, la devuelva sin ningun tipo de HTML

Código PHP:

  $resultado = $this->_stmt->get_result(); 
return $resultado;  
   

Y después en la vista donde quieras implementar eso, ya sería hacer el bucle, pero en la vista, no en la clase

Código PHP:

  $datos = $conection->lastQueryToCombo(); 
while($fila = $datos->>fetch_assoc()) 
        { 
            //se imprimen los option 
        }  
   

El código es solo un ejemplo, ya tu lo implementas con los nombres de variables/metodos correctos etc, espero que se haya entendido.

Un saludo

Muchas gracias a ambos, ya puedo marcar el tema como solucionado.

Espero no molestar a los usuarios del foro con tantas consultas

[NSD]

Esta muy bien planteado no querer tener código HTML en tus librerías, las hace muchísimo más reusables.

Tienes toda la razon, cuanto menos html tengas es mejor, pero hay un detalle importante:

Código PHP:

$datos = $conection->lastQueryToCombo();
 

Con eso estarias sacando una propiedad privada de la clase al exterior, rompiendo completamnete el concepto de encapsulamiento, ya no puedes asegurar que la clase se va a comportar ciempre igual.

Poner html dentro del metodo es una chapuza pero que mantiene al objeto encapsulado, una forma realmente correcta de hacerlo seria asi:

Código PHP:

<?php
class utiles 
{
    public static selectStmtToCombo(mysqli_stmt $stmt, $params=array())
    {   
        $combo = '<select id="'.$params['id'].'" name="'.$params['name'].'" '.($params['multiple'] ? 'multiple' : '').' '.($params['disabled'] ? 'disabled' : '').'>'
        $resultado = $stmt->get_result();
        while($fila = $resultado->fetch_assoc())
        {
            $combo .= '<option value="'.$fila['id'].'" '.($params['default']==$fila['id'] ? 'selected' : '').'>'.$fila['descripcion'].'</option>';
        }   
        $combo .= '</select>';
        return $combo;
    }
}
 
 
class OpMySQL
{
    public $tabla;
    
    private $_con;
    private $_stmt;
    
    public function __construct()
    {
        //Preparamos la conexion a la BD
        $this->_con = new mysqli(DB_HOST, DB_USUARIO, DB_PASSWORD, DB_DATABASE);
    }
    
    //Tomamos los locales para colocarlos en el formulario
    public function tomarLocalesParaForm()
    {
        //Creamos la consulta
        $sql = 'SELECT id_local id, nombre descripcion FROM locales';        
        //Preparamos la sentencia
        $this->_stmt = $this->_con->prepare($sql);
        //Ejecutamos la sentencia
        $this->_stmt->execute();
        return  $this->_stmt;
    }    
}
 

Código PHP:

$ob = new OpMySQL();
echo(utiles::selectStmtToCombo($ob->tomarLocalesParaForm()));
 

De esta forma, el html esta en una clase estatica llamada utiles, altamente reusable y mantenible que sirve para todas las clases del sistema, cambiando los parametros que recibe se puede personalizar el select tanto como se quiera.
Si bien la propiedad privada _stmt sale de la clase, no queda suelta en el sistema, sino que va a una clase estatica, el dia que se modifique el comportamiento de la clase, solo hay dos lugares para revisar, no todos los scripts que muestren un combo, saludos.

[Heli0s]

Te equivocas NSD, lo que sale no es el objeto privado, sino los datos que devuelve su metodo get_result(), es una practica muy tipica poner getters / setters de elementos privados para limitar su uso al externo, es completamente factible usarlo asi, y lo veo muchisimo mas reusable que metiendo los option, ninguna libreria debe llevar ninguna etiqueta html, nunca, es el método correcto, de hecho así se hace en MVC.

Un saludo

[NSD]

es el método correcto, de hecho así se hace en MVC.

No te olvides lo que es un patrón de diseño: "Es una solución genérica a un tipo de problema en un tipo de contexto definido".
Si el problema es la administración del código, en el contexto de que muchas personas trabajan en el simultáneamente en un gran proyecto, MVC es la mejor solución, fuera de ese contexto, hay otros patrones mejores.

Si se nos dijo que MVC es lo mejor o lo correcto, no hay que repetirlo en todos lados, hay que analizar primero.

En este caso por MVC tendrías:

La clase OpMySQL.
Un controlador que agarra ese conjunto de resultados y los pasa a un array.
Una vista que recibe un array con los resultados y los muestra.
El procedimiento que los arma.

Si tubieses que armar otra clase que tambien muestra un resultado en un combo en otro lado tendrias:
La clase OpMySQL2.
El mismo controlador anterior.
Otra vista.
Otro procedimiento que los arma.

en terminos generales: 3*N+4 Lugares para cambiar codigo.

Si usas una clase estatica tienes:
La clase OpMySQL.
La clase estatica.
El procedimiento que los arma.

Si tubieses que armar otra clase que tambien muestra un resultado en un combo en otro lado tendrias:
La clase OpMySQL2.
La misma clase estatica.
Otro procedimiento que los arma.

en terminos generales: 2*N+3 Lugares para cambiar codigo.

Si en MVC quisieras cambiar como se muestra el select a lo largo de todo el sitio, tendrias que cambiar todas las vistas.
Si usando la clase estatica quisieras cambiar como se muestra el select a lo largo de todo el sitio, tendrias que cambiar en un solo lugar.


Si en MVC quisieras cambiar como se muestra un select determinado, terndrias que cambiar una sola vista.
Si usando la clase estatica quisieras cambiar como se muestra un select determinado tienes que cambiar un solo procedimiento.

La solucion correcta es la que mejor se adapta al contexto. Si nada sabemos del contexto como en este caso, decir que la forma correcta es como se haria en el patron MVC es poco acertado.

PD: Si se estuviera usando un patron MVC, la clase OpMySQL NUNCA contendria en su interior funciones nativas mysqli, en su lugar, usaria un controlador de una clase gestora de bases de datos, que usaria un controlador de otra clase que usaría mysqli

Saludos.

[Heli0s]

No he dicho que MVC sea absolutamente lo mejor y lo único que se deba usar, la solución correcta es la que mejor se adapta al contexto, esas son tus palabras, el usuario pedía una clase sin HTML para que sea mas reutilizable, y esa fue la solución que le dí, y la más reusable, ya que podría usarlo para mostrar esos mismos datos donde fuese, en una lista de options, en una tabla o sin formato.

Un saludo