Добрый день.
Разбираюсь с программированием и с объектно ориентированным подходом.
Написал класс авторизации пользователей, прошу посмотреть, высказать свое мнение по поводу логики, кода.
Буду рад конструктивной критике.
Спасибо.
| Код |
<?php
if(!isset($_SESSION))
{
session_start();
}
error_reporting(E_ALL);
class Auth{
private $link;
public $sid;
function __construct(){
$this->link = mysql_connect("localhost", "root", "");
mysql_select_db("gbook");
$this->sid = session_id();
}
/*
функция "запомнить меня"
*/
public function RememberUser(){
setcookie ("login", $this->sid, time() + 100000);
}
/*
проверка корректности логина/пароля;
*/
public function validateUser($email, $password){
$q = "SELECT * FROM users WHERE login='{$email}' and password='{$password}'";
$r = mysql_query($q);
if (mysql_num_rows($r) == 1){
return TRUE;
} else {
return FALSE;
}
}
/*
авторизация пользователя;
*/
public function AuthUser($email){
$q = "UPDATE users SET session='{$this->sid}' WHERE login='{$email}'";
$r = mysql_query($q);
$_SESSION['login'] = $this->sid;
}
/*
проверка авторизованности пользователя;
*/
public function AuthUserNow(){
if (isset($_SESSION['login'])) {
return true;
} else {
if (isset($_COOKIE['login'])){
$login = $_COOKIE['login'];
if ($this->Getuid($login)>0) {
$this->RememberUser();
$q = "UPDATE users SET session='{$this->sid}' WHERE id='{$this->Getuid($login)}'";
$r = mysql_query($q);
$_SESSION['login'] = $login;
return true;
} else { echo 'нет такого'; return false; }
} else { echo 'авторизуйтесь'; return false; }
}
echo 'авторизуйтесь';
}
/*
возвращает id пользователя;
*/
public function Getuid($sid){
$q = "SELECT id FROM users WHERE session='{$sid}'";
$r = mysql_query($q);
if (mysql_num_rows($r) == 1){
$row=mysql_fetch_array($r);
return $row['id'];
} else return 0;
}
/*
возвращает имя пользователя;
*/
public function Getusername($sid){
$q = "SELECT login FROM users WHERE session='{$sid}'";
$r = mysql_query($q);
if (mysql_num_rows($r) == 1){
$row=mysql_fetch_array($r);
return $row['login'];
} else return 0;
}
public function logoutUser(){
if ($this->AuthUserNow()) {
unset($_SESSION['login']);
setcookie('login', '');
}
}
function __destruct(){
mysql_close($this->link);
}
}
?>
|
|
)