MVC PHP Estructurado, ¿asi esta bien?

[anacona16]

Cuando empeze a programar con PHP me aburria tener que hacer los mismo una y otra vez (claro no sabia mucho) luego con esto de MVC y algo de POO me di cuenta que se puede reutilizar mucho.

Siguiendo este tutorial http://www.jourmoly.com.ar/introducc...primera-parte/ aprendi MVC y vaya me di cuenta que todo se vuelve mas facil, ahora, segun yo esto es MVC, pero quiero que ustedes me lo confirmen.

Esta es la estructura del sitio, en un proyecto netbeans (No es el sitio completo solo el ejemplo)



Asi como en el ejemplo utilizan un archivo index.php que dirige segun el controlador y la accion cargado yo utilizo admin/productos.php (creo debo cambiar el nombre) en modelos tengo:

Código PHP:

<?php
    global $servidor, $bd, $usuario, $contrasenia;
    $db = new PDO('mysql:host=' . $servidor . ';dbname=' . $bd, $usuario, $contrasenia);
 
    function buscarTodosLosItems($db)
    {
        $datos = array();
        $sqlDefecto = "SELECT * FROM tb_productos ORDER BY codiprod DESC";
        $consultaProductos = $db->query($sqlDefecto);
        $cantRegistros  = $consultaProductos->rowCount();
 
        $pgnActual = isset($_GET['pgn'])? $_GET['pgn'] : 0;
 
        $pag = new Paginador();
 
        $datos = $pag->paginar($pgnActual, $cantRegistros);
        $inicio = $pgnActual * 10;
 
        $consultaProductos = $db->query("SELECT * FROM tb_productos ORDER BY codiprod DESC LIMIT $inicio, 40");
        return $datos[] = array("consultaProductos" => $consultaProductos->fetchAll(),
                                "cantidadRegistros" => $cantRegistros, "datos" => $datos, "pag" => $pag->getCantidadPaginas());        
    }
 
    function insertar($db)
    {
        $insertar = $db->prepare("INSERT INTO tb_productos VALUES (0,:referencia,:descripcion,:cantidad,:valor,'No disponible')");
        $insertar->bindParam(":referencia", strtoupper($_POST['referencia']), PDO::PARAM_STR);
        $insertar->bindParam(":descripcion", strtoupper($_POST['descripcion']), PDO::PARAM_STR);
        $insertar->bindParam(":cantidad", $_POST['cantidad'], PDO::PARAM_INT);
        $insertar->bindParam(":valor", $_POST['valor'], PDO::PARAM_INT);
        $insertar->execute();
 
        if($insertar->rowCount() > 0){
            alert('Guardado con exito!');
            redirect('productos.php');
        }else{
            alert('Error al guardar, intente de nuevo!');
            redirect('productos.php?controlador=items&accion=agregar');
        }
    }
 
    function update($db,$codProducto)
    {
        $updateProducto = $db->prepare("UPDATE tb_productos SET refeprod = ?, descprod = ?, cantprod = ?, valoprod = ? WHERE codiprod = ?");
        $updateProducto->bindParam(1, strtoupper($_POST['referencia']), PDO::PARAM_STR);
        $updateProducto->bindParam(2, strtoupper($_POST['descripcion']), PDO::PARAM_STR);
        $updateProducto->bindParam(3, $_POST['valor'], PDO::PARAM_INT);
        $updateProducto->bindParam(4, $_POST['cantidad'], PDO::PARAM_INT);
        $updateProducto->bindParam(5, $codProducto, PDO::PARAM_INT);
        $updateProducto->execute();
 
        if($updateProducto->rowCount() > 0){
 
            alert('Registro actualizado!');
            redirect('productos.php');
 
        }else{
            alert('Error al actualizar, intente de nuevo!');
            redirect('productos.php');
        }
    }
 
    function eliminar($db,$codProducto)
    {
        $deleteProducto = $db->exec("DELETE FROM tb_productos WHERE codiprod = $codProducto");
 
        if($deleteProducto > 0){
            
            alert('Registro eliminado!');
            redirect('productos.php');
 
        }else{
            alert('Se ha producido un error, intente de nuevo');
            redirect('productos.php');
        }
    }
 

en el controlador tengo:

Código PHP:

<?php
    function listar()
    {
        //Incluye el modelo que corresponde
        require '../modelos/productosModelo.php';
 
        //Le pide al modelo todos los items
        $productos = buscarTodosLosItems($db);
 
        //Pasa a la vista toda la información que se desea representar
        require '../vistas/listar.php';
    }
 
    function agregar()
    {
        require '../modelos/productosModelo.php';
 
        if($_POST){insertar($db);}
 
        require '../vistas/agregar.php';
    }
 
    function guardarEditar()
    {
        require '../modelos/productosModelo.php';
        $codProducto = $_GET['codigo'];
        if (isset($codProducto) && is_numeric($codProducto) && !empty($codProducto)) {
            update($db, $codProducto);
        }else{
            alert('mal');
            redirect('../admin/productos.php');
        }
    }
 
    function borrar()
    {
        require 'modelos/productosModelo.php';
 
        $codProducto = $_GET['codigo'];
        if (isset($codProducto) && is_numeric($codProducto)) {
            eliminar($db,$codProducto);
        }else{
            alert('mal');
            redirect('productos.php');
        }
    }
 

y en las vistas pues es solo HTML, el unico PHP es para listar que tengo esto:

Código PHP:

<?php foreach($productos["consultaProductos"] as $producto): ?>
                <tr align="center">
                    <td><?php echo $producto['codiprod'] ?></td>
                    <td><?php echo $producto['refeprod'] ?></td>
                    <td align="left"><?php echo $producto['descprod'] ?></td>
                    <td><?php echo $producto['cantprod'] ?></td>
                    <td align="right"><?php echo '$ '. number_format($producto['valoprod']) ?></td>
                    <td><img src="../img/marquilla.png" width="16" height="16" alt="Generar marquilla" /></td>
                    <td>
</tr>
<?php endforeach; ?>
 

Para cargar la vista de agregar utilizo una URL como esta:

Código PHP:

productos.php?controlador=productos&accion=agregar
 

Para eliminar asi:

Código PHP:

productos.php?controlador=productos&accion=borrar&codigo=<?php echo $producto[0] ?>
 

y bueno esto es lo mas notorio.

¿Asi esta bien?

Gracias por sus sugerencias y comentarios

[pateketrueke]

yo lo veo bien...

y recuerda que hay cientos de formas para implementar MVC, unas mas limpias que otras, pero al final todo depende de lo que necesitamos... si por ti esta bien, ¡que mejor!

[anacona16]

Gracias, si eres tu que me dice que esta bien, entonces esta muy bien.

Supongo que hay muchas formas como lo dices, de la manera como lo estoy haciendo tal vez no se la mejor, pero esta muchisimo mejor a como lo hacia normalmente, todo en un solo archivo, PHP-Consultas-HTML, etc.

[carlos_belisario]

Y porque no trabajas con clases??? en esa misma pagina donde seguiste el tutorial tienen una segunda parte donde lo implementan con clases la implementacion del MVC, es un buen comienzo pero el aprendizaje no termina en un solo tutorial, te invito a que te metas con la OOP y veas todas las ventajas que este nos da, saludos

[anacona16]

Gracias, tambien mire la segunda parte y nada he querido mas que hacer puro OOP pero la verdad todavia del todo no logro comprenderla, en mi sitio (que muy pronto publicare para que os deis una opinion) utilizo algunas clases que son de mucha ventaja y reutilizables (no las hice yo las hizo http://objetivophp.com), pero estoy aprendiendo para hacer mis propias clases.