[MySQL][PHP] Przebudowa klas/struktura klas Opcje

[Matkas]

Witam,
jako iż to mój pierwszy post na tym forum to wypadałoby się przedstawić.
Jestem 17nastolatkiem , który dopiero co raczkuje w php, pochodzącym z bardzo znanego miasta Bogatynia(7.08.2010), a na imię mam Mateusz! (IMG:style_emoticons/default/smile.gif)

A przechodząc do mojego problemu, to dawniej napisałem system logowania(o ile można to tak nazwać), ale teraz jak jestem starszy i mądrzejszy, to chciałbym Go przebudować, i zaczać bardziej doceniać OOP. Oczywiście wiem, że wszystko jest tu źle, dlattego chciałbym napisać wszystko od początku. Próbuje sobie uzmysłowić ile klas potrzebuje do logowania, i chodzenia po podstronach, w których wymagane jest zalogowanie.
Myślałem nad takimi:
class user_auth - Autoryzacja użytkownika, znajdują sie w niej funkcje potrzebne do zalogowania, informacje o użytkowniku(user_id itp.)
class session - Klasa zarządzająca sesjami oparta na bazie
class account_managment - Klasa do zarządzania kontem,zmiana haseł,informacji o profilu itd.

Problem mój polega na tym, że nie wiem jak te wszystkie 3 klasy połączyć, i w jaki sposób je dobrze napisać.
Próbowałem w mojej aktualnej klasie wprowadzić atrybuty, takie jak user_id,user_login,user_level. Lecz odwołując się do nich w innych plikach, nie uzyskuje żadnego rezultatu. Wygląda to mniejwięcej tak:

Nagłówek account.php

[PHP] pobierz, plaintext 
require_once ("mainfile.php");
[PHP] pobierz, plaintext 

Nagłówek mainfile.php

[PHP] pobierz, plaintext 
session_start(); 
ob_start();
ob_implicit_flush(0);
require_once('functions.php');
require_once('config/config.php');
require_once('class/class_database.php');
require_once('class/class_account.php');
$class_database = new class_database;
$class_account = new class_account;
[PHP] pobierz, plaintext 

Lecz w obu tych plikach nie jestem w stanie odwołać się do $class_account->user_id;
W jaki sposób zoptymalizować klasy?


Poniżej skrypt logowania o którym wcześniej wspominałem.

[PHP] pobierz, plaintext 
  function login($login_in_login_form,$password_in_login_form)
	{
	$login_exist = $this->check_login_exist($login_in_login_form);
	//Sprawdza czy login wpisany w formularzu istnieje
		if($login_exist==TRUE)
		//Jesli istnieje to ->
			{
			$password_match = $this->confirm_password_in_login($login_in_login_form,$password_in_login_form);
			//Sprawdza czy haslo pasuje do loginu
				if($password_match == TRUE)
				//Jesli pasuje to ->
					{
					$if_activated = $this->check_account_activated($login_in_login_form);			
					//Sprawdza czy konto jest aktywowane			
						if($if_activated == TRUE)
						//Jesli konto aktywowane to->
							{		
								$this->setsessions( $login_in_login_form);
								// Ustawia sesje
						$sucesfull = "Login sucesfull!  In(".RELOAD_TIME." seconds) you'll be redirected.";
						$to_display .= display_good(redirect($sucesfull));
						//Funkcja `display_good` wyswietla tekst w odpowiednim 'pojemniku'
						//Funkcja `redirect` przekierowuje na daną strone(tu 'account'), i wyswietla komunikat.
 
							}//Jesli konto nie aktywowane to
							else
							{	
			                                    $notactivated = "Account not activated";
						$to_display .= display_alert($notactivated));
							//Funkcje jak wyzej
							}
 
				}//Jesli haslo nie pasuje to wyswietl na ekranie
				else
				{
				$to_display .= display_wrong("Bad password.");				
				}		
			}//Jesli login nie istnieje
		else
		{
		$to_display .= display_wrong("This user does not exist.");
		}
	return $to_display;//zwroc wynik
	}  
[PHP] pobierz, plaintext 

Pozdrawiam, Mateusz

[Crozin]

1. Przecież napisałem, że od błędów są wyjątki, a nie zwracanie 0/1.
2.

To jak później się odwołać, do klasy wyświetlającej użytkownikowi 'błędy' logowania(konto nie zaaktywowane itp.)?

Tutaj wchodzisz w temat obsługi formularzy, czyli zagadnienia dosyć skomplikowanego - w tym celu powstają całe "subframeworki" w popularnych projektach.
3.

Tylko, nie mam pojęcia w jakim celu mam wykorzystać tą klase. Filtracja przesyłanych danych?

Pisząc o obiekcie Request miałem na myśli obiekt reprezentujący żądanie HTTP. A chyba wiesz jakie dane zawiera takie żądanie?
4.

// Funkcje ,które wyświetlają użytkownikowi komunikaty(np. złe hasło/login, brak aktywacji konta).

Pisząc o obiekcie Response miałem na myśli obiekt reprezentujący odpowiedź HTTP, czyli zestaw nagłówków (jak np. ciasteczka, prośba o przekierowanie, informacja o zarządzaniu pamięcią tymczasową) i treść odpowiedzi (którą generuje skrypt).
5.

Funkcje, które wykonują takie czynności jak:

Za każdym razem, gdy do opisu obiektu potrzebujesz używać spójnika "i" oznacza to, że najprawdopodobniej powinieneś podzielić go na mniejsze obiekty odpowiedzialne za poszczególne zadania. Oczywiście "dodaj/usuń/zmień" itp. traktuje się jako jedno.

[Matkas]

1. Przecież napisałem, że od błędów są wyjątki, a nie zwracanie 0/1.

Czyli mój kod wtedy wyglądałby w ten sposób:

[PHP] pobierz, plaintext 
if (!$activated) {
            $this->Error
	throw new Exception($user['login'][1]); 
            // Obsługując wyjątki, przejmowałbym parametry tablicy, i odpowiednio interpretował numer 'błędu'
}
 
if (!$password_match) {
	throw new Exception($user['login'][2]); 
}
 
if (!$login_exist) {
 
	throw new Exception($user['login'][3]); 
}
 
if (!$this->error) {
	return 1;
}
 
[PHP] pobierz, plaintext 

3. Pisząc o obiekcie Request miałem na myśli obiekt reprezentujący żądanie HTTP. A chyba wiesz jakie dane zawiera takie żądanie?

Zawiera informacje o Nas samych jeśli się nie myle?

5. Za każdym razem, gdy do opisu obiektu potrzebujesz używać spójnika "i" oznacza to, że najprawdopodobniej powinieneś podzielić go na mniejsze obiekty odpowiedzialne za poszczególne zadania. Oczywiście "dodaj/usuń/zmień" itp. traktuje się jako jedno.

Czy tak wyglądała by klasa, która jest odpowiedzialna za logowanie?

[PHP] pobierz, plaintext 
 
class UserAuth extends User
 
   {
     var $login;
     var $password;
      __construct($login,$password)
         {
               $this->login = $login;
               $this->password = $password;
         }    
      function login()
         {
            if (!$this->check_activated()) //Odwoluje sie do funkcji odziedziczonej z klasy User
           {
               $this->Error;
	   throw new Exception($user['login'][1]); 
              // Obsługując wyjątki, przejmowałbym parametry tablicy, i odpowiednio interpretował numer 'błędu'
           }
 
           if (!$this->password_match()) // Jak wyzej
                      {
	throw new Exception($user['login'][2]); 
                      }
 
           if (!$login_exist) //Jak wyzej
                       {
 
	           throw new Exception($user['login'][3]); 
                      }
 
           if (!$this->error) 
                       {
	              return 1;
                             //Przechodze do klasy obslugujaca sesje?
                       }
          }
      function logout()
         {
             //przechodze do klasy usuwajaca sesje?
         }
   }
 
[PHP] pobierz, plaintext 

Pozdrawiam, Mateusz