[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)

[mrpickles]

Zakręciłem się i pracowałem na klonie, a zrobiłem upload nie klona z jakimiś innymi drobnymi poprawkami. Już na pewno nie ma extends.

Nie byłem pewny czy chodzi o wyechowanie htmla, czy właśnie jak pytałem o zrobienie np. index do którego includuje index_view, który to zawiera html - ale miałem problem żeby oddzielić to w 100%.

Korzystając z okazji zanim będę przerabiał temat to dopytam właśnie o to.

Oddzielenie html od php na przykładzie z dużym echem - chodzi mi o sam sposób, a nie czy w tym pliku to rzeczywiście było potrzebne (IMG:style_emoticons/default/smile.gif)
mysite

[PHP] pobierz, plaintext 
<?php
session_start();
if (!isset($_SESSION['loggedin'])) {
    header("location:index.php");
}
require("mysite_view.php");
[PHP] pobierz, plaintext 

mysite_view

[PHP] pobierz, plaintext 
<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>
 
[PHP] pobierz, plaintext 

Jednak w przypadku return wydaje mi się to niemożliwe z uwagi na wtopienie w html php w petli foreach.

[PHP] pobierz, plaintext 
<?php
session_start();
 
if (!isset($_SESSION['loggedin'])) {
    header("location:index.php");
}
 
require __DIR__ . '/vendor/autoload.php';
use \Library\Library;
use \Library\Database;
 
$db = new Database();
$book = new Library($db);
if (isset($_POST['return'])) {
    $book->returnBook($_POST);
}
echo '<form name="return" method="post">';
 
foreach ($book->getBorrowedBooks() as $position) {
    echo $position->autor.$position->tytul.$position->rok."<input type='checkbox' name='return[]' value=".$position->idksiazki."></br>";
}
echo '<input type="submit" value="Return"></form><a href="mysite.php">Back</a>';
[PHP] pobierz, plaintext 

Czy chodzi w takim przypadku o nierodzielanie do dwóch plików, a o zastosowanie składni alternatywnej dla php? Czy jednak chodzi o fizyczne rozdzielanie?