PHP OO duda de concepto en MVC

[carlos_belisario]

Buenas estoy desarrollando un pequeño sistema por lo que no trabaje con ningun framework sino que tengo digamos un mini marco de trabajo con mis clases y un poco de MVC, pero tengo una duda grande, segun lo que yo entendia porlo que veia al momento de trabajar con cakePHP la logica aca practicamente esta en el controlador y el modelo se encarga unicamente de lo que es el trabajo con los datos (esto segun lo que yo veo si estoy equivocado regañenme y corrijan lo que estoy diciendo), ya que en el modelo solo se declara las relaciones con las base de datos y un poco de validacion como dije segun veo en lo poco que me he podido adentrar en este fw, entonces releyendo porque quiero hacer las cosas bien y estar claro en lo que a MVC se refiere hice mi minimarco pero me queda la duda, debo hacer la logica en el controlador y el modelo, por ejemplo suponiendo que tengo un controlador y un modelo para el usuario
UsuarioController

Código PHP:

public function login()
    {
        $this->model = new UsuarioModel();
        $user = $this->model->find('usuario',array('usuario'=>$_POST['usuario']));
        if(count($user) == 0){
            $_SESSION['errorUsuario'] = "no_esta";
            header("location:".$_SERVER['PHP_SELF']);
            exit;
        }
 

llamando al model y trabajando el model como lo hace cake incluido en el controlador?? o tengo que hacer esta logica en el modelo y solo hacer el llamado desde el controlador?? creo que me volvi a perder entre tanta lectura porque algunos autores hablan que la logica esta en el controlador y otros en el modelo, a quien creerle?? saludos y gracias por las respuestas que me puedan brindar

[GatorV]

La logica siempre debe de tratarse en el modelo, y esto es porque más de 1 controlador puede usar tu modelo, si la logica la incluyes en el modelo, cuando quieras hacer algo diferente tendrías que copiar el contenido de tu controller.

Es por eso que siempre se recomienda que la relación sea de modelo (capa de negocios) al controlador, y que uno o más controladores puedan leer este modelo y siempre actuen igual.

Te recomiendo leer este pequeño articulo que si bien no es PHPOO (es Ruby) habla muy bien de porque la logica debe de pertenecer al modelo: http://weblog.jamisbuck.org/2006/10/...ller-fat-model

Saludos.

[carlos_belisario]

gracias por la explicacion y el artículo cre que me quedo un poco mas claro, aunque me costo un poco porque mi ingles no es muy bueno (casi nulo ) pero con el traductor de google me ayude creo que aunque estava en ruby que nunca me he preocupado por manejar creo que la explicacion es muy generica para cualquier lenguaje, entonces todo lo que es la logica de comparacion y todo de mi login deberia de estar es en el modelo y solo llamar a un metodo de busqueda de usuarios y segun sea la respuesta de este ir a la vista correspondiente?? gracias por tu ayuda creo que ahora si estoy un poco mas claro, pero entonces me queda la duda hay mucho fw que trabajan la logica en el controlador por lo que no estarian aplicando correctamente el MVC aunque funcionen de manera correcta lo que se haga en ellos o estoy equivocado???

[carlos_belisario]

se me acaba de generar una duda, la validación en este patrón entonces en donde la haria directamente en el modelo donde esta toda la logica cierto, entonces las acciones del controlador serian unicamente para pasar los datos de la vista al modelo y visceversa??

habia editado pero asi nadien se va a dar cuenta de la ultima duda del tema

[NUCKLEAR]

Siguiendo estrictamente el patron la logica diria que si. Es lo que pienso.

Personalmente me gusta mas la idea de tener una carpeta con funciones en comun validadores, generadores de url's, etc. (Y lo que normalmente se hace)

Y simplemente no terminar cargando un modelo con algo que se voy a necesitar en toda la aplicacion.

A menos claro esta de que el validador solo se requiera en un caso en particular.

[carlos_belisario]

Iniciado por NUCKLEAR

Siguiendo estrictamente el patron la logica diria que si. Es lo que pienso.

Personalmente me gusta mas la idea de tener una carpeta con funciones en comun validadores, generadores de url's, etc. (Y lo que normalmente se hace)

Y simplemente no terminar cargando un modelo con algo que se voy a necesitar en toda la aplicacion.

A menos claro esta de que el validador solo se requiera en un caso en particular.

gracias por tu respuesta amigo me quitaste la duda que tenia , bueno yo tengo algunas funciones donde valido por tipo de dato y por campo vacio lo que tenia pensado es incluir estas en el modelo o controlador segun me terminaran de guiar ustedes y unicamente llamar a esta dependiendo del caso, saludos y gracias nuevamente

[historiasdemaria]

Un ejemplo muy claro de esto es la validacion de un email, entonces para no escribir cada vez la misma funcion para las clases que traten con un email, normalmente yo escribo una clase tipo Tools, como MiproyectoTools, luego se llama a sus metodos donde se necesite una validacion y ya esta

[carlos_belisario]

gracias por la respuesta amigo, eso mismo estava pensando implementar una clase donde tenga mis validaciones y solo llamarlas y llamar a sus metodos dependiendo al caso, gracias

[masterpuppet]

Que tal carlos_belisario,

Algo parecido se comento hace un tiempo, te dejo el link, http://www.forosdelweb.com/f68/valid...-model-876569/.
Creo que deberías refactorizar tus funciones de validación, a clases y luego dependiendo de la aplicación decides donde validar, a nivel de modelos, formulario, service layer o en todos :).
Te podes guiar por como lo manejan los diferentes frameworks, en Yii por ejemplo, podes definir un set de reglas en el modelo contra las que se validara posteriormente, te dejo la referencia para que te hagas una idea, en Zend es normal hacer la validación en el formulario, otro approach interesante, con el que he estado probando ultimamente es utilizar annotations + reflection, es algo interesante pero todavía no me convence del todo y seguramente hay mas, pero bueno espero te de una idea general de lo que se puede hacer.

Saludos.

[carlos_belisario]

excelente vi por encima la referencia que me dejaste de yii y me parece que maneje algo parecido con cake, entonces tendria que crear una clase para estas validaciones e incluirla en los modelos o me equivoco?? de echo creo que cuando lo trabajaba en cake podia definir mi propia regla con una exprecion regular que es lo que normalmente segun he leido se establece para ello, muchas gracias por las referencias voy a tomar un poco de las lecturas que me dejaste y a trabajar en ello

[carlos_belisario]

un poco fastidiado el dia de hoy me puse a trabajr un poco en la clase que me indicaban, las validaciones las hace segun lo que he colocado, pueden hacerme todas las criticas posibles

Código PHP:

<?php
/**
clase para realizar validaciones en el modelo
*/
class Validacion
{
    public $mensaje;
/**
declaramos la regla de validacion
*/
    public function rules($rule = array(),$data)
    {
        
        if(!is_array($rule)){
            $this->mensaje = "las reglas deben de estar en formato de arreglo";
            return $this;
        }       
        foreach($rule as $key => $rules){
            $reglas = explode(',',$rules['regla']);
            if(array_key_exists($rules['name'],$data)){
                foreach($data as $indice => $valor){                    
                    if($indice === $rules['name']){
                        foreach($reglas as $clave => $valores){                                         
                            if($valores === "no-empty"){                                
                                $respuesta = $this->_noEmpty($rules['name'],$valor);
                            }
                            if($valores === "numeric"){                             
                                $respuesta = $this->_numeric($rules['name'],$valor);
                            }
                            elseif($valores === "email"){
                                $respuesta = $this->_email($rules['name'],$valor);
                            }
                        }
                        break;
                    }
                }
            }
            else{
                $this->mensaje[$rules['name']] = "el campo $value no esta dentro de la regla de validación o en el formulario";                
            }
        }       
        if(!$respuesta){
            return $this;
        }
    }   
    /**
    metodo para verificar si esta vacio el campo
    */
    protected function _noEmpty($campo,$valor)
    {       
        if(isset($valor) && !empty($valor)){            
            return true;            
        }
        else{           
            $this->mensaje[$campo][] = "el campo $campo debe de estar lleno";
            return false;
        }
    }
    /**
    metodo para verificar que el campo sea numerico 
    */
    protected function _numeric($campo,$valor)
    {       
        if(is_numeric($valor)){
            return true;
        }       
        else{
            $this->mensaje[$campo][] = "el campo $campo debe de ser numerico";
            return false;
        }
    }
    /**
    metodo para verificacion de email
    */
    protected function _email($campo,$valor)
    {
        if(preg_match("/^[a-z]+([\.]?[a-z0-9_-]+)*@[a-z]+([\.-]+[a-z0-9]+)*\.[a-z]{2,}$/",$valor)){     
            $this->mensaje[$campo][] = "el campo $campo de estar en el formato de email [email protected]";
            return false;
        }       
    }
}
/*pruebas basicas a la clase sin incluirla en el modelo*/
$_POST['campo1'] = 1;
$_POST['campo2'] = "[email protected]";
$datos = $_POST;
$validacion =  new Validacion();
$regla = array(
            array('name'=>'campo1','regla'=>'no-empty,numeric'),
            array('name'=>'campo2','regla'=>'no-empty,email')
        );
$validaciones = $validacion->rules($regla,$datos);
print_r($validaciones);
?>
 

ahora bien, esta clase la incluyo en la clase modelo base de tal manera que pueda realizar la validaciones directamente llamando a el metodo rules de esta clase o que me pueden aconcejar, gracias de nuevo

[masterpuppet]

Que tal carlos_belisario,

Creo que los validators deberian ser clases separadas(con la interfaz correspondiente), podes ver el componente de Zend, Zend_Validator para que te hagas una idea, o en su defecto deberias tener un inflector(bien podria ser una clase tambien) en Validación para resolver el nombre de las funciones, porque cada vez que quieras agregar un nuevo validator vas a tener que agregar un if, algo bastante incomodo.

Saludos.

[carlos_belisario]

Iniciado por masterpuppet

Que tal carlos_belisario,

Creo que los validators deberian ser clases separadas(con la interfaz correspondiente), podes ver el componente de Zend, Zend_Validator para que te hagas una idea, o en su defecto deberias tener un inflector(bien podria ser una clase tambien) en Validación para resolver el nombre de las funciones, porque cada vez que quieras agregar un nuevo validator vas a tener que agregar un if, algo bastante incomodo.

Saludos.

gracias por tus apreciaciones sobre como debo mejorar lo que hice , voy a estudiar un poco lo que me indicaste y me coloco a hacer la clase creo que voy a tomar un poco el ejemplo de Zend_Validator que me hablaste, voy a revisar las librerias de Zend para guiarme un poco, además voy a investigar un poco mas el tema del inflector, ya que según lo que pude leer un poco en mi investigación del termino, mayormente resultados de ruby el inflector lo usan para la transformación de palabras para las conexiones con las db, ahora mi duda aunque tonta e ignorante pero duda al fin en que me ayudaria hacer el inflector?? ojo voy a seguir investigando el tema porque me parecio muy interesante gracias por la ayuda y la enseñanza que estas dando amigo masterpuppet , saludos

[masterpuppet]

El inflector lo que haria seria transformar los string del array rules, tal cual lo estas planteando las funciones para validar empiezan por undescore y utilizan lowercamelcase y en el array de rules vienen con el formato no-empty, numeric, la idea seria transformar el no-empty en _noEmpty, numeric en _numeric, esto:

Código PHP:

if($valores === "no-empty"){                                
    $respuesta = $this->_noEmpty($rules['name'],$valor);
}
if($valores === "numeric"){                             
    $respuesta = $this->_numeric($rules['name'],$valor);
}
elseif($valores === "email"){
    $respuesta = $this->_email($rules['name'],$valor);
}
 

podria ser algo asi:

Código PHP:

$validator = $this->_getInflectedName($valores);
if(!is_callable(array($this, $validator))){
   throw new BadMethodCallException(/*...*/);
}
$respuesta = $this->$validator($rules['name'], $valor);
 

La ventaja es que al crear nuevos validators, cosa que seguro ocurrira, no tendras que estar agregando if's.

Agregado:

otra sugerencia carlos, el nombre de las clases, metodos y variables deberia ser homogéneo, elegi español o ingles y apegate a ello.

Saludos.

[carlos_belisario]

Iniciado por masterpuppet

El inflector lo que haria seria transformar los string del array rules, tal cual lo estas planteando las funciones para validar empiezan por undescore y utilizan lowercamelcase y en el array de rules vienen con el formato no-empty, numeric, la idea seria transformar el no-empty en _noEmpty, numeric en _numeric, esto:

Código PHP:

if($valores === "no-empty"){                                
    $respuesta = $this->_noEmpty($rules['name'],$valor);
}
if($valores === "numeric"){                             
    $respuesta = $this->_numeric($rules['name'],$valor);
}
elseif($valores === "email"){
    $respuesta = $this->_email($rules['name'],$valor);
}
 

podria ser algo asi:

Código PHP:

$validator = $this->_getInflectedName($valores);
if(!is_callable(array($this, $validator))){
   throw new BadMethodCallException(/*...*/);
}
$respuesta = $this->$validator($rules['name'], $valor);
 

La ventaja es que al crear nuevos validators, cosa que seguro ocurrira, no tendras que estar agregando if's.

Saludos.

Magistral!!!! tu explicacion sobre como me ayudaria el inflactor en este caso y me ahorro todos los if que cree, entonces me sugieres que el inflactor sea una clase o seria mejor un metodo mas?? gracias por tu ayuda amigo masterpuppet

[masterpuppet]

En una primera instancia un metodo en la clase deberia ser suficiente.

Saludos.

[carlos_belisario]

bueno segui el concejo del amigo masterpuppet e hice el metodo inflector para la clase aca va no se si este muy ortodoxo pero funciona,

Código PHP:

private function _getInflectedName($text)
    {
        $_validator = preg_replace('/[^A-Za-z0-9]+/',' ',$text);
        $arrayValidator = explode(' ',$_validator);             
        if(count($arrayValidator) > 1){
            foreach($arrayValidator as $key => $value){                 
                if($key == 0){
                    $validator .= "_".$value;   
                }
                else{                   
                    $validator .= ucwords($value);
                }
            }
        }
        else{
            $validator = "_".$_validator;
        }               
        return $validator;
    }
 

gracias al amigo masterpuppet por la paciencia y la enseñanza en esto que no conocia y a los demás amigos que me orientaron tambien en el tema del MVC, gracias saludos


edito aca dejo un link de un inflector de zend, despues que me puse a hacer el metodo de mi clase , pero puede servir, aunque hay que hacerle algunas pruebas ya que en la prueba que hice el metodo camelize no funciono correctamente creo que por el resultado del preg_replace

preg_replace('/[^A-Z^a-z^0-9]+/'

saludos y gracias por la ayuda nuevamente

[IvanAsturS]

Me ha parecido muy interesante todo lo comentado en el post. Este es un tema en el que me estoy "actualizando" y he aprendido cosas. Muchas gracias a todos!

Un saludo