Formularz logowania OOP + PDO + Smarty Opcje

[_chris_]

Witam wszystkich

Jako że się uczę całe życie i staram rozwijać, teraz stanąłem przed kolejnym zadaniem. Chciałem zrobić sobie prosty formularz logowania (bez rejestracji itd) wykorzystując OOP i PDO do połączenia z bazą danych. Nie bardzo mogę sobie poradzić z organizacją kodu. Stworzyłem sobie klasę:

[PHP] pobierz, plaintext 
class Uzytkownik{
var imie;
 
public function __construct($_imie){
$this->imie = $_imie;
}
 
}
[PHP] pobierz, plaintext 

Mam też klasę DB zaimplementowaną jako Singleton

[PHP] pobierz, plaintext 
class DB{
private username = "xxx";
private password = "xxx";
private host = "xxx";
private dbName = "xxx";
 
public function __contruct(){
            $this->db = new PDO('mysql:host='.$host.';dbname='.$dbName, self::$username, self::$password);   
            $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); 
        }
 
public static function getInstance(){
            if(!isset(self::$instance)){
                $object = __CLASS__;
                self::$instance = new $object;
 
            }
            return self::$instance;
}
}
[PHP] pobierz, plaintext 

No i właśnie tutaj zaczynają się schody. Chciałbym mieć metodę do logowania. Czy ta metodę powinienem umieścić w klasie Użytkownika czy DB ? Jeśli w użytkowniku, to czy przypisywać zawsze użytkowniki instancję DB? Prosiłbym o pomoc, generalnie to opis tego jak Wy to robicie w swoich aplikacjach. Wiem że jak teraz źle coś zrobię, to potem poprawki będą ciężkie.

Generalnie najbardziej chodzi mi o rozwiązanie tego gdzie, co i w czym ma być w tych klasach żeby potem utworzenie formularza logowania zajęło raptem parę linijek, np
Index.php

[PHP] pobierz, plaintext 
if(isset($_SESSION['uzytkownik']) $smarty->display('index.tpl');
else header('location: login.php') 
[PHP] pobierz, plaintext 

Login.php

[PHP] pobierz, plaintext 
if(isset($_POST['submit']) if(zaloguj($_POST['login'], $_POST['password'])) header("location: index.php");
else $smarty->display('login_form.tpl');
[PHP] pobierz, plaintext 

Z góry dziękuję za porady

[pyro]

Po pierwsze twój kod jest zupełnie bez sensu, bo przykładowo:
1. Nieprawidłowe odwołania do właściwości klasy
2. Singleton razem z publicznym konstruktorem, do którego brak zresztą odwołania.
3. Tworzenie niepotrzebnych zalezności od innych klas
4. itd....

Co do głównego wątku: logowanie, rejestracja, obsługa użytkownika itp. to powinny być oddzielne klasy, a nie kolejne metody.