[PHP]Pierwsza klasa Opcje

[--gość--]

Witam uczę się właśnie programowania obiektowego i napisałem taką prostą klasę, czy moglibyście mnie nakierować na moje błędy?

[PHP] pobierz, plaintext 
<?php
 
/**
 * Klasa odpowiedzialna za upload
 *
 */
class Upload {
 
    /**
     * Tablica zawierająca dopuszczalne formaty plików, 
     * jeżeli pusta to pozwala na wgrywanie dowolnego rodzaju pliku
     * @var array 
     */
    protected $aAllowedExtensions = array();
 
    /**
     * Zawiera rozszerzenie pliku
     * @var string 
     */
    protected $sFileExtension = '';
 
    /**
     * Dopuszczalny rozmiar pliku w bitach, domyślnie 1 MB
     * @var int 
     */
    protected $iMaxSize = 8388608;
 
    /**
     * Tablica zawierająca kody błędów
     * @var array 
     */
    protected $aErrorCodes = array(
        1, //plik ma błędny typ
        2, //plik jest zbyt duży
        3, //brak pliku
        4, //ścieżka nie pozwala na zapis
        5. //inny błąd
    );
 
    /**
     * Ścieżka gdzie pliki mają być wgrywane
     * @var string 
     */
    protected $sUploadPath = '';
 
    /**
     * Nazwa wgranego pliku, jeżeli pusta zostanie wygenerowana losowa
     * @var string 
     */
    protected $sFileName = '';
 
    /**
     * Czy zachować oryginalną nazwę pliku, 
     * uwaga jeżeli true, to sFileName nie jest brane pod uwagę i 
     * zostanie nadpisana
     * @var bool 
     */
    protected $bOriginalName = false;
 
    /**
     * Zawiera pełną ścieżkę wgranego pliku
     * @var string 
     */
    protected $sResult;
 
    /**
     * Zawiera kod błędu
     * @var int 
     */
    protected $iErrorCode = 0;
 
 
    /**
     * Konstruktor
     * @param string $sUploadPath
     * @param array $aAllowedExtensions [optional]
     * <p>Dopuszczalne formaty plików</p>
     * @param int $iMaxSize [optional]
     * <p>Maksymalny rozmiar pliku wgrywanego, 
     * podawana w bitach, domyślnie 1MB (8388608 bitów)</p>
     * @return \lib_Upload 
     */
    public function __construct($sUploadPath, $aAllowedExtensions = array(), $iMaxSize = 8388608) {
        $this->sUploadPath        = (string) $sUploadPath;
        $this->iMaxSize           = (int) $iMaxSize;
        $this->aAllowedExtensions = $aAllowedExtensions;
        return $this;
        html_entity_decode($sUploadPath);
    }
 
    /**
     * @param type $sFileName
     * @return \lib_Upload 
     */
    public function setFileName($sFileName) {
        $this->sFileName = $sFileName;
        return $this;
    }
 
    /**
     * Zwraca kod błędu
     * @return int 
     */
    public function getErrorCode() {
        return $this->iErrorCode;
    }
 
    /**
     * Zwraca pełną scieżkę do pliku wgranego
     */
    public function getResult() {
        return $this->sResult;
    }
 
    /**
     * Metoda sprawdzająca czy plik ma poprawne rozszerzenie
     * return bool 
     */
    public function checkExtension($sFileName) {
        $aTmp = explode('.', $sFileName);
        if(!empty($aTmp)) {
            $this->sFileExtension = strtolower(end($aTmp));
        }
 
        if(empty($this->aAllowedExtensions) || in_array($this->sFileExtension, $this->aAllowedExtensions)) {
            return true;
        } else {
            return false;
        }
    }
 
    public function moveFile($sFileIndex) {
        if($this->bOriginalName) {
            $this->sFileName = $_FILES[$sFileIndex]['name'];
        } elseif(empty($this->sFileName)) {
            $this->sFileName = md5($_FILES[$sFileIndex]['name']).uniqid().'.'.$this->sFileExtension;
        } 
        if(!empty($this->sUploadPath) && substr($this->sUploadPath, -1) !='/') {
            $this->sUploadPath .= '/';
        }
        $this->sResult = $this->sUploadPath.$this->sFileName;
 
        if(move_uploaded_file($_FILES[$sFileIndex]['tmp_name'], $this->sResult)) {
            return true;
        } else {
            return false;
        }
    }
 
    /**
     * Metoda odpowiedzialna za upload
     * @param string $sFileIndex
     * @return boolean 
     */
    public function upload($sFileIndex) {
        if(isset($_FILES[$sFileIndex]) && is_uploaded_file($_FILES[$sFileIndex]['tmp_name']) ){
            if($_FILES[$sFileIndex]['size'] > $this->iMaxSize ){
                $this->iErrorCode = 2;
                return false;
            }
            if(!$this->checkExtension($_FILES[$sFileIndex]['name'])) {
                $this->iErrorCode = 1;
                return false;
            }
            if($this->moveFile($sFileIndex)) {
                return true;
            } else {
                $this->iErrorCode = 4;
                return false;
            }
 
        }else {
            $this->iErrorCode = 3;
            return false;
        }
    }
 
}
 
 
$oUpload = new Upload('tmp');
if($oUpload->upload('img')) {
//plik się wgrał poprawnie
} else {
echo $oUpload->getErrorCode(); //kod błędu
}
 
[PHP] pobierz, plaintext 

W zamyślę to będzie klasa matka dla klas odpowiedzialnych za wgrywanie i zabawę z obrazkami czy bardziej specyficznymi plikami

[IceManSpy]

Co do funkcji setFileName zrób jeszcze tak, że jeśli użytkownik nie wywoła tej metody (nie przypisze nazwy), to niech przypisze dla pliku losową nazwę.

Oprócz rozszerzenia powinieneś sprawdzać typ MIME: http://pl.wikipedia.org/wiki/Typ_MIME

Nigdzie nie wykorzystujesz: aErrorCodes .

Ten post edytował IceManSpy 3.02.2012, 23:23:59

[Fifi209]

I zamiast dziwacznych błędów rzucać wyjątkami.

P.S. Nie ten dział

[--gość--]

IceManSpy co do losowej nazwy to już to było:

[PHP] pobierz, plaintext 
if($this->bOriginalName) {
            $this->sFileName = $_FILES[$sFileIndex]['name'];
        } elseif(empty($this->sFileName)) {
            $this->sFileName = md5($_FILES[$sFileIndex]['name']).uniqid().'.'.$this->sFileExtension;
        }
[PHP] pobierz, plaintext 

w metodzie moveFiles

Co do tablicy aErrorCodes to racze ma funkcje przypominającą co który kod oznacza.

Co do typu mime to nie mam pojęcia jak sprawdzić prawdziwy typ mime pliku, bo wiem, że przez przeglądarke można zafałszować.

Fifi209 nie za bardzo rozumiem jeszcze używanie wyjątków (szczególnie kiedy i jak powienienem ich używać), jeżeli znasz jakiś artykuł na ten temat z chęcią go przeczytam. ewentualnie czy mógłbyś pokazać jak użyć przy takiej klasie?