MVC Opcje

[Kuziu]

Witam,

znów mam problem z zaplanowaniem przepływu wszystkich danych w aplikacji. Wedle poprzednich podpowiedzi, które uzyskałem na forum zrobiłem tak (na przykładzie dodawania newsa):

NewsModel.php

[PHP] pobierz, plaintext 
public function addChild($title, $content)
{
	$child = new News();
	$child->setName($name);
	$child->setContent($content);
	$child->save();
}
[PHP] pobierz, plaintext 

NewsController.php

[PHP] pobierz, plaintext 
public function addNews()
{
	try{
		$this->newsModel->addChild($this->input->getPostData('title'), $this->input->getPostData('content'));
		$this->redirect('index.php/news');
	}catch (Exception $e){
		echo 'Nie moglem dodac newsa';
	}
}
[PHP] pobierz, plaintext 

No i niby wszystko dziala ladnie, mam kontroler ktory przekazuje dane do modelu, model te dane zapisuje i wszystko jest good.

Ale teraz gdy planuję dodać nową kolumnę w bazie danych, muszę modyfikować Model, wszystkie kontrolery które uzyły danego Modelu. Dodatkowo jeśli gdzieś nie zmienię kontrolera to np. content zapisze mi sie jako title, bo zmieni sie kolejnosc zmiennych wchodzących do funkcji i wszystko sie pokrzaczy.

Dobrze to robię, twrząc dla każdego zadania nową funkcję w modelu i przekazując do niej dane jako parametry funkcji?

I też pytanie czy to nie jest bez sensu takie duplikowanie wielu funkcji które dostarcza mi ORM Propel. Np. żeby pobrać newsa o danym id, napisąłem funkcję w modelu:

[PHP] pobierz, plaintext 
/**
 * @return News - instancja klasy News
 */
public function getChildById($id)
{	
	$child = NewsQuery::create()->findOneById($id);
 
	if(!$child)
		throw new Exception('Nie ma takiego newsa.', 666);
 
	return $child;
}
[PHP] pobierz, plaintext 

Ale przecież równie dobrze mogłem od razu w kontrolerze pobrać newsa za pomocą funkcji dostarczonych przez ORM:

[PHP] pobierz, plaintext 
$news = NewsQuery::create()->findOneById($this->input->getParam('id'));
[PHP] pobierz, plaintext 

Jakie mam korzyści z przerzucania zadań do modelu. Nie dodaję sobie tym zbędnej pracy, pisząc masę metod które w sumie już posiadam?

Ten post edytował Kuziu 14.02.2011, 12:20:02

[darko]

Ciężko tak jednoznacznie nie znając specyfiki projektu odpisać Ci że robisz dobrze lub źle. Generalnie model powinien być jak najbardziej samodzielnym "klockiem", który pewne powtarzalne czynności dziedziczy ze wspólnej klasy bazowej. Odwzorowanie bytów (klas) powinno odzwierciedlać w idealnym przypadku relacje rzeczywiste, jakie zachodzą pomiędzy prawdziwymi "obiektami" w rzeczywistości. Myślę, że najlepiej dla Ciebie w tej sytuacji byłoby gdybyś poczytał o programowaniu kontraktowym i używaniu interfejsów tak często lekceważonych. Na etapie projektowym jest to bardzo pożyteczna metodyka programowania. Pamiętaj przy tym o zasadzie programowania pod interfejs, nigdy pod implementację. Następnie jeśli chcesz pisać dobre API poczytaj o TDD i wykorzystaniu PHPUnit. Piszę to po to, byś wiedział, że są pewne narzędzia, które w znacznym stopniu ułatwiają zarówno modelowanie obiektowe jak też uczą dobrych nawyków i pozwalają na tworzenie wydajnych, i "czytelnie napisanych" aplikacji, którymi w późniejszym czasie jest łatwiej zarządzać, a przez to obniża się późniejsze koszty konserwacji kodu, co w praktyce jest bardzo ważne.