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.

[Cysiaczek]

Nie zniechęcaj się, ale to jest kod strukturalny ujęty w klasy (pseudo oop). Nie jest to nic złego, ale aby nie psuć pojęcia OOP, to nazywajmy to poop (IMG:http://forum.php.pl/style_emoticons/default/snitch.gif)

Błędy, które od razu rzucają się w oczy.

1. Nazewnictwo klas i metod. Pisz angielskie wersje
2. W tym wypadku News jest jedną tabelą, więc dobrze, aby był tak traktowany (to sie nazywa mapowanie relacyjno obiektowe - tu w podstawowej wersji na jednej tabeli). Częśc metod, jak np. obsługaNews:: pobierzNewsa($id) może być w samej klasie News. Ni eoznacz to bynajmniej, że ta klasa obsługaNews jest zbędna.

[PHP] pobierz, plaintext 
<?php
class News
{
	public $id;
	public $tytul;
	public $tresc;
	public $autor;
	public $data;
	public $kategoria;
 
	function __construct($data)
	{
		// Wykorzystamy to, że php ma dynamiczne typowanie zmiennych
		if(is_integer($data))
		{
			$sql="SELECT * FROM news WHERE id='$id' LIMIT 1";
			$result=$db->query($sql);
			$current=$result->fetchRowAsArray();
		}
		elseif(is_array($data))
		{
			$current=& $data;
		}
 
		//Tu już Twoja wyorraźnia niech zadziała, bo da się to zrobić ładniej, ale ja nie 
chce zaciemniać kodu
		$this->id=$current['id'];
		//itd.
	}
 
	public function save()
	{
		// zapisa obiektu do bazy danych
	}
 
}
 
class NewsBase
{
	private $collection;
 
	public function __construct()
	{
		$this->collection=new NewsCollection(); // Obiekt kolekcji
	}
 
	public function loadSomeNews($howMany, $order)
	{
		$stack=array();
		$sql="SELECT * FROM news WHERE ORDER BY $order LIMIT $howMany";
		$result=$db->query($sql);
		foreach($result->fetch_array() as $row)
		{
			$obj=new News($row);
			$stack[]=$obj;
			$this->collection->add($obj);
		}
		return $stack;
	}
 
	public function findById($id)
	{
		if($this->collection->has($id))
		{
			return $this->collection->get($id);
		}
		else
		{
			$obj=new News($id);		
			$this->collection->add($obj);
			return $obj;
		}
	}
 
	function saveCollection()
	{
		foreach($this->collection as $obj)
		{
			$obj->save();
		}
	}
}
?>
[PHP] pobierz, plaintext 

Pisane z palca, bez sprawdzenia, traktuj to jako pseudokod (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Pozdrawiam.

p.s To nie jest jedyna metoda, anie też nie zawsze najlepsze. Jest za to szybka w implementacji i daje dość duże pole manewru.