[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]

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.

[b4rt3kk]

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.