Menú dinamico en mvc

[stevens82]

Hola compañeros aca con una duda, estoy desarrollando un menú que traigo de la BD, por ahora lo tengo en un solo archivo y lo necesito separar en modelo-vista-controlador, el gran problema con el que me encontre es que tiene una funcion recursiva y por ahi no se como llamarlo en la vista aca les dejo lo que tengo por si alguien me de una idea.

esta es la BD:

Código SQL:

CREATE TABLE IF NOT EXISTS `menu` (
  `id` INT(11) NOT NULL AUTO_INCREMENT,
  `id_parent` INT(11) NOT NULL DEFAULT '0',
  `link` VARCHAR(255) NOT NULL,
  `order` INT(11) NOT NULL DEFAULT '0',
  `title` VARCHAR(255) CHARACTER SET utf8 COLLATE utf8_polish_ci NOT NULL DEFAULT '',
  `level` tinyint(4) NOT NULL DEFAULT '0',
  PRIMARY KEY (`id`)
) ENGINE=InnoDB  DEFAULT CHARSET=latin1 AUTO_INCREMENT=24 ;
 
 
 
INSERT INTO `menu` (`id`, `id_parent`, `link`, `order`, `title`, `level`) VALUES
(12, 0, '', 0, 'Office Furniture', 0),
(13, 12, '', 0, 'Chairs', 0),
(14, 12, '', 0, 'Work Tables', 0),
(15, 12, '', 0, 'Workstations', 0),
(16, 12, '', 0, 'Storage', 0),
(17, 12, '', 0, 'Conference Tables', 0),
(18, 13, '', 0, 'with spagh', 0),
(19, 13, '', 0, 'without spagh', 0),
(20, 18, '#', 0, 'Triangle', 0),
(21, 18, '#', 0, 'Sqare', 0),
(22, 19, '#', 0, 'Simple', 0),
(23, 19, '#', 0, 'Complex', 0);
 

Y este el script php:

Código PHP:

   
$db = new PDO('mysql:dbname=db_menu', 'root', '123456', 
array(PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES utf8")); 
 
function createsubmenu($item) 
{ 
    global $db; 
    $subsel = $db->prepare('SELECT id, link, title, level  FROM menu WHERE id_parent = :parent ORDER BY `order`'); 
 
    $subsel->execute(array('parent' => $item)); 
    $text = ''; 
 
        foreach ($subsel as $i) { 
 
        $text .= '<li class="menucolor' . ($i['level'] - 1) . '">' 
            .'<a href="' . htmlspecialchars($i['link']) . '">' . htmlspecialchars($i['title']) . '</a>' 
            . createsubmenu($i['id']) . '</li>'; 
        } 
 
    if ($text == '') { 
        return ''; 
    } 
 
    return '<ul id="red" class="treeview-red">' . $text . '</ul>'; 
} 
 
echo createsubmenu(0);  
   

Gracias de antemano por su tiempo y ayuda.

[stevens82]

Ningun alma bondadosa que me ayude con alguna idea?

[Triby2]

Las almas estás pero quizás no en el momento cuando las necesitamos jejejeje

Tu problema es que aún no estás concibiendo el funcionamiento del patrón MVC. Desde las vistas no se llaman a las funciones, ni tienen lógica incrustada. Eso se realiza desde los controladores y desde ahí se pasan los resultados a las vistas, que por eso lo llaman "vista".

Después tienes que obtener primero todos los menús padres, recorrerlos y conseguir de cada uno los hijos para imprimirlos y ya.

Yo lo haría de este modo:

Código PHP:

#El siguiente código se debe de colocar en el controlador correspondiente dentro de la función que le corresponda que en este caso podría ser index() dentro del HomeController por poner un ejemplo de controlador.
 
# Primero, creo una función para ejecutar todas mis consultas a la base de datos, recuerda la siguiente norma: una función ha de contener UNA sola función, de este modo, consigues, además, poder reutilizar el mismo código sin tener que repetir una y otra vez las mismas lineas por cada consulta
function _exectueQuery($sql, $values) {
    global $db; 
    $subsel = $db->prepare($sql); 
 
    return $subsel->execute($values);
}
 
function getParentMenus() {
    $sql = "SELECT id, link, title, level FROM menu ORDER BY title";
 
    #Ejecutamos la consulta y obtenedremos todos los menús padres
    return _exectueQuery($sql, $data);
}
 
function getChildMenus($item) {
    $sql = "SELECT id, link, title, level FROM menu WHERE id_parent = :parent ORDER BY title";
 
    #Ejecutamos la consulta y obtenedremos todos los menús hijos
    return _exectueQuery($sql, $data);
}
 
# Esta función te retorna el menú ya construido
function getMenus() 
{
    $html = '';
    # Conseguimos los menús padres
    $menus = getParentMenus();
 
    #Primero recorremos los menus padres para colocarlos dentro del menu
    $html .= createsMenu($menus);
 
    return $html;  
}
 
# Y aquí la función de recursividad, donde consultará todos los niveles habidos y por haber dentro de una matriz multidimensional o en tu caso, los menús.
function createsMenu($menus, $color = '') 
{   
    # Tendrás que pasar como parametro el color del menú para poder hacerlo dinámico
    $html = '<ul id="' . $color . '"class="treeview-' . $color . '">';
 
    foreach ($menus as $menu) {
        #NO entiendo la parte de tu código ($i['level'] -1), si el valor de level es 0, no puede ser -0. Por eso lo dejé como lo hago ahora
 
        #Creamos la primera lista pero no lo cerramos todavía
        $html .= '<li class="menucolor-' . $color . '"><a href="' . htmlspecialchars($menu['link']) . '">' . htmlspecialchars($menu['title']) . '</a>';
 
        # Comprobamos si el actual menú tiene hijos
        $childs = getChildMenus($menu['id']);
 
        # Si los tiene, los recorremos para imprimirlos dentro del menú en curso, de lo contrario, cerramos <li></li>
        $html .= count($childs) > 0? createsMenu($childs) : '</li>';
    }
 
    # Cerramos la lista si ya no hay más registros 
    $html .= '</ul>';
 
    return $html; 
}
 

Una vez que tengas toda la lógica en el controlador, cuando llames a la vista pasandole el parámetro $html tendrías que tener algo como:

Código HTML:

<!DOCTYPE html>
<html>
<head>
    <title>Tu app</title>
</head>
<body>
    <nav><?=$html ?></nav>
 
</body>
</html>
 

Como ves, no he hecho uso de ninguna lógica de php para obtener el resultado, solo le paso el parametro a la vista ya.

Espero que te sirva en un futuro, he decidido darte esta pequeña clase, ya que otros no han sabido aplicar bien las explicaciones por entregarles pseudo-codigo a algiuen que está comenzando. Así te lo entrego de una vez, lo aprendes y ya no se te va a olvidar.

OJO! No lo he probado pero debería de funcionarte, si algo te sale mal, te garantizo que el fallo te saldrá dentro de la función createsMenu(); es muy probable que me haya podido confundir en los lugares que imprimo cada sección del html.

[vb2005]

Iniciado por Triby2

Las almas estás pero quizás no en el momento cuando las necesitamos jejejeje

Tu problema es que aún no estás concibiendo el funcionamiento del patrón MVC. Desde las vistas no se llaman a las funciones, ni tienen lógica incrustada. Eso se realiza desde los controladores y desde ahí se pasan los resultados a las vistas, que por eso lo llaman "vista".

Después tienes que obtener primero todos los menús padres, recorrerlos y conseguir de cada uno los hijos para imprimirlos y ya.

Yo lo haría de este modo:

Código PHP:

#El siguiente código se debe de colocar en el controlador correspondiente dentro de la función que le corresponda que en este caso podría ser index() dentro del HomeController por poner un ejemplo de controlador.
 
# Primero, creo una función para ejecutar todas mis consultas a la base de datos, recuerda la siguiente norma: una función ha de contener UNA sola función, de este modo, consigues, además, poder reutilizar el mismo código sin tener que repetir una y otra vez las mismas lineas por cada consulta
function _exectueQuery($sql, $values) {
    global $db; 
    $subsel = $db->prepare($sql); 
 
    return $subsel->execute($values);
}
 
function getParentMenus() {
    $sql = "SELECT id, link, title, level FROM menu ORDER BY title";
 
    #Ejecutamos la consulta y obtenedremos todos los menús padres
    return _exectueQuery($sql, $data);
}
 
function getChildMenus($item) {
    $sql = "SELECT id, link, title, level FROM menu WHERE id_parent = :parent ORDER BY title";
 
    #Ejecutamos la consulta y obtenedremos todos los menús hijos
    return _exectueQuery($sql, $data);
}
 
# Esta función te retorna el menú ya construido
function getMenus() 
{
    $html = '';
    # Conseguimos los menús padres
    $menus = getParentMenus();
 
    #Primero recorremos los menus padres para colocarlos dentro del menu
    $html .= createsMenu($menus);
 
    return $html;  
}
 
# Y aquí la función de recursividad, donde consultará todos los niveles habidos y por haber dentro de una matriz multidimensional o en tu caso, los menús.
function createsMenu($menus, $color = '') 
{   
    # Tendrás que pasar como parametro el color del menú para poder hacerlo dinámico
    $html = '<ul id="' . $color . '"class="treeview-' . $color . '">';
 
    foreach ($menus as $menu) {
        #NO entiendo la parte de tu código ($i['level'] -1), si el valor de level es 0, no puede ser -0. Por eso lo dejé como lo hago ahora
 
        #Creamos la primera lista pero no lo cerramos todavía
        $html .= '<li class="menucolor-' . $color . '"><a href="' . htmlspecialchars($menu['link']) . '">' . htmlspecialchars($menu['title']) . '</a>';
 
        # Comprobamos si el actual menú tiene hijos
        $childs = getChildMenus($menu['id']);
 
        # Si los tiene, los recorremos para imprimirlos dentro del menú en curso, de lo contrario, cerramos <li></li>
        $html .= count($childs) > 0? createsMenu($childs) : '</li>';
    }
 
    # Cerramos la lista si ya no hay más registros 
    $html .= '</ul>';
 
    return $html; 
}
 

Una vez que tengas toda la lógica en el controlador, cuando llames a la vista pasandole el parámetro $html tendrías que tener algo como:

Código HTML:

<!DOCTYPE html>
<html>
<head>
    <title>Tu app</title>
</head>
<body>
    <nav><?=$html ?></nav>
 
</body>
</html>
 

Como ves, no he hecho uso de ninguna lógica de php para obtener el resultado, solo le paso el parametro a la vista ya.

Espero que te sirva en un futuro, he decidido darte esta pequeña clase, ya que otros no han sabido aplicar bien las explicaciones por entregarles pseudo-codigo a algiuen que está comenzando. Así te lo entrego de una vez, lo aprendes y ya no se te va a olvidar.

OJO! No lo he probado pero debería de funcionarte, si algo te sale mal, te garantizo que el fallo te saldrá dentro de la función createsMenu(); es muy probable que me haya podido confundir en los lugares que imprimo cada sección del html.

Es un error de concepto que tu "controlador" genere código html

No hay que confundir la lógica a hacer un loop en un array de elementos para generar una lista de elementos en html.

Es totalmente correcto hacer un for dentro de la vista para imprimir asi los elementos.

Tu código tiene todo acumulado en un sólo método que lo hace todo, ese no es el objetivo de MVC

[Triby2]

Iniciado por vb2005

Es un error de concepto que tu "controlador" genere código html

No hay que confundir la lógica a hacer un loop en un array de elementos para generar una lista de elementos en html.

Es totalmente correcto hacer un for dentro de la vista para imprimir asi los elementos.

Tu código tiene todo acumulado en un sólo método que lo hace todo, ese no es el objetivo de MVC

Yo no dije, que el controlador genere odigo html, es uan locura. Lo que dije, es LLAMAR...

"Una vez que tengas toda la lógica en el controlador, cuando llames a la vista pasandole el parámetro $html tendrías que tener algo como"

Y si, se puede ejecutar un loop como for o foreach en la vista, pero no mucho mas de ahi, pues esos loops solo deben de ser para mostrar los datos del array.

Lo que el compañero pretendia hacer, es llamar a una funcion desde la vista... y encima dentro de un <li>

[stevens82]

Hola triby2 gracias por responder, estuve probando el codigo y me surgieron algunas dudas con respecto a las consultas que se hace en el controlador, estas no deberian estar en el modelo me refiero a las consultas function getParentMenus() y getChildMenus($item), trate de ponerlas en el modelo como siempre lo hago de esta manera:

Código SQL:

public FUNCTION getParentMenus()
    {
        $menu = $this->_db->query('SELECT id, link, title, level FROM meno ORDER BY title');
        RETURN $menu->fetchAll();
    }
 

Y en el controlador llamo al modelo de esta manera

Código PHP:

  class indexController extends Controller
{
    private $_menu;

    public function __construct()
    {
        parent::__construct();
        $this->_menu = $this->loadModel('index');
    }  
   

y en una funcion traigo los datos para despues renderizarlo en la vista
de esta manera:

Código PHP:

  public funcion index(){

//asi traigo los datos del modelo
$this->_view->menu = $this->_menu->getParentMenus(); 

//asi renderizo a la vista
$this->_view->renderizar('index', 'inicio'); 
}  
   

Y en la vista mostraba los datos con un foreach.

hice modificaciones pero sin ningun resultado positivo :(

[vb2005]

Iniciado por Triby2

Yo no dije, que el controlador genere odigo html, es uan locura. Lo que dije, es LLAMAR...

"Una vez que tengas toda la lógica en el controlador, cuando llames a la vista pasandole el parámetro $html tendrías que tener algo como"

Y si, se puede ejecutar un loop como for o foreach en la vista, pero no mucho mas de ahi, pues esos loops solo deben de ser para mostrar los datos del array.

Lo que el compañero pretendia hacer, es llamar a una funcion desde la vista... y encima dentro de un <li>

El problema es que en tu código están todos los conceptos mezclados.

En el mismo lugar hay responsabilidades del modelo, controlador y vista en el mismo lugar/método.

Tu vista lo único que tiene es un "echo $html" eso muestra de que hay responsabilidades que no fueron repartidas correctamente, porque el encargado de generar el html es la vista y no el controlador

[Triby2]

Iniciado por vb2005

El problema es que en tu código están todos los conceptos mezclados.

En el mismo lugar hay responsabilidades del modelo, controlador y vista en el mismo lugar/método.

Tu vista lo único que tiene es un "echo $html" eso muestra de que hay responsabilidades que no fueron repartidas correctamente, porque el encargado de generar el html es la vista y no el controlador

Lo puse asi expresamente, si esta aprendiendo mvc, lo suyo es que aprenda diferenciar donde va cada funcion. Y mi vista es UN EJEMPLO...

[Triby2]

Iniciado por stevens82

Hola triby2 gracias por responder, estuve probando el codigo y me surgieron algunas dudas con respecto a las consultas que se hace en el controlador, estas no deberian estar en el modelo me refiero a las consultas function getParentMenus() y getChildMenus($item), trate de ponerlas en el modelo como siempre lo hago de esta manera:

Código SQL:

public FUNCTION getParentMenus()
    {
        $menu = $this->_db->query('SELECT id, link, title, level FROM meno ORDER BY title');
        RETURN $menu->fetchAll();
    }
 

Y en el controlador llamo al modelo de esta manera

Código PHP:

  class indexController extends Controller
{
    private $_menu;

    public function __construct()
    {
        parent::__construct();
        $this->_menu = $this->loadModel('index');
    }  
   

y en una funcion traigo los datos para despues renderizarlo en la vista
de esta manera:

Código PHP:

  public funcion index(){

//asi traigo los datos del modelo
$this->_view->menu = $this->_menu->getParentMenus(); 

//asi renderizo a la vista
$this->_view->renderizar('index', 'inicio'); 
}  
   

Y en la vista mostraba los datos con un foreach.

hice modificaciones pero sin ningun resultado positivo :(

Bueno, el codigo ya lo tienes, y resuelve tu problema de recursividad, lo que hay que trabajar ahora es el resultado.

Es normal que el resultado no sea el esperado pues el codigo no lo probé. ¿podrias compartir el resultado de mi codigo?

En cuanto tus dudas, en el modelo solo se deberia de guardar los metodos para añadir, actualizar, leer y borrar registros de la tabla.

En el controlador, van todas las funciones que te compartí, y ademas las funciones basicas que llamarian a los metodos de añadir, actualizar, leer y borrar pasanxoles a vada uno de ellos los datos a tratar. Es decir,en el contro,ador, crearias el array de datos que necesitas manejar, y pasarias el array al meteodo que necesites para que el modelo lo trate, una vez eso, llamarias a la vista.

Jamas llames a la vista desde el metodo constructor del controlador, pues no es lo correcto, lo suyo es que tengas un metodo index que SI llamaria a la vista para mostrar los datos, despues tendrias un metodo store(), donde recibiria los datos de un formulario, por poner un ejemplo, crearia el array como tu lo necesites, y lo enviarias al modelo para guardarlo, despues llamarias a la vista que podria mostrar un mensaje confirmando que se guardo, por poner un ejemplo.

Tendrias otro metodo llamado edit() que obtendria los datos del registro para mostrarlo en el formulario para que el usuario lo pueda editar y al clicar guardar,llamarias al metodo update() del controlador, que montaria el array con los datos modificados para ser enviado el metodo de actualizar en el modelo, despues llamaria a una vista que podria contener el menaje de que se actualizo con exito.

¿me explico?