[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

[nospor]

Widze nadal kodu nie testujesz
FrontController.php

if ($response->getStatusCode() == 301) {
header('location:$file.php');
}

Po pierwsze zmienne w pojedynczych ciapkach nie sa parsowane, po drugie zmienna $file w ogole tam nie istnieje

Naucz sie uzywac === zamiast ==. Zaoszczedzi ci to kiedys sporo czasu na szukaniiu glupich bledow

return Yaml::parseFile('C:\xampp\htdocs\LibraryResponse\Core\db.yaml');
Gdy prosilismy bys wywalil konfig z gita, naprawde nie chodzilo nam bys go zastapil sciezka do pliku, do ktorego nie mamy dostepu

public static function getDB(): object
Czemu to zwraca object? Przeciez wiesz, ze to zwraca PDO a nie byle jaki objekt

Taki kod

[PHP] pobierz, plaintext 
$params = array();
        array_merge($params, $getParams);
        array_merge($params, $postParams);
        $this->params = $params;
[PHP] pobierz, plaintext 

Nie ma wiekszego sensu. Po mergujesz zawsze puste $params z jakas tablica? Przeciez wynikiem bedzie zawsze ta inna tablica. Co tu chcesz tak naprawde zrobic to zmergowac get i post czyli

[PHP] pobierz, plaintext 
        $this->params =array_merge($postParams, $getParams)
[PHP] pobierz, plaintext 

Skracaj zbedne bloki

[PHP] pobierz, plaintext 
if (is_callable([$controller_object, $action])) {
                return $controller_object->$action();
            } else {
                throw new Exception("Method $action (in controller $controller) not found and not returned response");
            }
[PHP] pobierz, plaintext 

Piszesz poprostu

[PHP] pobierz, plaintext 
if (is_callable([$controller_object, $action])) {
                return $controller_object->$action();
            }
throw new Exception("Method $action (in controller $controller) not found and not returned response");
[PHP] pobierz, plaintext 

Napotkałem się jeszcze na sformułowanie, że Response powinien zostać zwrócony z View za pomocą ResponseFactory, a nie z Controllera. To będzie następny etap ?

No raczej nie