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.