Pierwszy skrypt w OOP Opcje

[SN@JPER^]

Witam, mój pierwszy skrypt, klasa...

Dobrze załapałem póki co programowanie obiektowe? Jakie błędy robie? Co poprawić?

[PHP] pobierz, plaintext 
class UploadFiles{
    private $file;
    private $dir;
    private $mime = array('image/jpeg', 'image/gif', 'image/png');
    private $chmod=777;
 
    public function  saveFile($fileName, $dir) {
        $this->file=$_FILES[$fileName];
        $this->dir=$dir.'/'.$this->file['name'];
 
        if(isset($_FILES[$fileName])){
 
            if($this->file['error']==0){
                if(is_uploaded_file($this->file['tmp_name'])){
                    if(in_array($this->file['type'], $this->mime)){
                        if(move_uploaded_file($this->file['tmp_name'], $this->dir)===true){
                            chmod($this->dir, $this->chmod);
                            return true;
                        }
                    }else{
                        //Nieprawidłowy format pliku
                    }
                }
            }else{
                switch($_FILES[$this->file]['error']){
                    case 1: /* KOMUNIKAT o przekroczeniu wartości upload_max_filesize*/; break;
                    case 2: /* KOMUNIKAT o przekroczeniu wartości max_filesize*/; break;
                    case 3: /* KOMUNIKAT o wysłanym częściowo pliku*/; break;
                    case 4: /* KOMUNIKAT o nie wysłaniu żadnego pliku*/; break;
                }
            }
        }else{
            return false;
        }
    }
 
}
[PHP] pobierz, plaintext 

[SN@JPER^]

A nie lepiej recznie sprawdzac naglowek pliku?

Jak?

Dobra idea tego sprawdzania rozszerzenia?

[PHP] pobierz, plaintext 
class Upload{
    private $dir;
    private $type = array('jpg', 'gif', 'png');
    private $chmod = 0777;
    private $file;
 
    public function __construct($fileName, $dir){
        $this->file = $_FILES[$fileName];
        $this->dir = $dir;
 
        if(isset($this->file)){
            $nameFile = $this->createTitle($this->file['name']);
            if(!file_exists($this->dir.'/'.$nameFile)){
                if(is_uploaded_file($this->file['tmp_name'])){
 
                    if(move_uploaded_file($this->file['tmp_name'], $this->dir.'/'.$nameFile)){
                        chmod($this->dir.'/'.$this->file['name'], $this->chmod);
 
                        $d='';
 
                        $d = opendir($dir);
                        while(false !== ($f = readdir($d))) {
                            if($f == $this->file['name']){
                                $ex = explode('.', $f);
                                if(in_array($ex[1], $this->type)){
                                    //wgrano
                                }else{
                                    if(unlink($this->dir.'/'.$this->file['name'])){
                                        echo 'Zły format pliku!';
                                    }
                                }
                            }
                        }
                        closedir($d);
                    }
                }
            }else{
                echo 'Taki plik już istnieje!';
            }
        }
    }
 
    private function createTitle($title){
        $title = strtolower($title);
        $title = str_replace(' ', $this->replacechar, $title);
        return $title;
    }
}
[PHP] pobierz, plaintext 

Ten post edytował SN@JPER^ 5.03.2010, 17:23:45