Autoryzacja Opcje

[MStaniszczak]

Witam

Niestety z powodu ograniczenia wielkościtekstu dozwolonego na formum nie udało się tu poprawnie umieścicć mojej klasy do autoryzacji. Jeśli jest taka możłiwość postaram się aby została ona umieszczona na stronie.

Pozdrawiam
Marcin Staniszczak

Ten post edytował MStaniszczak 1.03.2005, 01:10:57

[Vengeance]

Widze, że stosujesz php5 więc ciut "ostrzej" będę oceniał ;]

[PHP] pobierz, plaintext 
<?php
 
function SQL_Auth_GetConfiguration($table) {
	global $dbConnection;
 
	return $dbConnection->Execute('SELECT * FROM '.$table);
}
 
?>
[PHP] pobierz, plaintext 

Czemu tu stosujesz global (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) To takie nieladne (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)
No i czemu nie opakujesz tego w jakieś klasy ?

[PHP] pobierz, plaintext 
<?php
 
if(stristr('class.authorization.inc.php', $_SERVER['PHP_SELF'])!==false) {
	header('Location: ../../index.php');
	die();
}
 
?>
[PHP] pobierz, plaintext 

Czy nie powinno byc jakiegos erroru czy cos zamiast przekierowania ? (co gdy user zmieni katalog w ktorym klasa jest umieszczona).


Skoro już php5, to może warto wykorzystać wyjątki zamiast die() ?

[PHP] pobierz, plaintext 
<?php
 
	public function getSurname() {
		if($this->isLogin)
			return $this->uSurname;
		else
			return  false;
	}
 
?>
[PHP] pobierz, plaintext 

Czy klasa autoryzacji nie powinna operować tylko na loginie i haśle? Ktos może niechcieć miec kolumny Nazwisko w swej aplikacji. Choć to dość indywidualna sprawa ;]

Przydałby się także jakiś "example" ;]