[PHP] Podstawy OOP, konstruktywne opinie mile widziane oraz nakierowanie na dobry trop Opcje

[in5ane]

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 

[in5ane]

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