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

[Kuziu]

Ale jeśli zrobię tak jak piszesz:

[PHP] pobierz, plaintext 
$news = $newsModel->createEntity(); // W najprostszym przypadku ograniczy się to do zwrócenia nowego, czystego obiektu
$news->setTitle($this->input->getPostData('title'));
[PHP] pobierz, plaintext 

to otrzymuję coś identycznego jak używając samego propela:

[PHP] pobierz, plaintext 
$news = new News();
$news->setTitle($this->input->getPostData('title'));
[PHP] pobierz, plaintext 

A co wtedy z walidacją przy zapisie itp. przecież w takim podejsciu przekazuje wpisywanie danych do Kontrolera.

No chyba ze metoda ->save(); zwracała by wyjątek przed zapisem, a wcześniej obiekt by się nie przejmował czy dane są poprawne czy nie, tylko przy zapisie miałbym wyjątek. Dobrze kombinuję?

Co do addChild() to za dużo we flashu siedze i z przywyczajenia palnąłem (IMG:style_emoticons/default/biggrin.gif)

A jeszcze jedno. W przypadku np. błędnych danych Model zwraca wyjątek do kontrolera że nie mógł np. stworzyć nowego użytkownika bo taki o podanym loginie już istnieje. A czy uprawnienia Admina itp. również powinien sprawdzać model? Wtedy musiałby wiedzieć w jakim trybie jest. Ewentualnie czy funkcja dodaj użytkownika a dodaj użytkownika z poziomu Admina powinna być drugą kopią, ewentualnie osobną metodą z wywołaniem tej poprzedniej?

Ten post edytował Kuziu 14.02.2011, 19:07:27

[rzymek01]

No chyba ze metoda ->save(); zwracała by wyjątek przed zapisem, a wcześniej obiekt by się nie przejmował czy dane są poprawne czy nie, tylko przy zapisie miałbym wyjątek. Dobrze kombinuję?

osobiście stosuję coś takiego, że klasa typu News ma swój stan poprawna/niepoprawna, stan klasy jest uaktualniany przy każdej zmianie pól obiektu, więc w zasadzie sprawdzenie poprawności danych odbywa się w klasie News, natomiast to klasa NewsModel sprawdza poprawność klasy News (np. przy edycji/dodawaniu) i wówczas w przypadku niepoprawności klasy News NewsModel rzuca wyjątkiem, który jest łapany przez kontroler

edit:
uprawnienia sprawdzane są przez kontroler (lub klasę do tego przeznaczoną, ale wywoływaną z poziomu kontrolera), gdyż badanie uprawnień nie nalezy w ogóle do zadań modelu news

Ten post edytował rzymek01 16.02.2011, 12:32:19