Rejestracja i logowanie Opcje

[huberthx]

Czy moglibyście zapoznać się z moim kodem rejestracji i logowania użytkowników.
Po ostatniej wpadce ze stronką chciałbym zasięgnąć opinii bardziej doświadczonych programistów, a że nie mam kogo się poradzić pisze dlatego na tym forum.
Oto kod:

[PHP] pobierz, plaintext 
<?php
session_start();
class Uzytkownik
{
    private $bazaDanych;
    //łączenie z bazą danych
    public function polaczZBazaDanych()
    {
        $this->bazaDanych = new mysqli('localhost','uzytkownik','haslo','bazaDanych');
    }
    //reozlaczenie z baza danych
    public function rozlaczZBazaDanych()
    {
        $this->bazaDanych->close();
    }
    //walidacja danych wprowadzonych przez użytkownika
    private function walidacja($dana)
    {
		//do uzupełnienia
        return $dana;
    }
    //rejestracja użytkownika
    public function zarejestrujUzytkownik($login,$email,$haslo1,$haslo2)
    {
		//łączenie z bazą danych
        $this->polaczZBazaDanych();
        //walidacja czterech zmiennych
        $login = $this->walidacja($login);
        $email = $this->walidacja($email);
        $haslo1 = $this->walidacja($haslo1);
        $haslo2 = $this->walidacja($haslo2);
        //zapytanie czy istnieje użytkownik o podanym loginie lub mailu
        $sqlSprawdzenieCzyIstniejeUzytkownikWBazieDanych = "SELECT * FROM uzytkownicy
            WHERE loginU = '".$login."' OR email = '".$email."'
        ";
        //wykonanie powyższego zapytania
        $rezultatSprawdzenia = $this->bazaDanych->query($sqlSprawdzenieCzyIstniejeUzytkownikWBazieDanych);
        //jezeli znaleziono użytkownika o takim  samym loginie lub emailu
        if ($rezultatSprawdzenia->num_rows > 0)
        {
            echo "<span class='zle'>Użytkownik o takim samym loginie lub email-u instnieje, Rejesrtacja nie powiodła się</span>";
            return false;
        }
        else
        {
			//sprawdzenie czy hasła są identyczne
        if ($haslo1 != $haslo2)
        {
			//jeżeli nie są identyczne hasła odpowiedni komunikat 
            echo "<span class='zle'>Hasła nie powtórzyły się</span>";
            return false;
        } 
        //zapytanie - rejestracja użytkownika - zapisanie do bazy danych
        $sqlRejestracjaUzytkownik = "INSERT INTO uzytkownicy (loginU,email,haslo) 
            VALUES ('".$login."','".$email."',SHA1('".$haslo1."'))";
            //wykonanie powyższego zapytania
        $rezultat = $this->bazaDanych->query($sqlRejestracjaUzytkownik);
        //jeżli rejestracja się powiodła odpowiedni komunikat
        if ($rezultat)
        echo "<span class='ok'>Zarejestrowano</span>";
        //jeżeli rejestracja się nie powiodła odpowiedni komunikat
        else
        echo "<span class='zle'>Rejestracja nie powiodła się</span>";
        }
        $this->rozlaczZBazaDanych();
    }
    //logowanie użytkownika
    public function zalogujUzytkownik($login,$haslo)
    {
		//walidacja
        $login = $this->walidacja($login);
        $haslo = $this->walidacja($haslo);
        //łączymy z bazą danych
        $this->polaczZBazaDanych();
        //zapytanie czy ligin i hasło się zgadzają
        $sqlLogowanie = "SELECT * FROM uzytkownicy WHERE
        loginU = '".$login."' AND haslo = SHA1('".$haslo."')";
        //wykonanie powyższego zapytania
        $rezultatLogowanie = $this->bazaDanych->query($sqlLogowanie);
        //jeżeli znaleziono dane o takim użytkowniku to logujemy
        if ($rezultatLogowanie->num_rows == 1)
        {
            $rezultatLogowanie = $rezultatLogowanie->fetch_assoc();
            //ustawienie sesji użytkownika - przechowuje login użytkownika
            $_SESSION['uzytkownik'] = $rezultatLogowanie['loginU'];
            //odpowiedzni komunikat po zalogowaniu
            echo "<span class='ok'>Zalogowano użytkownika ".$_SESSION['uzytkownik']." <a href='uzytkownik.php?akcja=wyloguj'>Wyloguj</a></span>";
        }
        else
        {
			//nie znaleziono użytkownika o takich danych - odpowiedni komunikat o tym fakcie
            echo "<span class='zle'>Nieudana próba logowania</span>";
        }//rozłączamyh z bazą danych
        $this->rozlaczZBazaDanych();
    }
    //wylogowywanie
    public function wyloguj()
    {
		//usuwanie zmiennej sesji
        unset($_SESSION['uzytkownik']);
        session_destroy();
        //odpowiedni komunikat informujący o wylogowaniu
        echo "<span class='ok'>Wylogowano</span>";
    }
    //pokazujemy info o zalogowanym użytkowniku lub o braku zalogowania 
    public function czyZalogowano()
    {
		//w zależności o istnieniu sesji użytkownika odpowiednia informacja
        if (isset($_SESSION['uzytkownik']))
        echo "<span class='ok'>Zalogowano jako ".$_SESSION['uzytkownik']."<a href='uzytkownik.php?akcja=wyloguj' >Wyloguj</a></a></span>";
        else
        echo "<span><a href='uzytkownik.php?akcja=zalogujFormularz'>Nie jesteś zalogowany - zaloguj się</a></span>";
    }
}
?>
 
[PHP] pobierz, plaintext 

Za ocenę dziękuję

Ten post edytował huberthx 25.11.2019, 21:16:51

[huberthx]

Poczytałem i oto wynik mojej nauki i prośba o ocenę.
Przepraszam za być może nieodpowiednie nazewnictwo metod bądź składowych
Oto zarys kodu:

[PHP] pobierz, plaintext 
<?php
 
class Db
{
	private static $instance;
	private $host = 'localhost';
	private $username = 'uzytkownik';
	private $password = 'haslo';
	private $database = 'bazaDanych';
	private $connection;
	public static function getInstance()
	{
		if (self::$instance === null)
		{
			self::$instance = new Db();
		}
		return self::$instance;
	}
	private function __construct()
	{
		$this->connection = new mysqli($this->host,$this->username,$this->password,$this->database);
	}
	private function __clone() {}
	public function getConnection()
	{
		return $this->connection;
	}
}
$Db = Db::getInstance();
 
class Uzytkownik
{
	private $login;
	private $haslo;
	public function setLogin($login) : void
	{
 
		$this->login = $login;
	}
	public function getLogin() : String
	{
		return $this->login;
	}
	public function setHaslo($haslo) : void
	{
		$this->haslo = $haslo;
	}
	public function getHaslo() : String
	{
		return $this->haslo;
	}
	private $db;
	public function bazaDanych($db)
	{
		$this->db = $db;
		return $this->db;
	}
	public function rejestracja($login,$haslo)
	{
		$this->setLogin($login);
		$this->setHaslo($haslo);
		$login = $this->getLogin();
		$haslo = $this->getHaslo();
		$this->db->query(/*ZAPYTANIE INSERT*/);
	}
}
 
$Uzytkownik = new Uzytkownik();
$Uzytkownik->bazaDanych($Db->getConnection());
$Uzytkownik->rejestracja('login','haslo');
 
[PHP] pobierz, plaintext