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]

Jak już wspomniał @phpion, sprawdzanie rozszerzenia zda się psu na budę.
Podstawową kwestią jest sprawdzenie typu mime pliku, np. w ten sposób:

[PHP] pobierz, plaintext 
$_avaiable_mimetypes = array('plain/text' , 'application/pdf' , 'application/x-pdf' , 'image/jpg' , 'image/jpeg' , 'image/png' , 'image/gif' , 'image/pjpeg' ,
        'application/vnd.ms-excel' , 'application/msword' , 'application/vnd.ms-powerpoint' );
if (!in_array($_FILES['file']['type'] , $_avaiable_mimetypes)) {
            $_errors['file_upload_error'] = '<p><span style="color:#f00;">Nieprawidłowy typ pliku.</span></p>';
        } 
[PHP] pobierz, plaintext 

PS: Kod jest żywcem wycięty, nie bawiłem się w edycję, więc proszę nie pytać, co to jest np. to: $_errors['file_upload_error'] (IMG:style_emoticons/default/smile.gif)