Forum PHP.pl

Cześć, otóż zacząłem sobie przyswajać czyste OOP w PHP. Wcześniej wiele pisałem już we frameworkach kod obiektowy, jednakże nigdy nie pisałem własnej klasy, ani dokładniej nie zagłębiałem się w sposób działania klas oraz ich obiektów. Z palca napisałem malutką klasę, czy ktoś może zerknąć i powiedzieć mi, czy dobrze rozumuję zasady działania programowania obiektowego i czy w miarę dobrym stylu piszę? Oczywiście pomijam MVC - oddzielanie widoku tutaj od zarządzania całą resztą. Proszę o konstruktywną opinię. W końcu człowiek musi zacząć programować własne aplikacje pisząc je w stylu obiektowym (pomijam frameworki). Poniżej prezentuję swój kod:

[PHP] pobierz, plaintext 
<?php
	class Article
	{
		public $id;
		private $title;
		private $content;
		private $author;
		private $comment;
 
		public function showArticle()
		{
			/* Reference to MySQL to receive information */
			return $this->id;
		}
		public function deleteArticle()
		{
			/* Reference to MySQL and delete article in MySQL */
			if (1 == 1)
				return true;
			else
				return false;
		}
		public function addArticle($title, $content)
		{
			/* Security input variables and adding the article to MySQL */
			return rand(); // Return added article ID, for example used rand() function
		}
		public function commentArticle($author, $comment)
		{
			/* Security input variables and adding the article to MySQL */
			$article_id = $this->id;
			// For example: 'INSERT INTO comments VALUES ('', '$author', '$comment', $article_id)'
		}
	}
 
	$oArticle = new Article();
	if (isset($_GET['id']))
	{
		$oArticle->id = $_GET['id'];
		echo 'Artykul o ID: '.$oArticle->showArticle().' - <a href="?id='.$_GET['id'].'&delete=true">usun</a>';
		if (isset($_GET['delete']) && $_GET['delete'] == 'true')
		{
			echo '<br /><br />';
			if ($oArticle->deleteArticle() == true)
				echo 'Artykul usuniety';
			else
				echo 'Blad';
		}
	}
	else
	{
		echo 'Dodaj nowy artykul:<br />$title = \'Testowy artykul\';<br />$content = \'tresc... tresc... tresc... tresc... tresc... tresc... tresc... tresc... tresc...\';<br /><br />';
		$title = 'Testowy artykul';
		$content = 'tresc... tresc... tresc... tresc... tresc... tresc... tresc... tresc... tresc...';
		echo $oArticle->addArticle($title, $content);
	}
?>
[PHP] pobierz, plaintext 

Może to kwestia zbyt małej ilości krwi w alkoholu, ale ta Twoja klasa nie ma najmniejszego sensu. W jakim celu deklarujesz obiekty skoro ich nawet nie używasz? Sama klasa jest bezużyteczna, tak samo jak jej metody.

Dodatkowo XSS.

Proszę o nakierowanie kolego b4rt3kk. A co do XSS to przecież nic nie zabezpieczałem w tym skrypcie.

To są Twoje obiekty:

[PHP] pobierz, plaintext 
                public $id;
		private $title;
		private $content;
		private $author;
		private $comment;
[PHP] pobierz, plaintext 

Będą one dostępne wewnątrz metod klasy (i do tego służą), tyle że nie nadajesz im żadnych wartości. Dzieje się to chociażby w ten sposób:

[PHP] pobierz, plaintext 
function setTitle($title) {
  $this -> title = $title;
}
[PHP] pobierz, plaintext 

Możesz to robić również w konstruktorze, zależy od potrzeb. Teraz obiekt $title ma nadaną wartość i można go użyć w innych metodach, np.

[PHP] pobierz, plaintext 
function addArticle() {
  $query = "INSERT INTO articles VALUES ('$this -> title')"; // przykładowe zapytanie
}
[PHP] pobierz, plaintext 

A jaki ma to sens? Żeby nie robić tego w ten sposób:

[PHP] pobierz, plaintext 
$oArticle->addArticle($title, $content);
[PHP] pobierz, plaintext 

Tylko w ten:

[PHP] pobierz, plaintext 
$article = new Article;
$article -> setTitle('jakis tytul');
$article -> addArticle();
[PHP] pobierz, plaintext 

To nie są żadne obiekty a właściwości klasy/obiektu. No nie mieszaj już i tak pokręconego kodu

Dziwne że można pisać we frameworku jak Kohnana, nie znając podstaw OOP

Takie są skutki korzystania z FW nie znając sposobu w jaki działają.

@in5ane - możesz napisać co ta klasa miałaby w zasadzie robić? Jest do dla mnie niezrozumiale.

Ten post edytował !*! 18.06.2013, 11:29:20

Po prostu zacząłem się uczyć OOP i chciałem coś sobie małego dla nauki tworzyć. Nie miałem pomysłu na klasę żadną. Możecie podsunąć jakiś pomysł i spróbuję na nowo napisać i może bardziej to pojmę, a Wy optymistyczniej to ocenicie. Od czegoś trzeba zacząć.

No nie wiem, może napisz klasę tworzącą miniaturki z obrazków? Nie będzie to wielce problematyczne, a edukacyjne.

Czy ta klasa jest już dobrą próbą programowania obiektowego? W dobrym kierunku idę?

[PHP] pobierz, plaintext 
<?php
	class Thumbnail
	{
		public $name;
		private $img;
		private $width;
		private $height;
		private $img_t;
		private $width_t = 150;
		private $height_t = 150;
 
		public function __construct($name)
		{
			$this->name = $name;
			$this->img = imagecreatefromjpeg($this->name);
			$this->width = imagesx($this->img);
			$this->height = imagesy($this->img);
		}
 
		public function createThumb()
		{
			$this->img_t = imagecreatetruecolor($this->width_t, $this->height_t);
			imagecopyresampled($this->img_t, $this->img, 0, 0, 0, 0, $this->width_t, $this->height_t, $this->width, $this->height);
			imagejpeg($this->img_t, 'thumb_'.$this->name, 100);
		}
 
		public function __destruct()
		{
			imagedestroy($this->img);
			imagedestroy($this->img_t);
		}
	}
 
	$oThumb = new Thumbnail('20120229028.jpg');
	$oThumb->createThumb();
?>
[PHP] pobierz, plaintext 

@edit:
Oczywiście, można by dodać jeszcze możliwość dodawania rozmiaru miniatury (opcjonalna rzecz). Ale ogólnie chcę wiedzieć, czy programowanie obiektowe (takie typowe) idzie w tym kierunku, jak ja napisałem tą klasę.

@edit2:
No i zrobiłem tylko dla JPG, oczywiście normalnie powinno się zczytać rozszerzenie (np. dla JPG, GIF i PNG).

Ten post edytował in5ane 18.06.2013, 19:43:09

człowiek/dziecko/rodzic/uczeń/student

pojazd/samochód/osobowy/dostawczy/wyścigówka/łódź/wiosłowa/motorowa

Teraz już lepiej Ci wyszło.

A mógłbyś powiedzieć, czy to jest poprawnie napisana klasa? W dobrym kierunku idę? Oczywiście nie mówię tutaj o funkcjonalności i użyteczności.

Jak najbardziej, wszystsko jest poprawnie i idziesz w dobrym kierunku. Jednak ten fragment:

[PHP] pobierz, plaintext 
$this->img = imagecreatefromjpeg($this->name);
$this->width = imagesx($this->img);
$this->height = imagesy($this->img);
[PHP] pobierz, plaintext 

mógłbyś umieścić w funkcji createThumb. Nie potrzebujesz aż tylu właściwości, na dobrą sprawę wystarczyłaby sama ścieżka do pliku, reszta mogłaby pozostać zmiennymi lokalnymi wewnątrz metody klasy. Ewentualnie mógłbyś dorzucić wysokość i szerokość miniaturki lub skalę pomniejszenia jako właściwości, reszta jest raczej zbędna.

[PHP] pobierz, plaintext 
$oThumb = new Thumbnail('20120229028.jpg');
$oThumb->createThumb();
[PHP] pobierz, plaintext 

Te "o" jest tu zbędne. Poczytaj o standardzie nazewnictwa PSR. Dorzuć też dokumentacje do tego kodu i będzie ok.

Po prostu stwierdziłem, że główny obrazek niech będzie w konstruktorze automatycznie wykonywany, a metoda createThumb żeby zajmowała się już tworzeniem tylko i wyłącznie miniatury. Tak czy siak dziękuję za odpowiedzi i rady. Wezmę się teraz za coś większego, albo rozbuduję tą klasę (tak dla nauki). Po stworzeniu poddam ją również ocenie.