Forum PHP.pl

Postanowiłem wziąć się na poważnie za programowanie obiektowe i stworzyłem swoją pierwszą klasę. Przeczytałem chyba wszystkie artykułu na temat programowania obiektowego, ale nadal czuję, że to nie jest jeszcze to.

Chciałem napisać klasę - system artykułów, zacząłem tak:

[PHP] pobierz, plaintext 
<?php
 
	// Article Class
 
	class Article {
 
		public function connectDB($host, $user, $pass, $database) {
 
			mysql_connect($host, $user, $pass) or DIE(mysql_error());
			mysql_select_db($database) or DIE(mysql_error());
			mysql_query("SET NAMES UTF8") or DIE(mysql_error());
 
		}
 
		public function newArticle($title, $text, $author) {
 
			mysql_query("INSERT INTO `articles` VALUES (null, '$title', '$text', '$author', now())");
 
		}
 
		public function editArticle($id, $title, $text, $author) {
 
			mysql_query("UPDATE `articles` SET `title` = '$title', `text` = '$text', `author` = '$author' WHERE `id` = '$id'");
 
		}
 
		public function deleteArticle($id) {
 
			mysql_query("DELETE FROM `articles` WHERE `id` = '$id'");
 
		}
 
	}
 
?>
[PHP] pobierz, plaintext 

A tak sobie tego używam:

[PHP] pobierz, plaintext 
<?php
 
	require_once('article.class.php');
 
	$article = new Article;
	$article->connectDB("localhost", "root", "", "article");
	$artile->newArticle("Tytuł artykułu", "Treść", "Autor");
 
?>
[PHP] pobierz, plaintext 

Chciałbym się dowiedzieć co o tym sądzicie, czy dobrze zacząłem to pisać czy nie. Aha i możecie podpowiedzieć co w takiej klasie powinno się znajdować, a co nie. Będę wdzięczny za wszystkie podpowiedzi

Ten post edytował emajl22 8.03.2011, 12:36:30

Moim zdaniem źle do tego podszedłeś. Pomyśl logicznie: dlaczego obiekt "artykuł" ma mieć możliwość np. usuwania się? W klasie Article trzymałbym informacje o artykule (czyli np. tytuł, treść). Jako metody stworzyłbym takie metody, które operują bezpośrednio na obiekcie (np. get_short_content()). Mechanizm dodawania/usuwania/edycji artykułów przeniósłbym do klasy modelu, np. Article_Model. Dla przykładu: metoda Article_Model::get_by_id($id) pobierałaby z bazy danych dane artykułu na podstawie jego id i zwracała je w postaci obiektu klasy Article. Wtedy miałoby to ręce i nogi

Poza tym zadbaj o bezpieczeństwo. Przykładowo: jeśli do metody usuwającej artykuł jako parametr przekażemy 1' OR 'x' = 'x to polecą nam wszystkie rekordy z tabeli. Ja osobiście w przypadku przekazywania identyfikatorów stosuję po prostu rzutowanie na liczbę całkowitą ((int)$id).

Przecież nie usuwam obiektu, tylko artykuł z bazy danych. Chyba, że źle zrozumiałem? :/


Wiem o tym, ale to robię przed wrzuceniem tego do metody, np:

[PHP] pobierz, plaintext 
$id = intval($_GET['id']);
$article->deleteArticle($id);
[PHP] pobierz, plaintext 

Ten post edytował emajl22 8.03.2011, 13:01:15

Pytanie: każdy obiekt będzie wywoływał nowe połączenie z bazą? I tak swoją drogą... masz obiekt, który nie robi niczego innego jak zwykłe operacje na bazie danych. Phpion dobrze zasugerował Ci oddzielenie logiki biznesowej. Może warto pomyśleć o akcesorach?

Nie mówię, że nie macie racji, ale do końca nie wiem o o chodzi z tym modelem :/ Może ktoś jaśniej na ten temat?

i to prawda, że w klasach nie powinno się nic echo'wać?

Model - logika biznesowa, czyli operacje na danych. I tak swoja drogą pozwolę sobie zalinkować do Zyxa, jest tam nieźle opisane MVC .

co ma piernik do wiatraka, a wlasciwie klasa do MVC?

Zechciej czytać cały temat.