[MySQL][PHP] Klasa upload Opcje

[potreb]

Witam. Napisałem bardzo prostą klasę uploadu plików na serwer. Proszę o jakieś uwagi.
Całość u mnie działa, więc teraz potrzebuję uwag, co można zmienić, wyrzucić itp.

[PHP] pobierz, plaintext 
class Upload {
    public $extensions = array('.jpg','.png','.gif','.doc','.rtf','.pdf','.sql','.txt');
    public $prefix;
    public $max_size;
    public $cf;
    public $info;
 
    private $tmp_name;
    private $name;
    private $error;
    private $size;
 
 
    public function __construct() {
        if(isset($_FILES['file']) && is_array($_FILES['file'])) {
            $this->tmp_name = $_FILES['file']['tmp_name'];
            $this->name = $_FILES['file']['name'];
            $this->error = $_FILES['file']['error'];
            $this->size = $_FILES['file']['size'];
        }
 
            $this->max_size = 200000;
            $this->cf = 3;
    }
 
    private function uploadExt($v) {
        $str = (strpos($v,'.')===false?'':'.'.substr(strrchr($v, "."), 1));
        return $str;
    }
 
    private function uploadName($v) {
        $str = preg_replace('/[^0-9a-zA-Z-\_.]/', '', $v);
        return $str;
    }
 
    private function uploadPrefix()
    {
        if(!$this->prefix) {
            $prefix = date("ymds_");
            return $prefix;
        } else {
            return $this->prefix;
        }
    }
 
    public function uploadFiles($upload_dir)
    {
        for($i=0; $i<$this->cf;$i++)
        {
            if($this->name[$i])
            {
                if(in_array($this->uploadExt($this->name[$i]), $this->extensions))
                {
                    if($this->size[$i] <= $this->max_size)
                    {
                        $target = $upload_dir . $this->uploadPrefix() . $this->uploadName($this->name[$i]);
                        if(move_uploaded_file($this->tmp_name[$i],$target)) {
                            $this->info[] = "Plik został wysłany";
                        } else {
                            $this->info[] = "Nie można wysłać pliku";
                        }
 
                    } else
                    {
                        $this->info[] = "Rozmiar pliku przekracza dopuszczalną wartość!";
                    }
                } else
                {
                    $this->info[] = "Plik o tym rozszerzeniu nie może być wysłany!";
                }
            } else {
                $this->info[] = "Plik nie został wybrany";
            }
        }
    }
 
}
 
$upload = new Upload;
$upload->cf = 2;
 
if(isset($_POST['upload'])) {
    $upload->max_size = 144550333;
    $upload->extensions = array('.png','.sql','.txt');
 
    $upload->uploadFiles('./files/');
    foreach($upload->info as $value) {
        echo "<p>".$value."</p>";
    }
}
 
 
 
echo "<form action='index.php' method='post' enctype='multipart/form-data'>";
for($i=0;$i<$upload->cf;$i++) {
echo "<p><input name='file[]' type='file' /></p>";
}
echo "<p><input type='submit' name='upload' value='Upload' /></p></form>";
[PHP] pobierz, plaintext 

[Spawnm]

Przenoszę (IMG:style_emoticons/default/smile.gif)

Przenoszę (IMG:style_emoticons/default/smile.gif)

[ayeo]

Witam!

Niewiele to ma wspólnego z OOP. Równie dobrze mogłeś zrobić samą funkcję uploadFiles(); Rozszerzenia zapisane są na sztywno w samej klasie, brak sprawdzania MIME. Co z rozszerzeniami typu .tar.gz? Brak jakiejkolwiek kontroli nad nazwą pliku. Klasa uzależniona od formularza (nazwa inputa), a dodatkowo pozwalająca na upload jednego pliku. Konstruktor powinien brać chyba nazwę inputa jako parametr, wtedy można uploadować dowolna ilość plików. Nie jestem pewien, ale tablica na komunikaty jest zbędna - wykluczają się wzajemnie. Samo API klasy też pozostawia wiele do życzenia. Ogólnie mówiąc nic specjalnego.

Pozdrawiam!

[potreb]

Dzięki za uwagi. Wiem, że to jest nic specjalnego i nie będzie.
Typy plików nie są ustawione na sztywno tylko w klasie, bo możesz ustawić własne.
Nazwa pliku jest czyszczona z niepotrzebnych znaków także kontrola nazwy jest. Także możliwość wysyłania wielu plików.
Wiem za API i ogólnie wszystko pozostawia wiele do życzenia, tak jak mówisz można zrobić wszystko w jednej funkcji.
BTW chcę czegoś się nauczyć, a zaczynanie od gotowych rzeczy to ominięcie

[mijagi]

Dodaj jakiś warunek sprawdzający poprawnośc obrazków, da się uploadować skrypt z rozszerzeniem obrazka. Później na stronie będą błędy o nieprawidłowych obrazkach. Sprawdzaj to np. getimagesize, a tu masz link pomocniczy : http://mijagi.interkuza.pl/index.php/2009/...-uploadu-w-php/

Ten post edytował mijagi 10.08.2009, 09:44:44

[skowron-line]

[PHP] pobierz, plaintext 
private function uploadExt($v) {
$str = (strpos($v,'.')===false?'':'.'.substr(strrchr($v, "."), 1));
return $str;
}
[PHP] pobierz, plaintext 

Zainteresuj się takową funkcją zamiast pisać jakieś cuda na kiju.
http://pl.php.net/manual/pl/function.pathinfo.php