[skrypt] Dodawanie zdjęć - OOP w php 4 Opcje

[Koniczynka]

Od niedawna zacząłem programować obiektowo, a przynajmniej tak mi się wydaje. Mam dlatego prośbę dla was, znających się na OOP od dawna, o sprawdzenie poniższej klasy (php 4).

[PHP] pobierz, plaintext 
<?php
class fotka {
 
		var $opis1;
		var $SUBMIT;
		var $nazwa1;
		var $rodzaj1;
		var $rozszerzenie1;
		var $kategoria;
		var $hotelnazwa;
 
		 function dodawanie2() 
	   {
				  if ($_POST['SUBMIT']  && $this->opis1 != '' && $this->nazwa1 != '' &&  $this->rodzaj1['extension'] == 'jpg'
				  || $_POST['SUBMIT']  &&  $this->opis1 != '' && $this->nazwa1 != '' &&  $this->rodzaj1['extension'] == 'jpeg'
				  || $_POST['SUBMIT2'] && $this->opis1 != '' && $this->nazwa1 != '' &&  $this->rodzaj1['extension'] == 'jpg'
				  || $_POST['SUBMIT2'] &&  $this->opis1 != '' && $this->nazwa1 != '' &&  $this->rodzaj1['extension'] == 'jpeg'
				  ) {
 
				$this->data = date("Y-m-d");
 
				$this->hotelnazwa2 = "$this->hotelnazwa$this->data$this->nazwa1";
 
				move_uploaded_file($this->rozszerzenie1, $this->nazwa1);
				$this->img = imagecreatefromjpeg("$this->nazwa1");
				$this->width  = imagesx($this->img);
				$this->height = imagesy($this->img);
 
				$this->width_mini = 250;
				$this->height_mini = 300;
				$this->img_mini = imagecreatetruecolor($this->width_mini, $this->height_mini);
 
				imagecopyresampled($this->img_mini, $this->img, 0, 0, 0, 0, $this->width_mini , $this->height_mini, $this->width  , $this->height);
 
 
				imagejpeg($this->img_mini, "$this->hotelnazwa2", 80);
				imagedestroy($this->img);
				imagedestroy($this->img_mini);
				 
 
				unlink("$this->nazwa1");
 
 
mysql_query("INSERT INTO fotki (
nazwa_fotki , opis_fotki , kategoria_fotki , do_fotki)
VALUES 
('$this->hotelnazwa2' , '$this->opis1' , '$this->kategoria' , '$this->hotelnazwa' )");
 
echo "Twoje zdjęcie $this->hotelnazwa2 zostało dodane <br />";
 
				}
 
 
				elseif ($_POST['SUBMIT'] &&  $this->hotelnazwa != '' && $this->opis1 != ''
				|| $_POST['SUBMIT2'] &&  $this->hotelnazwa != '' && $this->opis1 != '') {
				echo "Dodaj fotografi&ecirc;";}
 
				elseif ($_POST['SUBMIT'] && $this->opis1 == '' &&  $this->hotelnazwa != '' && $this->rodzaj1['extension'] == 'jpg' || 
				$_POST['SUBMIT'] && $this->opis1 == '' &&  $this->hotelnazwa != '' && $this->rodzaj1['extension'] == 'jpeg'
				||$_POST['SUBMIT2'] && $this->opis1 == '' &&  $this->hotelnazwa != '' && $this->rodzaj1['extension'] == 'jpg' || 
				$_POST['SUBMIT2'] && $this->opis1 == '' &&  $this->hotelnazwa != '' && $this->rodzaj1['extension'] == 'jpeg' ) {
				echo "Je&iquest;eli dodajesz zdjęcie, musisz dodać i opis";
				}
 
				elseif ($_POST['SUBMIT'] &&  $this->rodzaj1['extension'] != 'jpg' &&  $this->hotelnazwa != '' && $this->opis1 != '' ||
				$_POST['SUBMIT'] &&  $this->rodzaj1['extension'] != 'jpeg' &&  $this->hotelnazwa != '' && $this->opis1 != ''
				|| $_POST['SUBMIT2'] &&  $this->rodzaj1['extension'] != 'jpg' &&  $this->hotelnazwa != '' && $this->opis1 != '' ||
				$_POST['SUBMIT2'] &&  $this->rodzaj1['extension'] != 'jpeg' &&  $this->hotelnazwa != '' && $this->opis1 != '') {
				echo "Musisz wgrać zdjęcie o formacie JPG lub JPEG";}
 
						}
 
						}
?>
[PHP] pobierz, plaintext 

Niby wszystko działa tak, jak zamierzyłem. Ale czy skrypt został optymalnie wykonany ? Byłbym bardzo wdzięczny za wszelkie wskazówki i uwagi.

Ten post edytował Koniczynka 20.01.2007, 16:23:57

[Cysiaczek]

No niestety Cie rozczaruję, ale z OOP ma to tyle wspólnego, że jest ujęte w klasę. :\

OOP to programowanie zorientowane obiektowo. Nie chodzi zatem o to, żeby używać klas, tylko aby program korzystał z obiektów do wykonania swoich zadań. Obiekty te wchodzą ze sobą w różne relacje. Twój kod, to kod proceduralny zawarty w klasie. Kolejna sprawa, to hermetyzacja. używasz w ciele klasy tablic globalnych, co jest absolutnie niedopuszczalne, bo niweczy wszelkie korzyści płynące z obiektowości.
Na pocieszenie.
Każdy kiedyś zaczynał i nie ma takich błędów, których programiści nie popełnili. Poczytaj o projektowaniu, o wzorcach projektowych, bo to jest esencja OOP. Zapoznaj się z pojęciami polimorfizm, delegacja, hermetyzacja, abstrakcja, interfejs, agregacja, asocjacja... i jeszcze pewnie kilkanaście. W Wikipedii znajdziesz sporo wyjaśnień, ale polecam dobrą książkę.

Pozdrawiam.

[Koniczynka]

To sprowadziłeś mnie na ziemię, bo myślałem, że weszło mi to po kilku godzinach prób. Dziękuję - na pewno popracuję nad znalezieniem właściwego toru. Niebawem postaram się wkleić poprawiony skrypt.

[ferr]

No i warto dodac, bys zainteresowal sie OOP w kontekscie PHP5. Ten w PHP4 daje niestety male mozliwosci, o ile w ogole mozna to nazwac programowaniem obiektowym.

Ten post edytował ferr 20.01.2007, 22:09:48