Jakie metody w klasie News - jak ma to współgrać? Opcje

[Joachim Peters]

Witam,

jestem po lekturze naprawdę wielu tematów na tym forum, w okręg moich zainteresowań weszła także książka "PHP5 Obiekty, wzorce, narzędzia".
Mimo to mam problem co do ułożenia odpowiedniej klasy. Moje problemy:

1. Czy w klasie np. do obsługi newsów powinna znajdować się metoda do wyświetlania konkretnego newsa/newsów jeżeli nie stosuje ścisłego MVC (chce napisać stronę zintegrowaną z innym skryptem od którego będę używał kilka klas i funkcji, m.in. do wyświetlania w przeglądarce html)? Jeżeli nie to powinienem zrobić dla niej osobną funkcję (bez klasy)?

2. Idąc dalej przykładem klasy do obsługi newsów chciałbym zapytać czy lepiej definiować ID newsa już w konstruktorze czy ustalać go manualnie przez metodę np. setId()?

3. Czy opłaca się stworzyć osobną metodę do dodawania elementów do tablicy z danymi newsa (np. addParam($key, $value)(IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) , czy wyjdzie na to samo (w sensie obiektowym) przekazując pełną tablicę prosto do metody np. update()?

Poniżej zarys klasy:

[PHP] pobierz, plaintext 
<?php
class News {
	private $newsId	 = 0;
	private $params		= array();
 
	public function setId($id) { } // Ustawia id newsa na którym chcemy operować
	public function submit() { } // Jeżeli newsId > 0 to aktualizuje, jeśli odwrotnie to tworzy nowy rekord
	private function insert() { }
	private function update() { }
	public function delete() { }
	public function addParam($key, $value) { } // Dodaje parametry typu nazwa newsa, autor, treść, źródło do tablicy danych
	public function print() { } // Wyświetla newsa, jeżeli ustaliliśmy Id do wyświetla wybrany news, jeżeli nie to
 wszystkie
	private function printSpecific() { } 
	private function printAll($perPage) { }
}
?>
[PHP] pobierz, plaintext 

Jest tu sens OOP? Co zmienić aby było lepiej?

Pozdrawiam i z góry serdecznie dziękuje za wyjaśnienia!

[Cysiaczek]

Registry jest o tyle brzydki że trzeba wiedzieć co w nim jest.

@sedziwoj - Tak? To patrz (IMG:http://forum.php.pl/style_emoticons/default/aaevil.gif)

Wybaczcie niedoróbki, ale to tylko mała prezentacja oparta o proste tablice.

[PHP] pobierz, plaintext 
<?php
interface Basic{}
interface Advanced{}
 
class Config{}
class fwConfig extends Config implements Basic{}
class appConfig extends Config implements Basic{}
class moduleConfig extends Config implements Advanced{}
 
 
class Registry
{
	static public $interfaces=array('Basic', 'Advanced'); // najlepiej zautomotyzować
	static public $collection=array();
 
 
	static public function add($obj)
	{
		foreach(class_implements($obj) as $iface)
		{
			if(in_array($iface, self::$interfaces))
			{
				self::$collection[$iface][get_class($obj)]=$obj;
			}
		}
	}
 
	static public function getFor($obj)
	{
		$data=array();
		foreach(class_implements($obj) as $iface)
		{
			if(in_array($iface, self::$interfaces) && is_array(self::$collection[$iface]))
			{
				$data=array_merge($data, self::$collection[$iface]);
			}
		}
		return $data;
	}
 
	public static function getBasicCollection()
	{
		return self::$collection['Basic'];
	}
 
	public static function getAdvancedCollection()
	{
		return self::$collection['Advanced'];
	}
 
	public static function getAll()
	{
		return self::$collection;
	}
}
 
class News implements Basic,Advanced
{
	public function __construct()
	{
		extract(Registry::getFor($this));
		print_r($fwConfig);
	}
}
 
$f=new fwConfig();
$a=new appConfig();
$m=new moduleConfig();
 
print '<pre>';
Registry::add($f);
Registry::add($a);
Registry::add($m);
$obj=new News();
print '</pre>';
?>
[PHP] pobierz, plaintext 

@BlackBerry - Proszę, Twoje globale (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Pozdrawiam

[Black-Berry]

<?php
 (..)
?>

@.@ śliczne... Nie będę może polemizował bo jedyne co mi się nasuwa to że to cholernie skąplikowane a jedno słówko kluczowe global robi to samo. Ale masz 3,5k postów i śliczną konstrukcję rejestru więc pewnie się na tym znasz lepiej. Jak sobie przemyślę głębiej to pogadamy (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)

Ten post edytował Black-Berry 2.09.2008, 12:18:25