Witam,

Jest dużo rozbudowanych klas do upload-u plików ale ja potrzebowałem prostej która na każdym etapie ładowania pliku zwraca błąd jeśli taki wystąpi. Dodatkowo opcjonalnie można ustawić automatyczną zmianę nazwy ładowanego pliku na unikalną. Moze komus się przyda.

Zgodnie ze słusznymi uwagami nospora poprawilem klase ktora dodatkowo zwraca true, false. Poprawilem tez ladowanie pliku bez zmieniania nazwy gdzie jesli istnieje plik o tej samej nazwie zostanie zwrocony komunikat. Dodana zostala tablica zmieniajaca "type" w przypadku uploadowania w IE. Do systemu tworzacego unikalna nazwe zostalo dodane tez IP.


Kod klasy:

[PHP] pobierz, plaintext 
<?php
class Upload{	
 
	public $name_file;
	public $name;
	public $type;  
	public $size; 
	public $tmp; 
	public $message;
 
	public function Save($file_user, $upload_dir, $typ = array(), $rename = null, $prefix = null )
	{
		if (isset($_FILES[$file_user]['name'])){
			$uploadErrors = array(
			UPLOAD_ERR_OK				=> 'Plik został poprawnie załadowany.',
			   UPLOAD_ERR_INI_SIZE	   => 'Rozmiar pliku jest za duży. Ograniczenie na serwerze.',
			UPLOAD_ERR_FORM_SIZE   => 'Rozmiar pliku jest za duży.',
			UPLOAD_ERR_PARTIAL		=> 'Plik tylko częściowo załadowany.',
			UPLOAD_ERR_NO_FILE		=> 'Brak pliku do załadowania.',
			UPLOAD_ERR_NO_TMP_DIR => 'Brak tymczasowego folderu tmp !',
			UPLOAD_ERR_CANT_WRITE => 'Nie można zapisać pliku na dysk.'
			/* wymaga php 5.2.0
			UPLOAD_ERR_EXTENSION  => 'File upload stopped by extension.' */
			);
 
			$this->name	   = $_FILES[$file_user]['name'];
			$this->type		= $_FILES[$file_user]['type'];
			$this->size	   = $_FILES[$file_user]['size'];
			$this->tmp		= $_FILES[$file_user]['tmp_name'];
			$this->message	= $uploadErrors[$_FILES[$file_user]['error']];
 
			$uploadfile = $upload_dir . basename($_FILES[$file_user]['name']);
 
			if ($_FILES[$file_user]['error'] == 0){
				$this->type = explode('/',$this->type);
				// zamiana rozszerzen w przypadku uploadu w IE
				$replace_prefix = array('pjpeg','x-png');
				$new_prefix	 = array('jpeg','png');
				$new_type  = str_replace($replace_prefix, $new_prefix , $this->type[1]);
				if (in_array($new_type, $typ)){
					// jesli parametr zmiany nazwy jest nie ustawiony rename = null
					if($rename == null){
						if (file_exists($uploadfile)){
							$this->message = 'Plik o nazwie '.$_FILES[$file_user]['name'].' już istnieje. Zmień nazwę pliku i załaduj ponownie.';
							return false;
						}else{
							if (move_uploaded_file($_FILES[$file_user]['tmp_name'], $uploadfile)){
								$this->message = 'Plik został załadowany pomyślnie';
								$this->name_file = $this->name;
								return true;
							}else{
								$this->message = 'Błąd ładowania pliku';
								return false;
							}
						}
					}
					if($rename != null){
						if (move_uploaded_file($_FILES[$file_user]['tmp_name'], $uploadfile)){
							$this->message = 'Plik został załadowany pomyślnie';
							$ip = $_SERVER['REMOTE_ADDR'];
									$new_name = md5($ip . date('YmdHis'));
									if(rename($uploadfile, $upload_dir . $prefix. $new_name .'.'. $new_type)){
										$this->name_file = $prefix . $new_name . '.' . $new_type;
										return true;
									}else{
										$this->message = 'Nie można zmienić nazwy pliku.';
										return false;
									}
							return true;
						}else{
							$this->message = 'Błąd ładowania pliku';
							return false;
						}
					}
				}else{
					$this->message = 'Błąd, nie obsługiwany format pliku.';
					return false;
				}
			}else{
				$this->message;
				return false;
			}
		}else{
			$this->message = 'Błędny parametr "name".';
			return false;
		}
	}
}
?>
[PHP] pobierz, plaintext 

Wyjaśnienie: public function Save($file_user, $upolad_dir, $typ = array(), $rename = null, $prefix = null )

$file_user = nazwa pola name w formularzu uploadu ( wymagane )(string)
$upload_dir = sciezka do katalogu gdzie ma być zapisany plik (wymagane)(string)
$typ = tablica z formatami plikow jakie akceptujemy (wymagane)(array)
$rename = jesli ustawimy na 1 bedzie automatycznie zmieniac nazwe pliku na unikalna unikalna nazwa to ip + data + czas zaszyfrowane funkcja szyfrujaca md5 , to daje nam gwarancje ze nazwa pliku bedzie unikalna i nie ma szans aby ktos petla pobral sobie nasze pliki.
$prefix = jesli ustawimy parametr rename na 1 dodatkowo mozemy opcjonalnie ustawić prefix który bedzie dodawany do nazwy pliku po załadowaniu (opcjonalnie)(string)

Przyklad zastosowania:

[PHP] pobierz, plaintext 
<?php
// przyklad z automatyczna zmiana nazwy pliku 
$plik = $_POST['plik'];
$id   =  1
 
$upload = new Upload();
$image = array('jpeg','jpg','png','gif');
$upload->Save('plik','./upload/zdjecia/', $image, 1,$id);
   
echo $upload->message;  //  zwraca wszystkie komunikaty w tym bledy
echo $upload->file_name; // zwraca nazwę pliku
 
// przyklad bez zmieniania nazwy pliku. 
$plik = $_POST['plik'];
 
$upload = new Upload();
$image = array('jpeg','jpg','png','gif');
$upload->Save('plik','./upload/zdjecia/', $image);
   
echo $upload->message;  
echo $upload->file_name; 
?>
[PHP] pobierz, plaintext 

Klasa jest prosta i mam swiadomosc ze mozna ja o wiele bardziej rozbudowac ale wlasnie oto w niej chodzilo, ma sluzyc tylko do uploadu a obrobka zdjec to juz inne klasy sie zajma.

Pozdrawiam

Jesli zapoda sie nazwe, ktora nie istnieje, to poleci masa noticow a od Ciebie komunikat:
No cos nie tak

@nospor - Ty to każdemu umiesz to wytknąć

@autor - Powiedz mi, po co to jest klasa? Dlaczego nie funkcja? Co Ci daje to, że tworzysz obiekt tej klasy?

Pozdrawiam.

@Cysiaczek no co, musze sie na kims wyzyc

Z innych popierdół to Save powinno zwracac true lub false, a u CIebie nie ma nic. Wiem, ze jest $error, ale Save powinno to mieć tez w sobie

edit:
$this->error = 'Plik został załadowany';
No jesli to jest error...

edit: rozkrecam sie na dobre:
1) rename($uploadfile, $upolad_dir . $prefix. $new_name .'.'. $this->type[1]);
rename tez moze zwrocic blad
2) No dobra, zrobilo mi rename i... ups, a jak mam pobrac plik co mi go zrobil rename? Przeciez nie znam jego nowej nazwy
3) Proponuje zrobic upload pliku jpg z rename najpierw w IE a potem np. w FF. A nastepnie przyjrzec sie nazwom plikow. Jakies wnioski?

Klasa dlatego ze pisze we frameworku zend. Co do nospora to spokojnie, niech sie wyzyje, ma racje więc dobrze ze napisal. On juz wie dlaczego tak dokladnie sprawdzil akurat moja klase hehe.

nospor ma racje co do IE, nie wzialem pod uwage tej pseudo przegladarki. Niebawem bedzie update .

Ale... co ma pisanie w ZF do tego, że to jest bez sensu? Nie chcę znów wpadać w histeryczno-obiektowy ton, ale zrobienie z tego klasy to jak zakup budki z hot-dogami, żeby zjeść jednego. Zrób chociaż jakąś obsługę błędów, dodaj osobne metody zamiast argumentów - inaczej mówiąc zrób interfejs.

Pozdrawiam.

Nie ma powodu do wpadania w histerie Przy okazji patrzac na linki w twoim podpisie rozumiem czemu mnie ganiasz zeby dorzucic reszte zamiast samej klasy