Forum PHP.pl

Mam problem z weryfikacja uzytkownika, mianowicie robie system logowan po zalogowaniu kazdy uzytkownik tj. root user są przekierowani do dwóch róznych szablonów nie wiem ale nie chce mi to chodzić:

[PHP] pobierz, plaintext 
<?php
//Klasa błędu dla klasy User
class AuthError extends Error {}
 
//Klasa User odpowiada za obsługę operacji zwišzanych
//z użytkownikami aplikacji
class User {
	//Identyfikator użytkownika 
	private $_Id;
	//Adres E-mail użytkownika
	private $_Email;
 
	// Konstruktor klasy
	public function __construct($userdata = NULL){
		foreach ($userdata as $key => $value) { 
			switch (strtolower($key)) {
				case 'usr_id':
					$this->setId($value);
					break;
				case 'uinfo_email':
					$this->setEmail($value);
					break;
			}
		}
	}
 
	// Metody ustawiajšce wartoœci właœciwoœci klasy
	public function setId($id){$this->_Id = $id; }
	public function setEmail($email){$this->_Email = $email;}
 
	// Metody zwracajšce wartoœci właœciwoœci klasy
	public function getId() {return $this->_Id;}
	public function getEmail() {return $this->_Email;}
 
	// Metoda autoryzujšca użytkownika
	public static function authorize_user ($userLogin, $userPassword) {
		// SprawdŸ, czy istnieje konto użytkownika
		$sqlquery = 'SELECT * FROM Users WHERE Users.USER = "'.htmlspecialchars($userLogin).'"';
		$result = DBManager::Query($sqlquery);
		// Pobranie wyników zapytania
	   if ($user = &$result->fetchRow(DB_FETCHMODE_ASSOC)) {
		if ($user['PASSWORD'] == md5($userPassword)) return new User($user);
		if ($user['ID'] == 1) echo'user';
	   //{$insertGoTo="templates/root/root.template.php";   
	   //echo "<script language="JavaScript">";
	  // echo "window.location = '$insertGoTo' ";
	  // echo "</script>";}
	   if ($user['ID'] == 2) echo'admin';
 
		}
 
 
 
 
		//Błšd autoryzacji
		throw new AuthError('Błšd autoryzacji - Użytkownik lub/i hasło niepoprawne!');	
	}
}
?>
[PHP] pobierz, plaintext 

A gdzie tu masz przekierowanie? Pomine juz fakt, ze ten kod:

[PHP] pobierz, plaintext 
<?php
if ($user['PASSWORD'] == md5($userPassword)) return new User($user);
		if ($user['ID'] == 1) echo'user';
	   if ($user['ID'] == 2) echo'admin';
?>
[PHP] pobierz, plaintext 

Przy poprawnej weryfikacji usera nigdy nie dojdzie do tych echo user i admin, bo masz return, ktory konczy wykonywanie funkcji. No, ale moze to efekt zamierzony (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

według mnie najlepiej zastosować wzorzec fabrykę poza tym zastosować strukturę taką

abstract class Person{}

class User extends Person{}
class Admin extends Person{}


I użycie wyjątka w twoim kodzie jest niewłaściwie...inaczej mówiąc nadmiarowe, przy tak błachym błędzie stosuj klasyczne if-else

Poza tym lepiej od razu w zapytaniu sql sprawdzić hasło i login(ustaw go na unique, bo co się staie, gdy będzie dwóch użytkoników o takich samych loginach i hasłach?, chyba, że przy zakładaniu konta już tego pilnujesz), poza tym w konstruktorze używasz switcha, kontruktor raczej używa się do inicjalizacji zmiennych, metod, obiektów naprzykład przy kompozycji, a nie używasz właściwie jako metody klasycznej

to takie moje uwagi, poprostu chce ci pomóc a nie krytykować, sam często tu się radzę i dobrych rad słucham (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Ten post edytował marast78 8.11.2006, 14:36:10