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]

I czy powinienem przekazywać zawsze $user, a w kontrolerze pobierać aktualnie zalogowanego, czy Model sam powinien pobierać aktualnie zalogowanego user'a i domyslnie do niego przypisywać dane.

Model powinien jawnie otrzymać referencję do użytkownika jeżeli ta potrzebna jest mu do pracy. Generalnie to taka złota zasada OOP: nigdy nie wytwarzaj "z dupy" zależności. Innymi słowy - obiekt nie powinien korzystać z jakiś globalnych danych (chyba, że jest to naprawdę nieuniknione).

Dodatkowo powinieneś bardziej uważnie projektować publiczny interfejs Twoich obiektów.
UserModel::addFavouritePhoto($user, $photo); - Pomijając fakt błędnego użycia metody statycznej (powinieneś operować na obiekcie tutaj, nie na klasie) dodanie ulubionego zdjęcia raczej powinno odbywać się na poziomie obiektu użytkownika:

[PHP] pobierz, plaintext 
$user = ...;
$photo = ...;
 
$user->addFavouritePhoto($photo);
 
$userManager->persistUser($user); // utrwala dane np. poprzez zapisanie ich w bazie danych
[PHP] pobierz, plaintext