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

[Crozin]

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

I co w związku z tym? (IMG:style_emoticons/default/smile.gif) Musisz liczyć się z tym, że modyfikacja struktury obiektu (np. przez dodanie nowej właściwości) wiąże się z reguły z koniecznością jakiś tam modyfikacji kodu. Oczywiście zawsze możesz proces tworzenia obiektu nieco zautomatyzować, taki prosty przykład:

[PHP] pobierz, plaintext 
$news = $newsBuilder->fromArray($this->input->getPostData('news')); // POST['news'] to tablica
[PHP] pobierz, plaintext 

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

W przypadku stron internetowych właściwie nie da się tego rozwiązać inaczej. Taka specyfika środowiska webowego.

Sama walidacja jest dosyć sporym problemem... Na początku musisz zadecydować czy encja (tutaj będą to m.in. obiekty News czy Article) może posiadać błędny stan wewnętrzny czy nie. Innymi słowy czy da się zrobić coś takiego:

[PHP] pobierz, plaintext 
$news = new News();
$news->setTitle('...');
[PHP] pobierz, plaintext 

Czy metoda News::setTitle() powinna wywalić wyjątek InvalidArgumentException z komunikatem "podaj normalny tytuł aktualności, a nie jakieś śmieci".
Raczej na pewno będziesz musiał zrezygnować z takiego podejścia, ponieważ walidacja danych jest procesem zbyt skomplikowanym by względnie prosty obiekt News był w stanie sobie z nim poradzić. Prawdopodobnie skończysz z czymś w stylu:

[PHP] pobierz, plaintext 
$news = ...;
$newsManager->persistNews($news); // Metoda newsPersist przed jakimkolwiek zapisem wywołuje walidator i sprawa czy ten aby przypadkiem nie zwrócił jakiś błędów
[PHP] pobierz, plaintext 

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?

IMO raczej nie powinieneś po prostu dopuścić do możliwości wywołania metody NewsManager::persistNews() przez użytkownika bez odpowiednich uprawnień.