Validación valores seteados en una clase

[miktrv]

Hola buenas,

Quisiera saber cual es la mejor manera para validar los datos que se le pasan a una clase... yo para cada "modulo" de mi aplicación defino sus clases y sus modelos, pero no se si la validación de los valores que se le pasan a estos, debe de estar dentro, fuera, o dentro y fuera de la clase.. por ejemplo:

$miclase = new Coche();

$coche->setPuertas(5);

el método setPuertas, tendría que validar que se le pasa un número entero? o yo tengo que hacer una validación antes de que le llegue el dato por otra parte?

Un saludo y gracias!!

[abimaelrc]

Podrías hacerlo dentro del método de setPuertas.

Código PHP:

<?php
class FooBar
{
    private $_foo;
 
    public function setFoo($foo)
    {
        $this->_foo = (int)$foo;
    }
}
 

Algunos indican que los setter y getters solo deberían ser mostrar o guardar datos y que con otros métodos sea que hagas las validaciones. Es decir

Código PHP:

<?php
class FooBar
{
    private $_foo;
 
    public function setFoo($foo)
    {
        $this->_foo = $foo;
    }
 
    public function bar()
    {
        $bar = (int)$this->_foo;
        return $bar;
    }
}
 

Una de las muchas razones que posiblemente quieras guardar en esa variable, diferentes tipos de datos y en los métodos que quieras usar puedes especificar. Pero eso es a discreción tuya.

[masterpuppet]

Sigue este thread http://www.forosdelweb.com/f68/duda-...to-mvc-893946/

Saludos.

[miktrv]

Muy buena la información.. pero no me queda del todo claro... jajaja

Yo tengo por una parte, los modelos, por otra, las clases con los objetos de mi aplicación, (usuarios, etc), y por otra los controladores. Mi idea, es que, al hacer un set en el objeto, se valide ( pero sin tener en cuenta el tipo de error ) el contenido que recibe, por ejemplo:

Código PHP:

  class Coche {
 
 private $__puertas;
 
 public function setPuertas($puertas) {
  if(is_numeric($puertas)) {
     $this->__puertas = $puertas;
   }
 }
 
} 
   

Es decir, al menos el tipo de dato, con filter_var o lo que sea.

Y luego, en mis modelos, controlar de una manera más detallada todo lo demás..

Un saludo!

[abimaelrc]

Lo recomendable es ingresar el valor en el set no importando que tipo de datos sea y luego las validaciones en otro método.

[miktrv]

Hola buenas de nuevo,

Entonces, tengo que crear un método asociado a cada setter que valide el dato? ... pero, entonces cuando setee un dato, no llamaré al set, si no al nuevo método? que lío...

Un saludo!

[abimaelrc]

Pasa por el enlace que te indicó master para que veas lo que puedes hacer.

[miktrv]

De acuerdo, pero algún ejemplo básico?

Un saludo!

[GatorV]

De hecho a mi parecer es el mismo método setter el que debe de validar que tipo de dato, por ejemplo en Zend Framework cuando llaman al set, hace la validación por ejemplo:

Código PHP:

public function setRequest($request)
    {
        if (is_string($request)) {
            if (!class_exists($request)) {
                require_once 'Zend/Loader.php';
                Zend_Loader::loadClass($request);
            }
            $request = new $request();
        }
        if (!$request instanceof Zend_Controller_Request_Abstract) {
            require_once 'Zend/Controller/Exception.php';
            throw new Zend_Controller_Exception('Invalid request class');
        }
 
        $this->_request = $request;
 
        return $this;
    }
 

La idea es que el setter mismo valide que tipo de datos tenga para que la clase sea consistente.

Saludos.

[abimaelrc]

Iniciado por GatorV

De hecho a mi parecer es el mismo método setter el que debe de validar que tipo de dato, por ejemplo en Zend Framework cuando llaman al set, hace la validación por ejemplo:

Código PHP:

public function setRequest($request)
    {
        if (is_string($request)) {
            if (!class_exists($request)) {
                require_once 'Zend/Loader.php';
                Zend_Loader::loadClass($request);
            }
            $request = new $request();
        }
        if (!$request instanceof Zend_Controller_Request_Abstract) {
            require_once 'Zend/Controller/Exception.php';
            throw new Zend_Controller_Exception('Invalid request class');
        }
 
        $this->_request = $request;
 
        return $this;
    }
 

La idea es que el setter mismo valide que tipo de datos tenga para que la clase sea consistente.

Saludos.

Interesante, muchas de las lectuas lo que indican es que los setters y getters solo deben ser para el propósito de asignar y leer, por cuestiones de encapsulamiento a las propiedades. Las validaciones se hacen por otro lado. Siempre he tenido la duda si hacerlo en el setter por cuestión de eliminar procesos adicionales. Pero ya veo que hay diferencias en el estilo de programar.

[GatorV]

Nunca había visto eso que tú comentas, de hecho casí todos los programas que he visto (inclusive en otros lenguajes) el setter mismo valida el tipo de dato que va a recibir.

[_ssx]

Iniciado por abimaelrc

Interesante, muchas de las lectuas lo que indican es que los setters y getters solo deben ser para el propósito de asignar y leer, por cuestiones de encapsulamiento a las propiedades. .

No tendría entonces sentido tener setters y getter ya que en ese caso pues mejor variables publicas ya que el setter y el getter no harían NADA, el setter y el getter son metodos especiales para NO acceder directamente a la propiedad SINO poder darle algo de logica a la setteo y al getteo.

Vamos, pensemoslo si vamos a "hacer un setter" debe de ser como un "Cadenero" en la entrada de un antro el decide si entran vestidos con tennis o NO.. Sería un poco "ilogico" dejarlos entrar y después adentro decidir cuales vienen vestidos con tennis y cuales NO.

[abimaelrc]

Ok, pues corregido en mi mente el que se puede hacer con los setter. Pero no sé si mal-interprete por ejemplo: http://phpsenior.blogspot.com/2006/1...accesores.html

[GatorV]

Como dice el link, hay diferentes niveles, no le puedes agregar toda la lógica al setter, aunque es posible validar el tipo de dato, ahora sí que es a requerimiento, si tu objeto realmente por ejemplo no puede tener el mismo nombre que ninguno, y es elemental, si agrega el set del nombre ahí, todo depende del requerimiento del objeto, pero no tiene ningún problema que el mismo objeto valide que tipo de dato necesita.

El ejemplo de _ssx es muy bueno en este punto, para que dejas entrar datos que no necesitas a tu objeto si desde el set puedes revisar que sea un tipo de dato correcto (que sea usable o no lo puedes dejar para una validacion posterior).

Por ejemplo continuando lo que dice _ssx, puedes dejar pasar al antro a quien no tiene tenis, pero ya dentro del antro verificas que tenga dinero, ya que solo te interesa usuarios que no tengan tenis, lo demas ya lo dejas a la implementacion posterior a la hora de pagar().

Saludos.

[abimaelrc]

Gracias por el dato, ahora si que estoy plenamente convencido

[miktrv]

Perfecto.. ahora yo también lo veo más claro... mi lógica decía que tenía que implementar una cierta lógica.. ( valga la redundancia ) , por ejemplo, no me parecia bien que el setter de un numero pudiera comerse una string, y por eso decidí que, aunque no se hiciera así, debía de implementar al menos un nivel básico de validación para que almenos el set tubiera sentido, después de comprobar tras vuestros comentarios que es bueno hacer eso, me alegro mucho de ir por el buen caminoo0o!! jaja MUCHAS GRACIAS!

Un saludo!

[carlos_belisario]

Interesante, tenia la misma idea que platea abimaelrc, de echo en algunos ejemplos (veamos si consigo los link) hacen el setter y luego hacen la validacion, no habia caido en que se puede hacer una validacion directa en este mismo , bueno todos los dias se aprende algo, con respecto al tema que te indico el amigo masterpuppet es una clase super sencilla para hacer validaciones, en mi caso y no se si esta bien implementado las hago directamente en los modelos al momento de aplicar la lógica de la aplicacion, pero viendo bien lo que indican aca creo que tengo que mejorar un poco los setter que tengo en mis clases , saludos