[php]Początki OOP Opcje

[Mefiuu]

Witam. Rozpocząłem żmudne zagłębianie się w tajniki programowania zorientowanego obiektowo, przeczytałem kilka artykułów i pomyślałem że najlepiej jest sprawdzić swoją wiedzę w praktyce. Tak oto powstała klasa do uploadu plików. Przyznam się, że pobrałem sobie podobną klasę z phpclasses.org, aby zobaczyć mniej więcej jak to wygląda. Chciałbym Was zapytać czy ma to coś w ogóle wspólnego ze stylem OOP ?

class.upload.php

[PHP] pobierz, plaintext 
<?php
class Upload {
	var $folder;
	var $types = array('application/msword', 'application/octet-stream', 'image/jpeg'); 
 
	function __construct($folder) {
		$this->folder = $folder;
		$this->upload();
	}
 
	public function check() {
            if (isset($_POST['wyslij'])) {
		if($_FILES['plik']['error']>0) {
			echo "Wystąpił problem: ";
			switch($_FILES['plik']['error']) {
				case 1: echo "rozmiar pliku przekroczył wartość xMB.<br /><a href='java script:history.back(1)'>Wróć</a>"; break;
				case 2: echo "rozmiar pliku przekroczył wartość xMB.<br /><a href='java script:history.back(1)'>Wróć</a>"; break;
				case 3: echo "plik wysłany częściowo.<br /><a href='java script:history.back(1)'>Wróć</a>"; break;
				case 4: echo "nie wysłano żadnego pliku.<br /><a href='java script:history.back(1)'>Wróć</a>"; break;
				case 6: echo "nie można wysłać pliku: nie wskazano katalogu tymczasowego.<br /><a href='java script:history.back(1)'>Wróć</a>"; break;
				case 7: echo "nie zapisano pliku na dysku.<br /><a href='java script:history.back(1)'>Wróć</a>"; break;
				}
			}
		else {
			if (in_array($_FILES['plik']['type'], $this->types)) {
				return true;
			}
		}
	}
}
 
public function upload() {
		if($this->check()) {
			if (is_uploaded_file($_FILES['plik']['tmp_name'])) {
					if(!file_exists($this->folder.'/'.$_FILES['plik']['name'])) {
						if(move_uploaded_file($_FILES['plik']['tmp_name'], $this->folder.'/'.$_FILES['plik']['name'])) {
								throw new Exception('Dodano plik pomyślnie');
					}
					else {
						throw new Exception('Nie dodano pliku!');
					}
				}
				else {
					throw new Exception('Taki plik już istnieje!');
				}
			}
		}
		else {
			throw new Exception('Niepoprawny typ!');
		}
	}
}
?>
[PHP] pobierz, plaintext 

index.php

[PHP] pobierz, plaintext 
<?php
 
include('class.upload.php');
 
try {
	$upload = new Upload('files');
}
catch (Exception $e) {
		echo $e->getMessage();
}
 
?>
[PHP] pobierz, plaintext 

Mam co do tego pytania :

1) Wiem że w stylu OOP są wyjątki. Czy to, jak ja je zastosowałem to odpowiednia metoda? Czy jest ich za dużo ? Bo mi nie pasują za bardzo ...
2) Czy wyjątki powinno się dawać do udanych operacji ?
3) Czy w konstruktorze wykonywać metodę 'upload' czy odwołać się do niej poza konstruktorem, już w pliku index?
4) Jeśli mam operacje na bazie danych bądź plikach tekstowych to czy tworzyć destruktor do ich zamykania ?

To takie ogólne pytania i byłbym bardzo wdzięczny gdyby ktoś mi to 'sprawdził', wytknął błędy i polecił dobrą literaturę (IMG:style_emoticons/default/smile.gif)

Dziękuję i pozdrawiam.

Ten post edytował Mefiuu 3.08.2011, 20:46:10

[Mefiuu]

Bardzo dziękuję za odpowiedzi i pomoc ! Crozin, Twoje uwagi okazały się bardzo pomocne. Zmieniłem nieco mój kod i obecnie wygląda on tak:

class.uploader.php

[PHP] pobierz, plaintext 
<?php
class Uploader {
	public $folder;
	public $types = array('application/msword', 'application/octet-stream', 'image/jpeg'); 
	public $file;
 
 
	function __construct($file, $folder) {
		$this->folder = $folder;
		$this->file = $file;
	}
 
	public function checkType() {
		if($_FILES[$this->file]['error']>0) {
			switch($_FILES[$this->file]['error']) {
				case 1: throw new Exception ("rozmiar pliku przekroczył wartość xMB."); break;
				case 2: throw new Exception ("rozmiar pliku przekroczył wartość xMB."); break;
				case 3: throw new Exception ("plik wysłany częściowo."); break;
				case 4: throw new Exception ("nie wysłano żadnego pliku."); break;
				case 6: throw new Exception ("nie można wysłać pliku: nie wskazano katalogu tymczasowego."); break;
				case 7: throw new Exception ("nie zapisano pliku na dysku."); break;
			}
		}
		if (!in_array($_FILES[$this->file]['type'], $this->types)) {
			return false;
		}
		else {
			return true;
		}
	}
 
	public function uploadFile() {
		if(!$this->checkType()) {
			throw new Exception('Niepoprawny typ pliku!');
		}
 
		if (!is_uploaded_file($_FILES[$this->file]['tmp_name'])) {
			throw new Exception('Nie wysłano pliku!');
		}
 
		if(file_exists($this->folder.'/'.$_FILES[$this->file]['name'])) {
			throw new Exception('Taki plik już istnieje');
		}
 
		if(!move_uploaded_file($_FILES[$this->file]['tmp_name'], $this->folder.'/'.$_FILES[$this->file]['name'])) {
			throw new Exception('Nie dodano pliku');
		}
 
		return true;
	}
}
?>
[PHP] pobierz, plaintext 

index.php

[PHP] pobierz, plaintext 
<?php
 
include('class.uploader.php');
 
try {
	$upload = new Uploader('file','files');
	if ($upload->uploadFile()) {
		echo 'Udało się wysłać plik!';
	}
}
catch (Exception $e) {
		echo $e->getMessage();
}
 
?>
[PHP] pobierz, plaintext 

Mam teraz kolejne pytanie, czy poprawnie został użyty kod :

[PHP] pobierz, plaintext 
if ($upload->uploadFile()) {
		echo 'Udało się wysłać plik!';
}
[PHP] pobierz, plaintext 

?

Jednak dalej nie wiem, jak zmienić ten główny blok try-catch. Ponadto, jak Crozin wspomniałeś, nie powinienem stosować tablic globalnych, tylko je przekazywać do obiektu. W jaki sposób (przykładowo) mogę przekazać do obiektu tablicę $_FILES?

@Spawnm:
Nie wiem czy o to Ci chodziło, ale teraz do konstruktora przekazuję nazwę $_FILES i jeśli jest to $_FILES['avatar'] to wystarczy że to do konstruktora przekażę. Czy źle zrozumiałem?
Ponadto pozbyłem się wyjątku w udanej operacji, oraz kodu html.

Dziękuję jeszcze raz za wszelką pomoc i proszę o kolejne uwagi.

Pozdrawiam.


P.S. Spawnm to Twoje 400 'Pomógł'. Gratulacje (IMG:style_emoticons/default/wink.gif)