Pasar resultado de una consulta a una funcion

[richicasas]

Hola a todos. Espero que esten muy bien y me puedan colaborar con la siguiente consulta:

Tengo la siguiente funcion en un modelo:

Código PHP:

  function listar(){
            $db = new conexion();
            $db->conectar();
/*consultar() hace lo mismo que mysql_fetch_array, solo que yo lo tengo asi en caso de que me toque cambiar de motor  de BD.
*/
            $result = $db->consultar("SELECT * FROM usuarios ");
            while($row = $db->obtendatos($result))
                $result = $row[];
            return $result;
        }
/*
Intente pasar el objeto asi: return  $db->obtendatos($result); pero no me arroja nada. por eso hice el while haber si funcionaba y tampoco.
*/ 
   

Tengo el siguiente controlador

Código PHP:

  function lista(){
            require_once("modelo/usuario.php");
            $usuario = new usuario();
            $lista = $usuario->listar();
            require_once("vistas/usuario/lista.php");
        } 
   

La verdad no se que codigo colocar en la vista para que me liste todos los usuarios. Alguien me podria colaborar en como hacer que del modelo ( donde se hace la consulta de todos los usuarios) mande la lista de todos los usuarios y poder mostrarlos en la vista ???

Espero hayan entendido, muchas gracias de antemano.

[richicasas]

Bueno, al parecer ya solucione el problema. Pero quisiera saber ustedes que opinan de la solución y si saben de algun otro metodo mas eficaz. Gracias.

Modelo

Código PHP:

   
function listar(){
            $db = new conexion();
            $db->conectar();
            $result1 = $db->consultar("SELECT * FROM usuarios ");
            $i=0;
            while($row = $db->obtendatos($result1))
            {
                $result[$i]['id']     = $row['id'];
                $result[$i]['nombres'] = $row['nombres'];
                $result[$i]['apellidos']  = $row['apellidos'];
                $result[$i]['fechanaci']      = $row['fechanaci'];
                $result[$i]['correo']         = $row['correo'];
                $i++;
            }
            return $result;
        } 
   

Controlador

Código PHP:

  function lista(){
            require_once("modelos/usuario.php");
            $usuario = new usuario();
            $res = $usuario->listar();
            $numero = count($res);
            require_once("vistas/usuario/lista.php");
        } 
   

Y la vista

Código PHP:

  <?php
    for($i=0;$i<$total;$i++)
{
      echo $res[$i]['id']."";
      echo $res[$i]['nombres']."";
      echo $res[$i]['apellidos']."";
....
    
?>
<?php
     
}  
?>   

Me imprime todos los resultados como yo necesitaba.

Ahora la duda es si lo que hice esta bien o hay algo que funcione mejor o depronto viola las reglas de MVC.

Gracias.

[GatorV]

De hecho es correcto lo que haces, es pasar los datos del modelo a la vista, pero si no vas a filtrarlos o procesar algo es recomendable pasar el modelo directo a la vista para evitar hacer redundancia (como dobles ciclos).

Saludos.

[richicasas]

Lo que pasa es que en la url manejo controlador y accion como parametros, entonces siempre se llamara un controlador acompañado de una accion. Por eso lo hago asi. De todas maneras me alegra saber que voy bien encaminado. Gracias.

[richicasas]

Iniciado por richicasas

Lo que pasa es que en la url manejo controlador y accion como parametros, entonces siempre se llamara un controlador acompañado de una accion. Por eso lo hago asi.

Una pregunta:

Si yo en el modelo despues de hacer la consulta pongo algo como

Código PHP:

  require_once("vistas/lista.php"); 
   

No estaria mal visto desde la Ingenieria de Software, al mezclar vistas con modelos de una ???, pues como por ahi dice que el modelo es solo para los datos y que el controlador es el que maneja lo que se hace y lo que no.

De todas maneras me alegra saber que voy bien encaminado. Gracias.

[GatorV]

Ahí si, porque el modelo no tiene porque conocer la vista, la idea es que un modelo sea abstracto y genérico, la vista tiene que ser específica. Si tu atas un modelo a una vista, rompes precisamente con eso.

Saludos.