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

Powiem tak... Niemal większość rzeczy w Twoim managerze powinna być implementowana przez CNews
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ć 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!"

Wielkie dzięki za rozjaśnienie kilku istotnych rzeczy.

Zgodzę się, że:
-CreateNews - fakt, zbytnio obudowałem to klasą nadzorującą, lecz czy po usunięciu wywołania dwóch konstruktorów takie obudowanie jest niewłaściwe?
- Load - kilka przeładowań tej funkcji. Wolę uniknąć wieszania za jaja :-),
- ClearNewsBuffer - no tak, jakieś przyzwyczajenie po destruktorze w C++.

Jeżeli chodzi o resztę mam kilka pytań. Powiedzmy, że załadowałem $NewsManager->loadDefault(x), x ostatnich wpisów i chcę operować na jednym (załadowanym już w bufforze, aby nie operować na zapytaniach). Myślałem właśnie nad utworzeniem funkcji pokroju $NewsManager->selectNews(), lecz w przypadku błędnego ID wszystko się sypie.

Spodziewałem się wytknięcia błędu jak to nazwałeś 'mieszania modelu z widokiem', lecz zupełnie nie mam pojęcia o co w tym chodzi. W sensie, że klasa nie powinna mieć metody 'renderującej'?

Jeszcze raz dzięki za w ogóle pofatygowanie się w czytaniu moich wypocin.

Pozdrawiam

EDIT:
Czyli funkcja loadCokolwiek, powinna zwracać obiekt CNews w przypadku jednego wyniku lub tablicę obiektów jeżeli rezultat jest większy od jednego?

Zależy jak spojrzeć z tym obudowaniem... Ja bym czegoś takiego jak dodawanie i edycja czy usunięcie pojedynczego elementu nawet nie dodawał do klasy nadzorującej. Nadzorca robi to hurtowo, a więc edytujesz, usuwasz i dodajesz elementy kolekcji od razu jako wielokrotności. Jeśli miałbym coś w takim nadzorcy dodać to jedynie odwołania do klasy podlegającej. W środowisku Web byłoby to odwołanie się do kontrolera danej klasy. Innymi słowy gdybym chciał dodać nowy obiekt, to nie z poziomu zarządcy bym go tworzył, ale zwrócił się bezpośrednio do samej klasy podrzędnej. O co mi chodzi? Skoro masz klasę CNews, to na bank ma ona swój formularz dodawania i edycji. I to do niego bym kierował, a nie jakiejś sztucznej w nadzorcy, która robi dokładnie to samo. Innymi słowy przekierowanie na metodę create klasy CNews i jej własny widok

Jeśli chcesz select, to czemu w tym momencie zamiast id nie użyjesz offsetu lub indeksu tablicowego? I tak używasz tablicy... Jaka z tego korzyść? Próba odwołania do nie istniejącego elementu mogła by być sygnalizowana wyjątkiem.

Klasa nie tyle nie powinna mieć metody renderującej co powinna mieć własny widok lub kilka widoków i to do nich się odwoływać. Sam powiedz... Jeśli zrobiłbym taki render jak Ty walnąłeś, to skąd miałbym pewność, że dopasuje mi się to do całej strony? Walnąłeś bowiem wszystko w tr->td a skąd mam mieć pewność, że ów widok trafi do strony, która mi to opakuje potem w tablicę?

Wszystkie funkcje load powinny zwracać tablicę. Choćby z jednym tylko elementem. No chyba, że loadSingle mógłby zwrócić obiekt klasy CNews i wtedy ładnie byś miał od razu z mostu dostęp do zmiennej this. Ale z racji zgodności z innymi wariantami obstawałbym za tablicą i tam odwołaniem do iteratora i/lub metody current(). No i w razie tablicy mógłbym przewijać się po elementach metodami prev(), next(), current()

Teraz zrozumiałem bezsensowność zapisywania w tablicy pod indeksem równym ID. Dzięki jeszcze raz za rady.
Niedługo może wrzucę tu zmodyfikowaną i poprawioną wersję tej pełnej nieporozumień klasy .

Na początku tworząc w ogóle obiektowy system wpisów, stworzyłem samą klasę CNews w troszkę innej formie. Brakowało mi właśnie takiego 'nadzorcy', który robiłby to hurtowo, jednak w trakcie pisania CNewsManager widocznie zapomniałem o jego przeznaczeniu.

Z metod load, będę zwracać tablice z danymi newsów, bo po co wywoływać X konstruktorów i tworzyć tablicę obiektów, skoro można powybierać.
Zastanawiam się jeszcze nad metodą loadChosen(). Sednem sprawy są parametry (nie wiadomo ile użytkownik ich przekaże). Pogrzebię u wujka Google na ten temat.

Tak, czy siak kliknąłem zasłużone oraz symboliczne "Pomógł".

Pozdrawiam
Sagni

EDIT:
Miałem to samo Na szczęście szybko przeszedłem nad tym do porządku dziennego. Inna sprawa, że robiąc tak jak Ty proponowałeś, doprowadziłbyś do powstania tablicy NULLi, co też nie jest prawidłowe.

Jeszcze odniosę się do propozycji:

klasa CNews:
metody:
konstruktor, add, update, delete i settery oraz gettery dla poszczególnych pól
Większość tego już masz, ale nie zawsze prawidłowo. To czego Ci brakuje to brak obiektu walidacji, kontroli uprawnień i warstwy abstrakcji na zapis. Kompletnie olałeś te aspekty. Operacja na danych powinna być przeniesiona do jakiegoś abstrakcyjnego bytu, który dopiero decydowałby czy operacje działają jako plikowe czy na bazie danych. Gdyby tak przyjąć, to operacje add i update magła by być jedną metodą save(), która obiekt klasy CNews przekazała by wyżej gdzieś do warstwy modelu. On by się już sam tym zajął. I to w owej warstwie abstrakcji byś decydował na czym tak naprawdę pracujesz. Ów obiekt abstrakcyjny bym wstrzykiwał do klas z danymi pracującymi. Zobacz może jakiś ORM by mniej więcej załapać o co w tym mniej więcej chodzi.
Przypuśćmy klasa Storage, która działa wedle wzorca Strategia nie byłaby tu głupim pomysłem. Używałbyś zawsze tego samego obiektu, ale implementacja konkretnego silnika bazodanowego ( nawet poprzez kolejna abstrakcję, jaką jest PDO ) lub operacji plikowych byłaby dla klienta niewidoczna.

Zależy co chcesz osiągnąć... Czasem tablica obiektów mogła by być lepsza. Czemu? Przypuśćmy takie wywołanie

[PHP] pobierz, plaintext 
$manager = new CNewsManager(); //domyślny konstruktor, tylko utworzenie obiektu i niczego innego
$manager->loadDefault(); //domyślne załadowanie jakąś ilością newsów
$manager->next()->next()->current()->getTitle(); //pobranie tytułu z 3 (bo dwukrotne użycie next) newsa. 
[PHP] pobierz, plaintext 

Zauważ ciekawą rzecz... Iteruję metodami next po tablicy obiektów i gdy chcę konkretny daję current, co zwraca mi wprost obiekt klasy CNews, a więc mogę użyć jego własnego gettera getTitle. Mogłem użyć też przypuśćmy zamiast dwóch next() własnej metody index(2), która w przypadku niepowodzenia sypnęła by mi wyjątkiem. Tak samo obwarowałbym wyjątkami ogólnie iterację po tablicy wyników.

Prostą przyczyną braku tych aspektów jest brak wiedzy, jak się za to zabrać i z czym to się je.
Nigdy nie projektowałem jakiś większych frameworków, także wykładam się w tym miejscu brakiem doświadczenia. Widzę, że projektowanie relacji pomiędzy klasami szybko może się przeistoczyć w coś większego.

Jak na razie po kilku zmianach (m. in. utworzeniu metody save(), zamiast add() i upload() ), wyciąganie danych wygląda w ten sposób:

[PHP] pobierz, plaintext 
$pdo = new PDO('mysql:host=localhost;dbname=newsy', 'root', '');
$pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
 
$NewsManager = new CNewsManager($pdo, 'news');
try
{
	$aNews = $NewsManager->loadDefault(10);
}
catch(Exception $e)
{
	echo 'Błąd menedżera wpisów!:'. $e->getMessage() .' w linii '. $e->getLine();
}
 
for($i=0, $j=count($aNews); $i<$j; ++$i)
{
	echo '<h1>'.$aNews[$i]->getTitle().'</h1>';
	echo '<h3>'.$aNews[$i]->getDate().'</h3>';
	echo '<p>'.$aNews[$i]->getContent().'</p>';
}
[PHP] pobierz, plaintext 

Zmieniłem także dodawanie parametrów w konstruktorze CNews (tytuł, zawartość itp) na opcjonalne, w razie tworzenia nowego newsa wykorzysta się $News->save($title, $content itp).

EDIT:
Masz może jakieś tutoriale lub książki traktujące o takiej warstwowości?

Pozdrawiam

Osobiście nie spotkałem jakichś podręczników do tego. To kwestia dużej ilości czytania, między innymi na temat wzorców, oraz grzebania wewnątrz kodu frameworków, czyli raczej coś, co dla początkujących osob jest mało przyjemne. Myślę, ze na początek powinieneś poczytac o samym wzorcu Strategia (a także innych podstawowych), by zrozumieć o co w nim chodzi. Wtedy złapiesz jak to mniej więcej można zaimplementować. Gdybyś to zaimplementował, to zapewne wyszedłbyś z połaczeniami do jakiegoś pliku konfiguracyjnego, gdzie byś raz a porządnie określał wszystko o formie (file, mysql, pdo czy xml) źródła danych i potem FW sam by się tego trzymał.

Co do uprawnień to poczytaj choćby tutaj wpisy i artykuły o ACL oraz rozwiń to. A walidacja... Nieco napomknięto w artykule o refaktoryzacji na wortalu. Ogólnie jednak lepiej podejrzeć jak to robią większe FW. No cóż... Niestety w programowaniu wiele rzeczy nie jest ładnie opisanych i zrozumiale. Najwięcej człowiek uczy się przegladając kod dużych projektów, a to nie jest dla wielu ani łatwe, ani przyjemne. Można poczytać książki, można znaleźć trochę tutoriali, ale one tylko najprostsze rzeczy przedstawią i może ciut ostrzegą przed wpadkami poważnymi, ale porządnej implementacji tam raczej nie uświadczysz.

Czytając teraz troszkę o Strategii, zrozumiałem, że gdybym zrobił Interface INews o metodzie save();, to potem tworząc dwie klasy implementujące ten interface, w różny sposób wykonywana by była ta czynność. Z tego co zrozumiałem sugerujesz dodać klasę/warstwę abstrakcji zapisu, która wczytywałaby z pliku config, w jaki sposób dokonywać zapisu wpisów oraz innych czynności. Dobrze rozumuję?

No widzisz... Czytanie się opłaca Co do konfiga, to myślałem o tym, by podczas żądań strona sobie z konfiga czytała czego używa i wybierała odpowiednią implementację obsługi źródła danych. Dla całości serwisu miałbyś bowiem dostępne metody zdefiniowane w interfejsie i tylko ich byś używał. To JAK by to było od strony silnika rozwiązane byłoby już problemem samej klasy. Dla Twojego kodu istniały by tylko wywołanie obiektu klasy Storage i jej metod. To jednak jaka klasa by była w ramach Storage odpowiedzialna za reakcję w danej chwili, działoby się "pod maską". Mógłbyś też poczytać o wzorcu Factory (metoda wytwórcza), Abstract Factory i Builder, bo je też można wykorzystać i nawet sądzę, że któraś z nich będzie lepsza w Twoim wypadku. Abstract Factory daje Ci transparentność kosztem jednak skomplikowania (masz jeden interfejs od strony klienta, ale w środku możesz mieć nawalone od groma zależności ), Factory daje większą swobodę ( duża rozszerzalność i niezależność ), a Builder skupia sie bardziej na utworzeniu obiektu, ale dzięki temu mogą to być bardzo skomplikowane i zróżnicowane twory, a sam wzorzec łatwo się skaluje, ponieważ istnieje w nim pewien nadzorca kierujący całym procesem tworzenia. Dzięki temu całość jest super przenośna, nawet na środowiska wieloprocesorowe czy klastry. Poczytaj trochę o wszystkich i oceń co u Ciebie najbardziej się będzie liczyć oraz wybierz właściwe rozwiązanie.