Está bien el ejemplo de MVC?

jaume5900 · #1 (**permalink**) 14/04/2015, 10:34

Hasta donde se esto podría ser MVC:

Controller:

Código:

<?php

require __DIR__ . '/../classes/class.routing.php';

$controll = new Routing;

$controll->get_model($_GET['page']);

$model = new IndexModel();
$model->get_page($_GET['page']);

$controll->get_view($_GET['page']);
?>

Model:

Código:

<?php
/**
 * Created by PhpStorm.
 * User: root
 * Date: 14/04/15
 * Time: 15:39
 */

class IndexModel{
    public function get_page($get){
        if($get == null){
            echo 'ERROR';
        }else{
            echo 'OK:    ' . $get;
        }
    }
}

View:

Código:

<!DOCTYPE html>
<html>
<head>
</head>
<body>
Hellow MVC!
</body>
</html>

hhs · #2 (**permalink**) 14/04/2015, 11:20

Si es un aproximación, pero cual es la duda o problema con esto ?

NSD · #3 (**permalink**) 14/04/2015, 11:23

El controlador esta mal, debería ser el mismo routeador, y por supuesto, no debería relacionarse con la vista en ningún momento, menos aun llamarla.
Eventualmente hasta se podría prescindir de un controlador, si lograras definir una buena estructura para tus urls.

El modelo parece correcto. Aunque el escenario que planteas es extremadamente básico.

La vista esta bien porque no contiene código php.

jaume5900 · #4 (**permalink**) 14/04/2015, 11:43

NSD que está mal? Es que soy novato en el tema...

Código:

class Routing{

    public function get_model($get){
        if(file_exists(__DIR__ . '/../models/' . $get . '.model.php')){
            require __DIR__ . '/../models/' . $get . '.model.php';
        }else{
            exit('404 ERROR');
        }
    }
    public function get_view($get){
        if(file_exists(__DIR__ . '/../views/' . $get . '.php')){
            require __DIR__ . '/../views/' . $get . '.php';
        }else{
            exit('404 ERROR');
        }

    }

    public function get_class($class){
        if(file_exists(__DIR__ . '/../classes/class.' . $class . '.php')){
            require __DIR__ . '/../classes/class.' . $class . '.php';
        }else{
            echo 'The class is not existing';
        }
    }

}

Aquí está lo que tengo en routing... Me podéis ayudar con los fallos que tenga?

Si, es básico pero yo estoy empezando a usar este modelo, mejor comenzar con lo basico XD

NSD · #5 (**permalink**) 14/04/2015, 19:16

MVC no es el mejor patrón de diseño para la web, al menos no tal como se aplica en otros ambientes.

En un sistema web moderno deberías plantearte una estructura siguiendo las siguientes consideraciones:

- Cada url corresponde o bien a un método publico de una clase o bien a un recurso estatico.
- El ruteador debe determinar si la url solicitada corresponde a una cosa o a la otra y actuar en consecuencia.
- El modelo son clases simples que nada conocen aparte de como hacer su función especifica.

Una forma basica pero que puede servir como punto de partida podria ser tener 2 carpetas, una llamada "view" y otra llamada "api" o "model" o como se te antoje, "api" puede ser un nombre mas acertado que "model" aunque terminaran siendo lo mismo.

Dentro de "view" van todos los recursos estaticos, eston son archivos html, css, js, imagenes, etc. Dentro de esta carpeta ESTA PROHIBIDO poner una sola linea de php en algún archivo.

Dentro de la carpeta "api" van las clases y el routeador.
Una medida de seguridad básica es que a la api solo se accede por POST mientras que a la vista solo se accede por GET, eso ya te da una pauta de como separar una cosa de la otra.

Te pongo el tipico "Hola mundo" siguiendo esta estructura que seria el equivalente a lo que tu planteaste.

/.htaccess

Código Apache:

Ver originalOptions +FollowSymLinks
RewriteEngine on
 
RewriteCond %{REQUEST_METHOD} =POST
RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
RewriteRule ^([a-zA-Z0-9_-]+)/([a-zA-Z0-9_-]+)$ route.php?class=$1&method=$2 [PT,L]
 
RewriteCond %{REQUEST_METHOD} =GET
RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
RewriteRule ^(.*)$ view/$1 [PT,L]

/route.php

Código PHP:

Ver original<?php
    list($class, $method) = [$_GET["class"], $_GET["method"]];
    $post_params = $_POST;
    unset($_POST, $_GET);
    
    require("api/$class.php");
    
    $class = ucfirst($class);
    $instance = new $class();
    echo json_encode($instance->$method($post_params));

/api/topic.php

Código PHP:

Ver original<?php 
    class Topic {        
        public function usuarios($parametros)
        {
            return ["jaume5900", "hhs", "NSD"];
        }        
    }

/view/index.html

Código HTML:

Ver original<!DOCTYPE html>
<html>
    <head>
        <title>Hola mundo</title>
    </head>
    <body>
        Usuarios que participaron en este post:
        <ul></ul>
        <script>
            var api = new XMLHttpRequest();
            api.onreadystatechange = function() {
                if(this.readyState == 4 && this.status == 200) 
                {
                    console.log(this.responseText);
                    var data = JSON.parse(this.responseText), ul = document.getElementsByTagName("ul")[0];
                    for(var nro=0; nro<data.length; nro++)
                        ul.innerHTML += "<li>"+data[nro]+"</li>";
                }           
            }.bind(api);        
            api.open('POST', 'topic/usuarios', true);
            api.send();     
        </script>
    </body>
</html>

Una pregunta razonable seria ¿porque no mezclar php y html? si a priori eso es mas facil, la respuesta es porque estamos en 2015 y ese tipo de técnicas quedaron obsoletas, son de otra época, cuando php tenia que salir al rescate de las limitaciones de javascript y html era mas limitado, hoy en día me atrevería a decir que es exactamente al revés.

Por supuesto que es solo mi punto de vista, no dudo que otros pensaran diferente. Tu intenta sacar tus propias conclusiones.