[PHP][MYSQL] Rozwiązanie problemu w klasie Opcje

[Matado]

Piszę dla siebie klasę do logowania. Jak widać, jest w niej wylogowywanie, logowanie, ale sprawdzanie hasła takiego prawdziwego jeszcze nie ma.
Zrobię to w najbliższym czasie, razem z soleniem hasła + sha1. Ale mniejsza o to.

Mam małe pytanie jeśli chodzi o metodę get(). Wydaje mi się że trochę brzydko wygląda łączenie się z bazą danych w jednej klasie odpowiadającej za logowanie. Jak skonstruowalibyście tą metodę get(), żeby pobierała dane, ale bez łączenia się z bazą?
Czy dobrym pomysłem jest tu zrobienie osobnej klasy, która łączy się z bazą i pobiera dane, dalej include do pliku z klasą itd?

[PHP] pobierz, plaintext 
<?php
 
session_start();
error_reporting(0);
 
class Auth
{
    private $name;
    private $password;
    public $messages;
 
    public function get()
    {
        $pdo= new PDO ('mysql:host=localhost;dbname=trial','root','');
        $stmt= $pdo-> prepare ('SELECT name,password FROM experiment WHERE name=:name ');
        $stmt-> bindValue (':name', $_POST['username'] , PDO::PARAM_STR);
        $stmt-> execute();
      
        while ($row= $stmt-> fetch())
        {
            $this-> name= $row['name'];
            $this-> password= $row['password'];
        }
        $stmt-> closeCursor();
    }
    
    public function login($login, $pass)
    {
        if (! empty($login) && ! empty($pass))
        {
            if ($this-> name== $login && $this-> password== $pass)
            {
                $_SESSION['user']= $this-> name;
                $this-> messages= 'Welcome: ' . $_SESSION['user']
                . ' <a href="execute.php?logout=true">Signout</a>';
            }
            else
            {
                $this-> messages= 'Invalid username or password';
            }
        }
        else
        {
            $this-> messages= 'Fill in all fields';
        }
    }
    
    public function signout()
    {
        if ($_GET['logout'])
        {
            unset ($_SESSION['user']);
            session_destroy();
            session_regenerate_id;
            header ("Location: form.php");
        }
    }
    
    public function show()
    {
        return $this-> messages;
    }
}
 
?>
[PHP] pobierz, plaintext 

[Matado]

Ok, dzięki. Myślałem też żeby do tej klasy wrzucić ORM zamiast zapytań w SQL, żeby to porządniej wyglądało, ale nie wiem czy się opłaca.
Niby wydajność by spadła, ale przy małej liczbie użytkowników wydalałoby, no i bardziej czytelny kod.

@pejott a w czym to jest zrobione 15 linijka kodu (IMG:http://forum.php.pl/style_emoticons/default/blink.gif)

Ten post edytował Matado 3.04.2009, 21:47:16