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]

Thx Crozin.

Znów jest jaśniej (IMG:style_emoticons/default/smile.gif) Rozumiem, że np. mój model nie musi posiadać metody:

[PHP] pobierz, plaintext 
createUser($login, $password, $costam, $costam)
[PHP] pobierz, plaintext 

Tylko prostą:

[PHP] pobierz, plaintext 
createUser()
[PHP] pobierz, plaintext 

oddającą nową instancję User'a a kontroler może jego dane uzupełnić:

[PHP] pobierz, plaintext 
$user = $this->userModel->createUser();
$user->setLogin('dane pobrane z jakiejs klasy odpowiadajacej za GET/POST/itp.');
$user->setPassword('j/w');
[PHP] pobierz, plaintext 

A dopiero potem powinienem zapis przekazać znów do modelu, zamiast wykorzystać funkcję save() propela.

[PHP] pobierz, plaintext 
$this->userModel->persistUser($user);
[PHP] pobierz, plaintext 

I ta metoda może zwrócić wyjątek gdy np. dane będą błędne. Tak?


Zaczynam też rozumieć o co Ci chodzi z minusami propela i to mnie wkurza bo się do niego już strasznie przywyczaiłem i na prawdę dobrze mi się nim operuje.

-------------------

Co do singletonów, to zazwyczaj w konstruktorze pobieram sobie instancję za pomocą:

[PHP] pobierz, plaintext 
$this->userModel = UserModel::getInstance()
[PHP] pobierz, plaintext 

Ale pomyślę o tym co napisałeś i spróbuję korzystać z singletonów tak jak mówileś, tylko w racjonalnych przypadkach.