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.

[_chris_]

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.

1. No nie ma odwołan nie chciałem kopiować tutaj wszystkiego.
2. Oczywista oczywistość... Moje niedopatrzenie
3. No właśnie to jak to zrobić jest takie moje pytanie?

Co ma być w użytkowniku, czy ma mieć jakies metody, czy te metody do DB przenieść? Nie proszę o wytykanie tylko błędów, ale o naprowadzenie jak to zrobić poprawnie.

[!*!]

Nie proszę o wytykanie tylko błędów, ale o naprowadzenie jak to zrobić poprawnie.

Jak użyjesz wyszukiwarki na forum (tej google) z frazą "logowanie OOP", masz kilkanaście tematów z tym związanych i w prawie każdym drugie tyle odpowiedzi, które z pewnością rozwieją wszelkie wątpliwości.

[pyro]

A wytykanie błędów to nie jest naprowadzanie jak zrobić poprawnie?

1. Połączenie mógłbyś zaaplikować za pomocą Dependency Injection
2. To samo z logowaniem, rejestracją, itp.

[_chris_]

Czyli rozumiem, że dla każdej akcji: logowanie, rejestracja, przypominanie hasła mam mieć osobną klasę? Ja trochę poszedłem tokiem myślenia MVC, tak jak jest w cakePHP, a tam po prostu poszczególne operacje są funkcjami i stąd taki mój początek. Potem coś zmodzę, będę prosił o ocenę.