Aporte: Class para el uso de plantillas

[masterpuppet]

Que tal Hidek,

Primero, gracias por compartir, y segundo te dejo un par de sugerencias

- No deberías lanzar una excepcion en el render, deberías controlarlo en el __construct, ya que es obligatorio para la clase un template y no se debería poder instanciar sin este, luego podrías implementar __toString y en el hacer la llamada a render, de esta forma podrías hacer el echo sobre el objeto directamente.

- La excepción debería pertenecer al componente, Khaus_View_Template_Exception.

Y tienes if/else que me parece que no son necesarios, un ejemplo:

Código PHP:

public function templatePath($filename)
{
    // IF: La ruta del template es correcta
    if (is_file($filename)) {
        $this->_filename = $filename;
    } else {
        $message = sprintf('Template no encontrado (%s)', $filename);
        throw new Exception($message);
    }
}
 

podria ser:

Código PHP:

public function templatePath($filename)
{ 
    if (!is_file($filename)) {
        throw new Khaus_View_Template_Exception(
            sprintf('Template no encontrado (%s)', $filename));    
    }
    $this->_filename = $filename;
}
 

me parece que queda mas legible.

Saludos.