Abstracción de clase

[andresenrique]

Buenas tardes, mi problema es el siguiente, estoy intentando crear una clase sobre una ya existente, por la cuestión de modularidad, es decir, si en X o Y momento quiero migrar a otra clase para subir archivos, solo modificaria mi clase existente sin necesidad de ir a cada módulo donde cargue archivos y modificarlos, al igual como se hace con las abstracciones de base de datos.

Estoy usando PEAR::HTTP_Upload para cargar los archivos.

Mi clase:

Código PHP:

  <?php 
 
class Subir 
{    
    function __construct() 
    { 
        // Instanciamos la claseHTTP_Upload 
        $this->subir = new HTTP_Upload('es');  
    } 
     
    private $archivo; 
     
    function cargaArchivo($archivo) 
    { 
        // Cargamos el archivo a subir 
        $this->file = $this->subir->getFiles($archivo); 
        print_r($this->file); 
         
    } 
     
    private $propiedad; 
     
    function getProp($propiedad) 
    { 
        // Cargamos para mostrar la propiedad que queremos 
        $this->file->getProp($propiedad); 
    } 
     
    private $carpeta; 
     
    function moverA($carpeta) 
    { 
        // Movemos archivo a la carpeta indicada 
        $this->movido = $this->file->moveTo($carpeta); 
    } 
     
    function getMensaje() 
    { 
        // Obtenemos mensaje 
        $this->movido->getMessage(); 
    } 
     
    function esValido() 
    { 
        // Verificamos si el archivo es válido 
        $this->file->isValid(); 
    } 
     
     
    function sinArchivo() 
    { 
        // Verificamos si no se subió un archivo 
        $this->file->isMissing(); 
    } 
     
    function hayError() 
    { 
        // Verificamos si hubo algún error 
        $this->file->isError(); 
    } 
     
    function msjError() 
    { 
        // Mostramos el mensaje de errror 
        $this->file->errorMsg();     
    } 
     
    private $var; 
     
    function errorPEAR($var) 
    { 
        // Verificamos si hubo algun error con PEAR::IsError 
        PEAR::isError($var); 
    } 
} 
?>   

Aquí en donde uso mi clase:

Código PHP:

  <? 
//previamente el la clase cargada con autoload 
$upload         = new Subir(); 
 /// Cargamos archivo proveniente del formulario 
    $upload->cargaArchivo('xml'); 
    // Verificamos si el archivo es valido 
    if ($upload->file->esValido()) 
    { 
        // Movemos el archivo a la carpeta asignada 
        $movido = $upload->MoverA(DIR_UPLOAD); 
        // Verificamos si hubo un error 
        if (errorPEAR($movido)) 
        { 
            echo 'Correctamente subido a: ' .$upload->getProp('ext'); 
        } 
        else 
        { 
            // Mostramos mensaje de error 
            $upload->getMensaje();     
        } 
    } 
    // Verificamos si se subió un archivo 
    elseif ($upload->sinArchivo()) 
    { 
        echo 'Debes subir un archivo'; 
    } 
    // Verificamos si se pudo mover el archivo 
    elseif ($upload->moverA) 
    { 
        // Mostramos mensaje de error 
        echo $upload->msjError(); 
    }   

Le he hecho var_dump para ver donde está el problema y no lo encuentro para nada, la clase está instanciada, pero ni idea de porque no funciona.

Lo probé a 'patica' sin usar la clase que yo he hecho, usando directamente la clase PEAR::HTTP_Upload y funciona -perfectamente-.

Código PHP:

  <?php 
require_once "HTTP/Upload.php"; 
 
$upload = new HTTP_Upload("en"); 
$file = $upload->getFiles("f"); 
 
if ($file->isValid()) { 
    $moved = $file->moveTo('uploads/'); 
    if (!PEAR::isError($moved)) { 
        echo 'File was moved to uploads/' . $file->getProp('name'); 
    } else { 
        echo $moved->getMessage(); 
    } 
} elseif ($file->isMissing()) { 
    echo "No file was provided."; 
} elseif ($file->isError()) { 
    echo $file->errorMsg(); 
} 
?>   

Saludos

[yoguuu]

Pregunta para los expertos ^^

¿ En este caso que propone "andresenrique" no sería apropiado un "Template pattern" o bién un "Adapter/wrapper pattern" ?

En la wikipedia hay ejemplos, puedes mirar a ver si esos patrones de diseño te sirven.

Pero alguién podría confirmarlo pliiiiiiisss ???

Gracias ;)

[GatorV]

El problema que veo es, primero en tu clase solamente llamas a algunas cosas, y nunca regresas con "return" el valor que regresa el método.

Posteriormente en donde implementas llamas a métodos sin usar la instancia de la clase, y luego llamas a otros métodos pero sin los parentesis, lo que es como si estuvieras leyendo una propiedad de la clase en lugar de un método.

SAludos.

[jaskel]

Mi primer consejo es que te leas un buen manual de POO, el segundo es que procures ser ordenado y limpio con el código.

Declara los atributos de la clase al principio de la misma definiendo siempre si son publicos, privado o protegidos, haz lo mismo con los metodos y define los getters y setters de la clase si es necesario, sigue una nomenclatura estandar para definir atriubutos, metodos y clases. por ejemplo defines el metodo moverA y luego lo invocas llamando a MoverA.

Basicamente entiende lo que haces y luego aplicalo de forma clara y ordenada, una vez que que lo hayas hecho de esa manera y te siga sin funcionar vuelve a preguntar :)

[isra00]

Creo que el patrón que vendría bien sería, no Template sino Strategy: construir un "uploader" abstracto (clase abstracta o interfaz) y seguir una u otra "estrategia" (PEAR, el de PHP por defecto, otro futuro...). Más info en http://www.ibm.com/developerworks/li...p-designptrns/. Siento no poder ayudarte con el tema del uploader, pero estoy bastante verde en esa librería.