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 

[blooregard]

Teoretycznie sprawdzanie typu mime też da się obejść

Oczywiście, wszystko da się obejść.
Ale nie jest to już tak trywialne, jak zmiana rozszerzenia.

Ale oczywiście, nic nie stoi na przeszkodzie, by sprawdzać również rozszerzenie. Generalnie - powinno się walidować wszystko, co się da, jeśli dopuszczamy upload plików na serwer przez "niezaufane" źródła (czyli użyszkodników)