[SOLUCIONADO] Excepciones apropiadas

Karmac · #1 (**permalink**) 10/07/2013, 15:51

Buenas noches.

Esto es tan solo una pregunta de buenas practicas, tengo el siguiente codigo:

Código PHP:

 
<!DOCTYPE HTML> 
<html> 
<head> 
    <meta charset="UTF8"> 
</head> 
<body> 
 
<?php 
 
define( 'CONTROLLER_PATH', 'controllers/' ); 
define( 'EXT', '.php' ); 
 
 
// Dividimos la URL. 
$requestURI = explode( '/', $_SERVER['REQUEST_URI'] ); 
// Eliminamos los espacios del principio y final. 
$requestURI = array_values( array_filter( $requestURI ) ); 
 
$controllerName = ucfirst( $requestURI[0] ); 
$controllerPath = CONTROLLER_PATH . $controllerName . EXT; 
 
try 
{ 
    // Comprobamos que el controlador existe. 
    if ( ! file_exists( $controllerPath ) ) 
    { 
        throw new DomainException( 'El archivo <code>' . $controllerPath . '</code> no existe.' ); 
    } 
 
    require_once $controllerPath;     
 
    if ( ! is_callable( $controllerName ) ) 
    { 
        throw new RuntimeException( 'El archivo <code>' . $controllerPath . '</code> debe contener un objeto <code>' . $controllerName . '</code>.' ); 
    } 
} 
catch ( RuntimeException $e ) 
{ 
    echo $e->getMessage(); 
} 
catch ( DomainException $e ) 
{ 
    echo '<strong>Error 404.</strong> ' . $e->getMessage(); 
} 
 
?> 
 
</body> 
</html>

Como podeis ver lanzo dos excepciones, pero, ¿son las apropiadas para cada caso? No se si me explico, ¿son las excepciones que acompañan a cada tipo de error o deberia utilizar otra de las clases predefinidas?

Y ya de paso otra pregunta, ¿como puedo comprobar si un objeto es instanciable (o si esta definido en su defecto)? Notareis en el codigo que is_callable() no funciona.

Muchas gracias a todos. Saludos.

Italico76 · #2 (**permalink**) 10/07/2013, 16:18

Te ayudo con tu seguna pregunta..... method_exists() e is_callable() son para METODOS y no para los objetos en si

Si quieres saber si algo es un objeto es con is_object() sino si quieres saber si es de un tipo particular haces uso del operadir instanceof

Luego si es un objeto ya puedes preguntarle a ese objeto por un METODO

Karmac · #3 (**permalink**) 10/07/2013, 16:33

is_object() comprueba que una variable sea un objeto, y instanceof comprueba que una variable sea instancia de un objeto. Lo que necesito es saber si un objeto se puede llamar pero antes de llamarlo, o en su defecto, llamarlo sin que produzca ningun error y comprobar si existe.

Saludos.

jonni09lo · #4 (**permalink**) 10/07/2013, 16:40

Has probado con is_callable??

Saludos

Karmac · #5 (**permalink**) 10/07/2013, 16:44

Es la que está en el código, no funciona (o la empleo mal).

Saludos.

jonni09lo · #6 (**permalink**) 10/07/2013, 16:50

Tu incluyes el controlador para luego crear una instancia de ese controlador?

Saludos

Karmac · #7 (**permalink**) 10/07/2013, 16:58

Me gustaría no tener que crear una instacia si pudiese ser.

jonni09lo · #8 (**permalink**) 10/07/2013, 17:01

Mmm pues me gustaría que nos explicaras cual quieres que sea el funcionamiento de ese código, seria bueno saber cual debe ser el comportamiento para de esta manera poder o mejorar el código propuesto o tratarlo de hacer de otra manera

Saludos

masterpuppet · #9 (**permalink**) 10/07/2013, 17:06

En la doc las excepciones tienen una pequeña descripción que indica cuando se deberían utilizar, en tu ejemplo todas son Runtime pero esto como tantas otras cosas es subjetivo.
Sobre como comprobar si un objeto es instanciable, simplemente comprueba que la clase existe con class_exists

Saludos.

Italico76 · #10 (**permalink**) 10/07/2013, 17:11

Cita:

Iniciado por masterpuppet

Sobre como comprobar si un objeto es instanciable, simplemente comprueba que la clase existe con class_exists

Pues en realidad una clase puede ser abstracta o puede ser concreta pero con su constructor privado (o protected aunque no tiene sentido protected) ...en conclusion, eso no es suficiente

Código PHP:

Ver original<?php
 
Class MedejoInstanciar{
}
 
Class NoMeInstancies{
   private function __construct(){}
 
   public function some_function(){
      // ...
   }
}
 
 
var_dump(get_class_methods('NoMeInstancies'));
 
$classes = array ('Algo','MedejoInstanciar','NoMeInstancies');
 
foreach ($classes as $class)
{
    if (class_exists($class)){
        echo "$class existe .. <br/>";
        if (method_exists($class,'__construct')){
            echo "Constructor existe en $class<br/>";   
            $metodos_publicos = get_class_methods($class);        
            if (array_search('__construct',$metodos_publicos))
                echo "Constructor es Publico => $class es INSTANCIABLE";
            else
                echo "De malas con $class pues NO es instanciable";
        }else
            echo "Constructor NO existe en $class<br/>"; 
    }else
        echo "$class NO existe .. <br/>";
}

Es un ejemplo.... no es para que lo uses asi!

masterpuppet · #11 (**permalink**) 10/07/2013, 17:40

@Italico76, eso es buscarle el pelo al huevo en el contexto del OP es mas que suficiente, y si necesitas tanta introspección utiliza Reflection.

Italico76 · #12 (**permalink**) 10/07/2013, 17:47

Cita:

Iniciado por masterpuppet

@Italico76, eso es buscarle el pelo al huevo en el contexto del OP es mas que suficiente, y si necesitas tanta introspección utiliza Reflection.

Me parece importante chequear si el constructor es privado.. pues en clases con todos metodos estaticos es buena idea impedir la instanciacion...

Ahora... nada de esto es infalible .... pues ni siquiera con Reflection puedes darte cuenta que un Singleton es instanciable ... o si ?

Voy a cambiar una parte del codigo:

Código PHP:

Ver originalforeach ($classes as $class)
{
    if (class_exists($class)){
        echo "$class es una clase... <br/>";
        $metodos_publicos = get_class_methods($class);        
        $public_construct_exists = array_search('__construct',$metodos_publicos);
 
        if (method_exists($class,'__construct')){
            echo "Constructor existe en $class<br/>";             
            if ($public_construct_exists)
                echo "Constructor es Publico => $class es INSTANCIABLE";            
                
        }else{
            echo "Constructor NO existe en $class<br/>"; 
            if (!$public_construct_exists){
                echo '... pero SI es instanciable (de no ser abstracta)<br/>';
            }
                
        }
 
    }else
        echo "$class NO es una clase (!) <br/>";
 
    echo '<p/>';
}

Ahi me cubro del caso de mas casos ... ojala no se considere "duplicado" pues mi unica intencion es ayudar

masterpuppet · #13 (**permalink**) 10/07/2013, 18:20

Yo entiendo que hay precondiciones y que en el directorio controllers hay controladores y no papas, de todas formas como te comente antes, si es necesaria tanta introspección preferible Reflection

Código PHP:

Ver original$reflector = new ReflectionClass('FooClass');
if ($reflector->isInstantiable()) {
    ...
}

y bueno sobre el antipattern de Singleton no es necesario saber si es instanciable siempre se puede forzar la instanciacion con Reflection.

Saludos.

Italico76 · #14 (**permalink**) 10/07/2013, 18:34

@masterpuppet: nunca use ninguna de las clases de Reflection asi pues no conocia el metodo isInstantiable() pero es bueno saber hay una forma facil e incluso mejor de chequeo, +1

Karmac · #15 (**permalink**) 11/07/2013, 01:13

Bien, ya está solucionado, muchas gracias @masterpuppet y @Italico76.

Respecto a todo lo que comentáis… ni siquiera había planteado que los controladores sean abstractos, estáticos o contengan rosas, es para un tutorial y me basta con mostrar el ejemplo del funcionamiento, es decir que el index.php haga de enrrutador y llame a los controladores. Luego es cosa de cada uno implementarlo y moldearlo a su manera.

Saludos y gracias de nuevo.