[PHP] Logowanie, zwrot/wypozyczenie ksiazek OOP - mocno początkujący Opcje

[mrpickles]

Cześć,
Zaczynam naukę OOP i w napisałem skrypt który umożliwia rejestracje/logowanie/zwrot/wypożyczenie książek.
Mam świadomość istnienia SOLID oraz PSR - czytałem o PSR 1 / 2 i staram się przestrzegać.
Nie znam MVC ani testów np. PHPUnit - to będą kolejne kroki w nauce.

Czy taki skrypt można określić obiektowym, czy jest to jakaś hybryda? Zanim pójdę dalej jw chciałbym nauczyć się dobrych nawyków i w prawidłowy sposób posługiwać się OOP.

Skrypt nie wszedł na forum, umieściłem na githubie

Link do GitHub

Moje dodatkowe pytania:

1. Tworzenie obiektu Database w konstruktorze innych klas, czy jest to prawidłowe?
2. Metoda checkCredentials klasy log - czy nie narusza zasady pojedynczej odpowiedzialności? Czy nie powinna np. wyszukiwać użytkownika,a następnie inna metoda powinna weryfikować dane?

Dzięki za wszystkie uwagi i poświęcony czas (IMG:style_emoticons/default/smile.gif)

[nospor]

Njapierw klasa dziedziczy po Database
class RegistrationForm extends Database

A potem ta sama klasa w konstruktorze dostaje obiekt Database
public function __construct(Database $connection)

Rozumiem, ze zapomniales usunac extednds Database ?


Klasa Database

[PHP] pobierz, plaintext 
require_once('config\config.php');
class Database
{
    private $host=DB_HOST;
    private $user=DB_USER;
    private $password=DB_PASSWORD;
    private $dbname=DB_NAM
[PHP] pobierz, plaintext 

powinna byc niezalezna od jakiegos tam konfigu. Dane konfiguracyjne powinna dostac w konstruktorze

Klasa Database

[PHP] pobierz, plaintext 
 try {
            $this->connection= new PDO($dbh, $this->user, $this->password, $options);
            $this->isConnected =true;
        } catch (PDOException $e) {
            $this->error=$e->getmessage();
            $this->isConnected = false;
        }
[PHP] pobierz, plaintext 

Powinna raczej pluc wyjatkiem na zewnatrz w przypadku braku polaczenia a nie tlumic go wewnatrz siebie. Teraz tak to masz zrobione,ze pomimo braku polaczenia, inne klasa odpalaja zapytania na polaczeniu, ktorego moze teoretycznie nie byc

rowCount a nie rowcount
To samo dotyczy wywolan metody rowCount z PDO.

staraj sie czasami czytac co piszesz, np to:
$log->getIsLoged()
nie ma wiekszego sensu. Czy jak sie pytasz kogos, czy ma bulke to mowisz tak:
"Daj mi czy masz bulke"

Czy moze tak:
"Masz bulke?"

Tak samo tutaj powinno byc poprostu
$log->isLoged()

Juz nie wspomnie o braku jednego "g"

Powinienies miec jeden plik glowny/publiczny przez ktory przechodza wszystkie akcje, np index.php i on dopiero powinien odpalac poszczegolne akcje i inicjowac mase niezbednych rzeczy. Teraz masz 6 plikow glownych a w kazdym polowa kodu jest powtorzona

Naprawde nie musisz wszystkieg pchac w php. Np ten plik

[PHP] pobierz, plaintext 
<?php
session_start();
if (!isset($_SESSION['loggedin'])) {
    header("location:index.php");
}
echo <<<END
<a href="borrow.php"> Borrow book </a></br>
<a href="return.php"> Return book </a></br>
<a href="history.php"> Account history </a></br>
<a href="logout.php"> Logout </a></br>
END;
[PHP] pobierz, plaintext 

Moze wygladac tak:

[PHP] pobierz, plaintext 
<?php
session_start();
if (!isset($_SESSION['loggedin'])) {
    header("location:index.php");
}
?>
<a href="borrow.php"> Borrow book </a></br>
<a href="return.php"> Return book </a></br>
<a href="history.php"> Account history </a></br>
<a href="logout.php"> Logout </a>
[PHP] pobierz, plaintext