[MySQL][PHP]Klasa wpisu. Opcje

[Sagnitor]

Witam!

Niedawno rozpocząłem pisanie sobie zbioru przydatnych klas i funkcji, które roboczo nazywam moim mini 'frameworkiem'. Trafiła tam już m. in. klasa Uploader'a, którą pomogliście mi dopracować.
Stwierdziłem, że przy wielu projektach przydatna stanie się klasa wpisów inaczej newsów. Nowości występują w większości stron internetowych.
Przechodząc do sedna sprawy:

Myśląc abstrakcyjnie news posiada swoje prywatne pola jakimi są tytuł, treść, data itp. Jednak brakowało mi możliwości zarządzania wpisami tak bardziej 'z góry'.
Wymyśliłem klasę zarządzającą wpisami CNewsManager. Cała konstrukcja wydaje mi się o tyle korzystna (także pod względem wydajności), że do pobrania newsów wystarczy jedno zapytanie do bazy danych. Przejdźmy do kodu.

Klasa CNewsManager:

[PHP] pobierz, plaintext 
class CNewsManager
{
	private $aNewsBuffer = array(); // Bufor newsów
 
	private $sTable;				// Nazwa tabelki
	private $pdo;					// Baza danych
 
	public function __construct($pdo, $sTable)
	{
		$this->sTable = $sTable;
		$this->pdo = $pdo;
	}
 
	/* Funkcja tworzą nowy wpis, zawartość rozszerzona pobierana jest opcjonalnie */
	public function createNews($title, $content, $exContent = NULL)
	{
		if($title !== NULL && $content !== NULL)
		{
			if($exContent !== NULL)
			{
				$oNew = new CNews($this->pdo, $this->sTable, NULL, $title, $content, NULL, $exContent);
				$oNew->upload();
				return $oNew;
			} else {
				$oNew = new CNews($this->pdo, $this->sTable, NULL, $title, $content, NULL);
				$oNew->upload();
				return $oNew;
			}
		} else return false;
	}
 
	/* Funkcja ładująca wpisy do buffera newsów. Jeżeli nie podano id począštkowego i końcowego,
	   zapytanie pobiera wszystkie rekordy z tabelki. ZAPYTANIE DO BAZY! */
	public function load($idStart = NULL, $idLimit = NULL)
	{
		if($idStart !== NULL && $idLimit !== NULL)
		{
			foreach($this->pdo->query('SELECT * FROM `'.$this->sTable.'` ORDER BY id DESC LIMIT '.$idStart.', '.$idLimit) as $row)
			{
				$this->aNewsBuffer[$row['id']] = $row;
			}
			return $this;
 
		} else {
 
			foreach($this->pdo->query('SELECT * FROM `'.$this->sTable.'`') as $row)
			{
				$this->aNewsBuffer[$row['id']] = $row;
			}
			return $this;
		}
	}
 
	/* Funkcja opróżniająca buffer newsów */
	public function clearNewsBuffer()
	{
		for($i=0, $j=count($this->aNewsBuffer); $i < $j; ++$i)
		{
			$this->aNewsBuffer[$i] = NULL;
		}
		return $this;
	}
 
	// Funkcja pobierająca obiekt newsa na podstawie podanego ID. Obiekt pobierany jest z bufora newsów.
	public function selectNews($nId)
	{
		if(isset($this->aNewsBuffer[$nId]))
		{
			$aNews = $this->aNewsBuffer[$nId];
			if($aNews['ex_tresc'] !== NULL)
			{
				$oNews = new CNews($this->pdo, $this->sTable, $aNews['id'] , $aNews['tytul'], $aNews['tresc'], $aNews['data'], $aNews['ex_tresc']);
				return $oNews;
			} else {
				$oNews = new CNews($this->pdo, $this->sTable, $aNews['id'] , $aNews['tytul'], $aNews['tresc'], $aNews['data']);
				return $oNews;
			}
		} else return false;
	}
 
	/* Wyrenderowanie wszystkich wpisów zawartych w bufferze */
	public function showAll()
	{
		for($i=1, $j=count($this->aNewsBuffer); $i<$j; ++$i)
		{
			if($this->aNewsBuffer[$i] !== NULL) 
			{
				if($this->aNewsBuffer[$i]['ex_tresc'] !== NULL)
				{
					$oNews = new CNews($this->pdo, $this->sTable, $this->aNewsBuffer[$i]['id'], $this->aNewsBuffer[$i]['tytul'], $this->aNewsBuffer[$i]['tresc'], $this->aNewsBuffer[$i]['data'], $this->aNewsBuffer[$i]['ex_tresc']);
					$oNews->show();
				} else {
					$oNews = new CNews($this->pdo, $this->sTable, $this->aNewsBuffer[$i]['id'], $this->aNewsBuffer[$i]['tytul'], $this->aNewsBuffer[$i]['tresc'], $this->aNewsBuffer[$i]['data']);
					$oNews->show();
				}
			}
		}
		return $this;
	}
 
}
[PHP] pobierz, plaintext 

Klasa CNews:

[PHP] pobierz, plaintext 
class CNews
{
	private $id;		// ID wpisu
	private $date;		// Data
	private $title;		// Tytuł
	private $content;	// Zawartość podstawowa
	private $exContent;	// Zawartość rozszerzona
	private $author;	// Autor wpisu
 
	private $table;
	private $datebase; 	// Baza danych (PHP Data Objects)
 
	/* Konstruktor klasy wpisu. OPTYMALIZACJA! */
	public function __construct($datebase, $sTable, $nId, $strTitle, $strContent, $date, $strExContent = NULL)
	{
		$this->datebase = $datebase;
		if ($strTitle !== NULL && $strContent !== NULL && $sTable !== NULL)
		{
			$nId === NULL ? $this->id = NULL : $this->id = $nId;
			$date === NULL ? $this->date = date('d.m.Y, H:i') : $this->date = $date;
			$this->title = $strTitle;
			$this->content = $strContent;
			$this->table = $sTable;
			$strExContent === NULL ? $this->exContent = NULL : $this->exContent = $strExContent;
			/* Jeszcze autor */
			return true;
		} else return false;
 
	}
 
	public function edit($strTitle, $strContent, $strExContent = NULL)
	{
		if($strTitle !== NULL && $strContent !== NULL)
		{
			$this->title = $strTitle;
			$this->content = $strContent;
			if($strExContent !== NULL)
				$this->exContent = $strExContent;
		} else return false;
	}
 
	/* Funkcja aktualizująca wpis w bazie danych. 
	   Zależnie od statusu news jest tworzony lub uaktualniany. 
	   ZAPYTANIE DO BAZY! 										*/
	public function upload()
	{
		if($this->title !== NULL && $this->content !== NULL && $this->table !== NULL)
		{
			if($this->id !== NULL)
			{
				$this->datebase->exec("UPDATE ".$this->table." SET tytul='".$this->title."', tresc='".$this->content."', ex_tresc='".$this->exContent."' WHERE id='".$this->id."'");
				return true;
			} else {
				$this->datebase->exec("INSERT INTO ".$this->table." VALUES(0,'".$this->title."','".$this->content."', '".$this->exContent."', '".$this->date."', '".$this->author."')");
				return true;
			}
		}  else return false;
	}
 
	public function delete()
	{
		if($this->id !== NULL)
			$this->datebase->exec("DELETE FROM ".$this->table." WHERE id='".$this->id."'");
	}
 
	/* Funkcja pobierająca tytuł */
	public function getTitle()
	{
		return ($this->title !== NULL ? $this->title : false);
	}
 
	/* Funkcja pobierająca zawartość podstawowąš */
	public function getContent()
	{
		return ($this->content !== NULL ? $this->content : false);
	}
 
	public function getExContent()
	{
		return ($this->exContent !== NULL ? $this->exContent : false);
	}
 
	public function show()
	{
			echo '<tr>
						<td><h2>'. $this->title .'</h2>
								<span>'. $this->date .'</span>
						</td>
				</tr>';
				if($this->exContent === NULL)
					echo '
					<tr>
							<td colspan="2"><div class="wiadomosc">'. $this->content .'</div></td>
					</tr>';
				else
					echo '<tr>
							<td colspan="2"><div class="wiadomosc">'. $this->content .'<div style=" float: right; margin-right: 35px; margin-top: 15px;"><a href="?home=home&amp;more='.$this->id.'">Czytaj więcej...</a></div></div></td>
						</tr>';
	}
 
	public function show_ex()
	{
			echo '<tr>
						<td><h2>'. $this->title .'</h2>
								<span>'. $this->date .'</span>
						</td>
				</tr>
				<tr>
						<td colspan="2"><div class="wiadomosc">'. $this->exContent .'</div></td>
				</tr>';
	}
}
[PHP] pobierz, plaintext 

Tutaj pojawia się moje pytanie: czy koniecznością jest przekazywanie takiej ilości parametrów do konstruktora CNews?
Chciałbym także poznać wasze opinie co do budowy i mojego sposobu myślenia w tym przypadku. Chętnie dowiem się jakie rozwiązania proponujecie oraz jakie porady macie dla mnie co do pisania kodu.

Teraz skrawek kodu przykładowego:

[PHP] pobierz, plaintext 
$pdo = new PDO('mysql:host=localhost;dbname=newsy', 'root', '');
 
$NewsManager = new CNewsManager($pdo, 'news');
$NewsManager->load(1,11);
$NewsManager->selectNews(5)->show();
[PHP] pobierz, plaintext 

Pozdrawiam

Jakieś propozycje?

PS. Nie wiem czy temat umieściłem w odpowiednim dziale, w razie problemów proszę moderatora o przeniesienie.

Już niedługo dodam także klasę komentarzy.

Pozdrawiam

[thek]

Powiem tak... Niemal większość rzeczy w Twoim managerze powinna być implementowana przez CNews (IMG:style_emoticons/default/wink.gif)
CreateNews - to tylko opakowanie tworzenia newsa, na dodatek extendedContent powinna sama klasa CNews wyłapać, a nie, że różne konstruktory wywołujesz. Konstruktor i tak sprawdza czy tam jest jakaś wartość, więc nawet jeśli wywołasz ją w klasie nadzorczej to i tak konstruktor będzie sprawdzał, czy ona jest. Bo tak konstruktor napisałeś.

Load - za ładowanie całej tabeli w przypadku braku początku i końca, to za jaja by Cię trzeba było powiesić (IMG:style_emoticons/default/wink.gif) W takiej sytuacji powinieneś ładować ostatnich X newsów. To co chcesz robić to tylko na desktopie, gdzie masz taki obiekt cały czas w pamięci, a nie, że tworzony za każdym razem od nowa z każdym żądaniem! To wstęp do paginacji. Inna sprawa, że powinieneś się tam zastanowić nad sensem przeciążenia metody load() lub stworzenie kilku wariantywnych:
loadSingle - ładuje jeden wybrany,
loadRange - ładuje zakres od-do po id lub kolejności w bazie danych - Twój wybór,
loadDefault - ładuje X ostatnich,
loadChosen - ładuje wybrane przez użytkownika.

ClearNewsBuffer - po co pętla? $this->$aNewsBuffer = array(); i pozamiatane

SelectNews - pisałem wyżej o load...

ShowAll - znowu extendedContent rozróżniane jak w CreateNews, grrrrr... Poza tym "dziedziczy" ona błąd klasy CNews... Mieszanie modelu z widokiem. Od kiedy model na pałę wartości wali w kod HTML? A to robi CNews->show()

Poza tym gratuluję pomysłu "chainingu"... SelectNews( nieistniejąceId )-> show();
Jak zrobisz show() na wartości false?

Ja już nawet nie ruszam tematu: "Chłopaki... Nie możemy użyć RDBMS. Musimy przejść na pliki XML jako formę przechowania danych." i jedno wielkie "K..wa mać! Framework tego nie przewiduje!"