Klasa news - czy to można nazwać OOP? Opcje

[adriano321]

Jestem dopiero co zaczynającym "programistą" (haha) PHP. Programowanie proceduralne nie sprawia mi już większych kłopotów więc chcę wkroczyć na wyższy poziom. Niestety same podstawy tj. teoria nie wystarczy, żeby napisać coś samemu czyt. dobrze napisać. Więc tu się zwracam z prośbą do Was: Czy to coś poniżej ma jakiś związek z OOP? I czy z własnych doświadczeń możecie coś zaproponować, jakieś rozwiązania dalsze?

Klasa ObsługaNewsa z założenia ma być dostępna tylko z poziomu panelu administracyjnego.

Kolejne pytanie:
Jak rozwiązać problem wyświetlania kilku newsów na jednej stronie? Utworzyć kilka egzemplarzy klasy News? Tu zastanawiam się jak zrobić to, aby wykonać tylko jedno zapytanie do bazy danych?

Czy można się w jakiś sposób pozbyć dodawania $db w parametrach metod??

[PHP] pobierz, plaintext 
<?php
class News {
 
	public $id;
	public $tytul;
	public $tresc;
	public $autor;
	public $data;
	public $kategoria;
 
	function __construct() 
	{
	}	
 
}
 
class ObslugaNews extends News { 
 
	public function dodajNewsa ($db) 
	{
		$sql = "INSERT INTO `news` (`id` ,`title` ,`text` ,`date` ,`edit_date` ,`id_author` ,`id_kategory`) VALUES ('','$this->tytul','$this->tresc','$this->data','','$this->autor','$this->kategoria')";
 
		if (@!$db->query($sql))  echo " z tego!";
	}
 
	public function edytujNewsa ($id,$db) 
	{
		$sql="UPDATE `news` SET `title`=$this->tytul ,`text`=$this->tresc ,`edit_date`='aktualna_data' ,`id_kategory`=$this->kategoria WHERE `news`.`id`=$id LIMIT 1";
 
		if (@!$db->query($sql))  echo " z tego!";
	}
 
	public function pobierzNewsa ($id,$db) 
	{
		$sql="SELECT `id` ,`title` ,`text` ,`date` ,`edit_date` ,`id_author` ,`id_kategor
y` FROM `news` WHERE id='$id' LIMIT 1";
 
		if (@!$wynik=$db->query($sql))  echo " z tego!";
		$wynik = $wynik->fetch_array();
 
		return $wynik;
	}
 
	public function ustawNewsa ($tytul,$tresc,$kategoria) 
	{
		$this->tytul = strip_tags(ltrim(addslashes($tytul)));
		$this->tresc = strip_tags(addslashes(ltrim($tresc)));
		$this->kategoria = strip_tags(ltrim(addslashes($kategoria)));
	}
}
?>
[PHP] pobierz, plaintext 

Adriano.

[Whisller]

@adriano321
1. Nie rób echo w metodzie klasy (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Bo dajmy na to będziesz chciał komunikat o błędzy zapisać do pliku albo też wysłać mailem, co wtedy? Zwracaj go w postaci stringa albo też wyrzucaj wyjątek.
2. W metodzie getOne nie musisz wartości przypisywać to tablicy $args jeśli nazwy pól w bazie danych są takie same jak indeksy w tablicy argumentów przyjmowanej przez konstruktor klasy News.
Tzn. możesz sobie skrócić pracę pisząc
$r = mysql_fetch_assoc($sth);
return new News( $r );
3. Zamiast robić metodę getNewsList można by pokusić się o to aby klasa NewsManager dziedziczyła po jakiejś klasie - lub też otrzymywała obiekt klasy - w której określał byś jakie pola chcesz pobrać oraz kryteria do WHERE, tak mi się wydaje.
A chesz zostawić tą metodę tak jak jest to daj albo sprawdzanie czy zmienna $sort jest ustawiona na DESC albo ASC lub też daj wartość domyślną dla tego argumentu( public function getNewsList($sort='ASC') ).