MVC Opcje

[Damiankossek]

Witam,
Mam pytanie ponieważ nie wiem czy dobrze robię, przykładowo mam w modelu metodę która sprawdza czy wszystkie zależności są poprawne, jeżeli nie to wyskakuje błąd i przykładowo

[PHP] pobierz, plaintext 
 
if($extensionCfg['ver_req'] > $systemCfg['version'])
	{
		$array['error'][] = 'Zbyt niska wersja Twojego oprogramowania. Wymagana '.$config['ver_req'];
		$array['status'] = FALSE;
	}
 
if(!model_system_extensions::extensionExists($config['req_ext']));
        {
                $array['error'][] = 'Brak wymaganych zależności';
                $array['status'] = FALSE;
        }
 
[PHP] pobierz, plaintext 

Czy dopuszczalnym jest w modelu pisanie tekstu ? Macie jakieś lepsze metody na takie sprawy jak zwracanie wielu błędów ? (Kohana Framework)

[erix]

No wyjątku tu nie zrobisz, bo tylko jeden błąd wtedy zwróci.

Czy dopuszczalnym jest w modelu pisanie tekstu ?

A dlaczego nie?

Macie jakieś lepsze metody na takie sprawy jak zwracanie wielu błędów ?

Chyba prościej i sensowniej się nie da. Ewentualnie jakąś sensowniejszą nomenklaturę, ale nie wiem, jak masz ją określoną w projekcie. (IMG:style_emoticons/default/wink.gif)

[Damiankossek]

No to w takim razie dzięki (IMG:style_emoticons/default/smile.gif) bo tak mnie to Gryzlo czy dobrze robie ... Ale skoro jest to dopuszczalne to w takim razie dzięki bardzo

Pozdrawiam

[zend]

Mnie osobiście boli takie rozwiązanie (IMG:style_emoticons/default/happy.gif) Zapisz kody błedów w jednym miejscu i za pomocą klasy/funkcji pobieraj je, np

Kod

getMsg('toLowSoftwareVersion' , array('required' => '123' , 'current' => '122'));

[LSM]

Unikam pisania tekstów bezpośrednio w klasach - jeśli przyjdzie zmienić system na inną wersję językową trochę kiepsko zmieniać potem i wyszukiwać w plikach polskich tekstów. Stosuje stałe zdefiniowane w jakimś pliku konfiguracyjnym. Każdy komunikat mam wtedy w jednym miejscu.
np:

[PHP] pobierz, plaintext 
define('MSG_WRONG_APP_VERSION', 'Zbyt niska wersja Twojego oprogramowania. Wymagana ' .$config['app_ver'] );
// lub
$config['msg']['errors']['msg_wrong_app_version'] = ' Zbyt... ' ;
[PHP] pobierz, plaintext