[PHP] klasa logowania, Pytanie o model i controller Opcje

[slimboj]

Witam,
Właśnie piszę klasę odpowiadającą za logowanie. Staram się to zrobić zgodnie z modelem MVC.
Model pobiera informacje z bazy i zwraca 0 jeśli się nie udało pobrać danych i 1 jeśli się udało. Wygląda to mniej więcej tak:

[PHP] pobierz, plaintext 
<?
 
class MdlLogin
{
public function login($tb_prefix, $db_logon)
 {
  $select = mysql_query("SELECT * FROM ".$tb_prefix."users WHERE user_login = ".$_POST['login']." AND user_password = ".md5($_POST['password'])."");
  if(!$do_select = mysql_fetch_array($select))
  {
   $db_logon = 0;
   session_destroy();
  }
  else
  {
   $db_logon = 1;
   $_SESSION['user'] = $do_select['user_login'];  
  }
 }
}
 
?>
[PHP] pobierz, plaintext 

I teraz moje pytanie, a właściwie kilka.

1. Czy przekazywanie wyniku zapytania przez parametr $db_logon jest poprawne, a jeśli nie to jak inaczej to zrobić?
2. Czy sprawdzenie wyniku funkcji login zrobić w kontrolerze czy w modelu?
3. Czy funkcja session_destroy() w przypadku niepowodzenia i przypisanie wartoście do $_SESSION['user'] mają być wywoływane w modelu czy w widoku?

Pozdrawiam,
Max

Ten post edytował slimboj 13.05.2009, 12:37:31

[Leliw]

1. Czy przekazywanie wyniku zapytania przez parametr $db_logon jest poprawne, a jeśli nie to jak inaczej to zrobić?
2. Czy sprawdzenie wyniku funkcji login zrobić w kontrolerze czy w modelu?
3. Czy funkcja session_destroy() w przypadku niepowodzenia i przypisanie wartoście do $_SESSION['user'] mają być wywoływane w modelu czy w widoku?

IMHO
1. Chyba raczej nie za działa, w definicji metody parametr zwracany powinien być poprzedzony znakiem & (public function login($tb_prefix, &$db_logon)). Możesz tak, ale chyba lepiej zwrócić rezultat przez return i zdecydowanie lepiej użyć wartości true i false.
2. Zależy po co sprawdzasz. Jeśli po to, aby wyświetlić komunikat "logowanie nieudane" - w kontrolerze, jeśli np. zalogowani użytkownicy mają dostęp większej ilości danych - w modelu, tam gdzie dane są udostępniane widokom.
3. Najlepiej tam gdzie używasz session_start().