[Kohana] czy tak ma wyglądać model? Opcje

[terabit]

Witam,

tak sobie grzebie na forum, szukam różnych ciekawych rzeczy o kohanaphp...
no i znalazłem coś takiego: http://phpfi.com/327959 w temacie http://forum.php.pl/index.php?s=&showt...st&p=487669
i się zastanawiam czy właśnie tak powinien wyglądać model? Tyle ifów i nawet jakies komunikaty:

[PHP] pobierz, plaintext 
<?php
$message = 'Witaj '.$display_name.'<br/><br/> Zarejestrowa?e? si? w serwisie '.config::item('config.custom_page_domain').'. Aby aktywowa? konto kliknij w poni?szy link:<br /><br />'.html::anchor('authentication/activate/'.$activation_key, 'Aktywuj Konto').'<br /><br/>'.url::site('authentication/activate/'.$activation_key).'<br /><br />Wiadomo?? wygenerowana automatycznie. Prosimy na ni? nie odpowiada?.';
?>
[PHP] pobierz, plaintext 

Myślałem że takimi rzeczami zajmują się metody controllera...

Czy tak powinno się to robić?

Ten post edytował terabit 11.08.2008, 21:50:41

[LBO]

wiec wedlug mnie pobinien byc w kontrolerze i przekazywany do widoku gdy bedzie taka potrz

Wiesz, boli mnie jak widzę też interpretację Widoku w MVC serwowana przez frameworki :/

[PHP] pobierz, plaintext 
<?php
// w akcji
if(($list = $model->getList()) !== null) {
	$this->view->set('list', $list);
} else {
	$this->view->set('error', 'Nie znalezione');
		$this->vie->setTemplate('Error');
}
?>
[PHP] pobierz, plaintext 

W sumie wtedy można powiedzieć, że robisz to w kontrolerze.
W Agavi natomiast widok to zupełnie inna sprawa, każdy rodzaj widoku to osobna klasa, ze swoją logiką i metodami.

[PHP] pobierz, plaintext 
<?php
// w akcji
try {
	if(($list = $model->getList()) !== null) {
		$this->setAttribute('list', $list); // przekazuje dane do widoku.
		return 'List'; // a widok juz sie zajmie tym i albo wypluje html, albo json, albo coś jeszcze inn
ego
	} else {
		return 'NotFound'; // nie znalezione -> tam komunikat
	}
} catch (Exception $e) {
		// model rzucił błędem
	   return 'Error'; // tam komunikat, że wystąpił błąd
}
?>
[PHP] pobierz, plaintext 

<?php
foreach($query as $row)
{
$this->session->set('isLogin', TRUE);
$this->session->set('id', $row->user_id);
$this->session->set('login', $row->user_name);
$this->session->set('email', $row->user_email);
$this->session->set('lastvisit', $row->user_last_login);
$this->session->set('role', $row->role_id);
$this->db->from('users');
$this->db->set(array('user_logins_count' => $row->user_logins_count+1, 'user_cookie_key' => $cookie_key, 'user_last_ip' => $this->input->ip_address(), 'user_last_login' => mktime()));
$this->db->where(array('user_id' => $data[0], 'user_cookie_key' => $data[1]));
$this->db->update();
?>[/php]
oczywiście troszkę inaczej... (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

To już robisz jak chcesz, teoretycznie model powinien zamknąć w sobie całość działania aplikacji, więc kod powyżej się nie kłoci z tym.
W agavi (jak i w Symfony) istnieje obiekt sessionUser, w którym w odpowiedniej metodzie inicjalizacyjnej uruchamiam model użytkownika, pobieram dane i ustawiam wartości zmiennych sesji. Ale jakby tego nie było, prawdopodobnie bym użył modelu.

Ten post edytował LBO 11.08.2008, 22:46:34