Klasa logowania usera Opcje

[michalboss]

Witam

Napisałem taką klasę logowania użytkownika w systemie.
Nie jest jeszcze skończona, ale chciałem prosić o ewentualne wskazanie błędów w w/w kodzie, ponieważ dopiero zaczynam OOP w php.

[PHP] pobierz, plaintext 
<?php
 
class Login
{
 
    private $_dbh;
    private $_config;
    private $_lang;
    private $_isLoggin = FALSE;
    private $_idSession;
    private $_email;
    private $_password;
 
    public function __construct($dbh, $config, $lang)
    {
        $this -> _dbh = $dbh;
        $this -> _config = $config;
        $this -> _lang = $lang;
    }
 
    /**
     * Metoda sprawdza czy user jest zalogowany w systemie
     * 
     * @access public
     * @return bool
     */
    public function checkLoggin()
    {
        return $this -> _isLoggin;
    }
 
    /**
     * Metoda sprawdza czy istnieją zmienne sesyjne - jeśli tak, to loguje usera w klasie
     * 
     * @access public
     * @return bool
     */
    protected function isLoggin()
    {
        if($_SESSION['isLoggin'] == TRUE)
        {
            $this -> _isLoggin = $_SESSION['isLoggin'];
            $this -> _idSession = session_id();
            $this -> _email = $_SESSION['email'];
            return TRUE;
        }
        else
        {
            return FALSE;
        }
    }
 
    /**
     * Metoda logowania - sprawdzania czy użytkownik jest zalogowany i czy istnieją zmienne formularza logowania 
     * Jeśli tak, to sprawdza w bazie poprawność e-mail oraz hasła i dokonuje logowania w systemie 
     * 
     * @access public
     * @return void
     */
    public function userLogin()
    {
        $errorMsg = '';
        if(!($this -> _isLoggin) && isset($_POST['logEmail']) && isset($_POST['logPassword']))
        {
            $sql = $this -> _dbh -> prepare('SELECT A.IdUser, A.Email, A.Password, A.Salt, A.FirstName, A.LastName, A.ErrorLogin, A.ErrorLoginTime, A.Current  '
                    . 'FROM `users` A '
                    . 'WHERE A.Email=:email LIMIT 1');
            $sql -> bindValue(':email', $_POST['logEmail'], PDO::PARAM_STR);
            $sql -> execute();
            if($sql -> rowCount() > 0)
            {
                if($row = $sql -> fetch())
                {
                    if(self::encode($_POST['logPassword'], $row['Salt']) == $row['Password'])
                    {
                        $_SESSION['isLoggin'] = TRUE;
                        $_SESSION['email'] = $row['Email'];
                        $_SESSION['IdUser'] = $row['IdUser'];
                        $this -> isLoggin();
                    }
                    else
                    {
                        $errorMsg = $this -> _lang['errorLogin'];
                    }
                }
            }
            else
            {
                $errorMsg = $this -> _lang['errorLogin'];
            }
 
            if(!empty($errorMsg))
            {
                throw new Exception($errorMsg);
            }
        }
        else if(!($this -> _isLoggin) && isset($_SESSION['isLoggin']) && $_SESSION['isLoggin'] == TRUE)
        {
            $this -> isLoggin();
        }
    }
 
    /**
     * Metoda wylogowuje uzytkownika i usuwa sesję
     * 
     * @access public
     * @return void
     */
    public function userLogout()
    {
        unset($_SESSION);
        unset($this -> _isLoggin);
        unset($this -> _idSession);
        unset($this -> _email);
        session_destroy();
        header("Location: index.php?msg=logout");
    }
 
    /**
     * Metoda hashuje hasło używając podanej soli z kartoteki usera
     * 
     * @access public
     * @param string $Password
     * @param string $Salt
     * @return string
     */
    public static function encode($Password, $Salt)
    {
        return hash('sha256', $Salt . hash('sha256', $Password . $Salt));
    }
 
    /**
     * Metoda tworzy sól podanego hasła
     * 
     * @access public
     * @param string $Password
     * @return string
     */
    public static function salt($Password)
    {
        return substr(hash('sha256', microtime() . substr($Password, 0, 3)), 0, 5);
    }
 
    public static function timemicro()
    {
        list($z1, $se) = explode(" ", microtime());
        return ((float) $z1 + (float) $se);
    }
 
}
?>
[PHP] pobierz, plaintext 

[michalboss]

Trochę zmieniłem kod klasy - może to jest trochę bardziej prawidłowe od poprzedniego?
Jeśli chodzi o metody statyczne to zdefiniowałem je aby także użyć w innych klasach jak user -> rejestracja.

[PHP] pobierz, plaintext 
<?php
 
class Login
{
 
    private $dbh;
    private $config;
    private $lang;
    private $isLogged = FALSE;
    private $idSession;
    private $email;
    private $idUser;
 
    public function __construct($dbh, $config, $lang)
    {
        $this->dbh = $dbh;
        $this->config = $config;
        $this->lang = $lang;
    }
 
    /**
     * Metoda sprawdza czy user jest zalogowany w systemie
     * 
     * @access public
     * @return bool
     */
    public function checkLogged()
    {
        return $this->isLogged;
    }
 
    /**
     * Metoda sprawdza czy uzytkownik jest zalogowany (w klasie) - jeśli tak, to zapisuje dane z klasy do sesji
     * 
     * @access public
     * @return bool
     */
    protected function setSessionData()
    {
        if($this->isLogged == TRUE)
        {
            $_SESSION['isLogged'] = $this->isLogged;
            $_SESSION['idUser'] = $this->idUser;
            $_SESSION['email'] = $this->email;
        }
    }
 
    /**
     * Metoda sprawdza czy uzytkownik jest zalogowany (sesję) - jeśli tak, to zapisuje dane z sesji do klasy
     * 
     * @access public
     * @return bool
     */
    protected function getSessionData()
    {
        if(isset($_SESSION['isLogged']) && $_SESSION['isLogged'] == TRUE)
        {
            $this->isLogged = $_SESSION['isLogged'];
            $this->idUser = $_SESSION['idUser'];
            $this->email = $_SESSION['email'];
            $this->idSession = session_id();
        }
    }
 
    /**
     * Metoda logowania - sprawdza czy użytkownik nie jest już zalogowany i czy istnieją zmienne formularza logowania 
     * Jeśli tak, to sprawdza w bazie poprawność e-mail oraz hasła i dokonuje logowania w systemie
     * Sprawdzana jest także ilość błędnych logowań - max 5 oraz czas od pierwszego błędnego logowania. 
     * 
     * @access public
     * @return void
     */
    public function userLogin()
    {
        $errorMsg = '';
        if(!($this->isLogged) && isset($_POST['logEmail']) && isset($_POST['logPassword']))
        {
            $sql = $this->dbh->prepare('SELECT A.IdUser, A.Email, A.Password, A.Salt, A.FirstName, A.LastName, A.ErrorLogin, A.ErrorLoginTime, A.Current  '
                    . 'FROM `users` A '
                    . 'WHERE A.Email=:email LIMIT 1');
            $sql->bindValue(':email', $_POST['logEmail'], PDO::PARAM_STR);
            $sql->execute();
            if($sql->rowCount() > 0)
            {
                if($row = $sql->fetch())
                {
                    $timeActual = strtotime(date("Y-m-d H:i:s"));
                    $timeLogin = strtotime($row['ErrorLoginTime']);
                    $timeCheck = $timeActual - $timeLogin;
                    $timeCheckMin = round(60 - ($timeCheck / 60), 0);
 
                    if(self::decode($_POST['logPassword'], $row['Password']) == TRUE)
                    {
                        if($row['Current'] == 0)
                        {
                            $errorMsg = $this->lang['errorLoginCurrentError'];
                        }
                        else if($row['ErrorLogin'] >= 5 && ($timeCheck <= 3600))
                        {
                            $errorMsg = $this->lang['errorLoginTime'] . ' ' . $timeCheckMin . ' min.';
                        }
                        else
                        {
                            $this->isLogged = TRUE;
                            $this->email = $row['Email'];
                            $this->idUser = $row['IdUser'];
                            $this->idSession = session_id();
                            $this->setSessionData();
                            if($row['ErrorLogin'] > 0 || $row['ErrorLoginTime'] != NULL)
                            {
                                $this->resetErrorLogin();
                            }
                        }
                    }
                    else
                    {
                        if($row['ErrorLogin'] >= 5 && ($timeCheck <= 3600))
                        {
                            $errorMsg = $this->lang['errorLoginTime'] . ' ' . $timeCheckMin . ' min.';
                        }
                        else
                        {
                            $this->updateErrorLogin($_POST['logEmail']);
                            $errorMsg = $this->lang['errorLogin'];
                        }
                    }
                }
            }
            else
            {
                $this->updateErrorLogin($_POST['logEmail']);
                $errorMsg = $this->lang['errorLogin'];
            }
 
            if(!empty($errorMsg))
            {
                throw new Exception($errorMsg);
            }
        }
        else if(!($this->isLogged) && isset($_SESSION['isLogged']) && $_SESSION['isLogged'] == TRUE)
        {
            $this->getSessionData();
        }
    }
 
    /**
     * Metoda resetuje ilośc błędów logowania oraz czas, 
     * dla użytkownika, w przypadku powodzenia logowania
     * 
     * @access public
     * @return void
     */
    protected function resetErrorLogin()
    {
        $sql2 = $this->dbh->prepare('UPDATE `users` SET ErrorLogin = 0, ErrorLoginTime = NULL '
                . 'WHERE Email=:email');
        $sql2->bindValue(':email', $this->email, PDO::PARAM_STR);
        $sql2->execute();
    }
 
    /**
     * Metoda zwiększa ilośc błędnych logowań oraz zapisuje czas pierwszego błednego logowania
     * 
     * @access public
     * @return void
     */
    protected function updateErrorLogin($logEmail)
    {
        $sql2 = $this->dbh->prepare('UPDATE `users` SET '
                . 'ErrorLogin = ErrorLogin+1, '
                . 'ErrorLoginTime = if(ErrorLoginTime IS NULL, \'' . date("Y-m-d H:i:s") . '\', ErrorLoginTime) '
                . 'WHERE Email=:email');
        $sql2->bindValue(':email', $logEmail, PDO::PARAM_STR);
        $sql2->execute();
    }
 
    /**
     * Metoda wylogowuje uzytkownika, niszczy sesję i usuwa dane
     * 
     * @access public
     * @return void
     */
    public function userLogout()
    {
        unset($_SESSION);
        unset($this->isLogged);
        unset($this->idSession);
        unset($this->email);
        unset($this->idUser);
        session_destroy();
        header("Location: index.php?msg=logout");
    }
 
    /**
     * Metoda hashuje hasło używając podanej soli z kartoteki usera
     * 
     * @access public
     * @param string $Password
     * @param string $Salt
     * @return string
     */
    public static function encode($password, $salt, $rounds = 10000)
    {
        // pass in the password, the number of rounds, and the salt
        // $5$ specifies SHA256-CRYPT, use $6$ if you really want SHA512
        return crypt($password, sprintf('$6$rounds=%d$%s$', $rounds, $salt));
    }
 
    /**
     * Metoda rozszyfrowuje podane hasło
     * 
     * @access public
     * @param string $Password
     * @param string $Salt
     * @return string
     */
    public static function decode($password, $dbPassword)
    {
        // extract the hashing method, number of rounds, and salt from the stored hash
        // and hash the password string accordingly
        $parts = explode('$', $dbPassword);
        $testPassword = crypt($password, sprintf('$%s$%s$%s$', $parts[1], $parts[2], $parts[3]));
        return ($dbPassword === $testPassword) ? TURE : FALSE;
    }
 
    /**
     * Metoda tworzenia 16-znakowej soli
     * 
     * @access public
     * @return string
     */
    public static function salt()
    {
        //return substr(hash('sha256', microtime() . substr($Password, 0, 3)), 0, 5);
        return substr(str_replace('+','.',base64_encode(md5(mt_rand(), true))),0,16);
    }
}
?>
[PHP] pobierz, plaintext