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]

@cysiaczek
Ja osobiscie nie wstawiał bym pobierania danych newsa w klasie reprezentujacej jego obiekt (News), bo i po co ma ona wiedziec jak i skad pobierac te dane?

Pozwolicie ze przedstawie swoja idee (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
<?
 
/**
 * Klasa reprezentujaca pojedynczy news
 *
 */
 
class News
{
	/**
	 * W konstruktorze mozemy ustawic dane obiektu. Dla newsa to np. title, body, vi
sibility etc.
	  *
	 * @param array $args
	 */
	public function __construct(array $args)
	{
		isset($args['id'])		   ? $this->setId($args['id'])				   : '';
		isset($args['title'])	  ? $this->setTitle($args['title'])			: '';
		isset($args['body'])	   ? $this->setBody($args['body'])			  : '';
		isset($args['visibility']) ? $this->setVisibility($args['visibility']) : $this->setVisibility(false);
	}
 
	/**
	 * @param int $id
	 */
	public function setId($id)
	{
		$this->id = (int)$id;
	}
 
	/**
	 * @param string $title
	 */
	public function setTitle($title)
	{
		$this->title = $title;
	}
 
	/**
	 * @param string $body
	 */
	public function setBody($body)
	{
		$this->body = $body;
	}
 
	/**
	 * @param mixed $visibility
	 */
	public function setVisibility($visibility)
	{
		$this->visibility = (int)$visibility;
	}
 
	private $id			= null;
	private $title	  = null;
	private $body	   = null;
	private $visibility = null;
}
 
/**
 * Klasa reprezentujaca managera newsow
 *
 */
class NewsManager
{
	/**
	 * W zaleznosci do tego czy przekazany zostaje ID newsa wykonujemy jego edycje l
ub tez dodajemy nowy
	 *
	 * @param object $news Obiekt reprezentujacy pojedynczy news
	 */
	public function save(News $news)
	{
		if ( $news->getId() != null ) {
			$this->edit($news);
		} else {
			$this->add($news);
		}
	}
 
	private function edit(News $news)
	{
		// Tutaj edycja
	}
 
	private function add(News $news)
	{
		// Tutaj dodanie nowego
	}
 
	private function delete($id)
	{
		// Usuniecie
	}
 
	/**
	 * @param int $id ID newsa ktory chcemy pobrac
	 * @return object Obiekt klasy News
	 */
	public function getOne($id)
	{
		// Tutaj wyciagamy informacje na temat newsa, z pliku, bazy etc.
		// ...
 
		$args = array();
		$args['id']		 = $result['id'];
		$args['title']	  = $result['title'];
		$args['body']	   = $result['body'];
		$args['visibility'] = $result['visibility'];
 
		return new News($args);
	}
 
 
	/**
	 * @return array Tablica obiektow klasy News
	 */
	public function getMany()
	{
		$result = array();
 
		foreach( $news AS $oneNews ) {
			$args['id']		 = $result['id'];
			$args['title']	  = $result['title'];
			$args['body']	   = $result['body'];
			$args['visibility'] = $result['visibility'];
 
			$result[] =  new News($args);
		}
 
		return $result;
	}
}
 
 
// Wyciagniecie pojedynczego newsa 
$newsManager = new NewsManager();
$oneNews = $newsManager->getOne(4);
// Teraz odwolujac sie do danych newsa robimy to przy uzyciu $oneNews->getTitle(), ->getBody(), ...
 
 
// Wyciagniecie kilku newsow
$newsmanager = new NewsManager();
foreach( $newsManager->getMany() AS $oneNews ) {
	// Tutaj takze mozemy sie odwolywac jak do listingu powyzej tzn. $oneNews->getTitle(), ->getBody()... i bedziemy mieli dane newsa na ktorym w danej interacji operujemy.
}
?>
[PHP] pobierz, plaintext 

To chyba tyle, wydaje mi sie ze nie jest to najgorsze rozwiazanie z kilku wzgledow.
1. Mozemy w latwy sposob robic kopie obiektow - przy np. porownywaniu obiektow przed i po edycji
2. Nie wazne skad pobierzemy dane w NewsManager'rze bo i tak operujemy na obiekcie
3. Nie jestesmy uzaleznieny od nazewnictwa w bazie danych poniewaz stosujemy getery wiec gdy nam sie zmienia nazwa w bazie np. z title na title_a ( a uwierzcie mi takie rzeczy sie zdarzaja (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) ) zmieniamy to tylko w managerze