Mi lehetett a baj a kodommal? Elutasitottak allasrol php gyakornoki

Abszolút nem vagyok PHP szakértő, de ez MVCS, nem MVC. És én totálisan indokolatlannak látom itt a service réteget, ezzel együtt azt érzem, hogy nem teljesen világos neked, hogy mi kerül a service-be és mi a modelbe. Mondjuk jelen esetben szerintem a service-be kell semminek se kéne lennie.

Ezen kívül, az én ízlésemnek kicsit túl sok a view-ba való kód az index.php-ben, ugyanúgy, túl sok a nem oda való logika a view-ban. Szerintem nincs ez rendesen elszaparálva, nagyon keveredik az egész.

A db.php is olyan fura kicsit. Eleve, ha az .env fájl nem létezik, felesleges vizsgálni bármi mást is, azonnal kell dobni egy exceptiont. De nálad se a file_exists, se a mysqli nem dob kivételt. Illetve a mysqli dob, ha a MYSQLI_REPORT_STRICT be van állítva, de nálad ezt nem látom.

Az exceptiont is letudtad annyi, hogy Exception aztán kész. Én ezt úgy szoktam megoldani, hogy minden kivételre külön hibaosztályom van, mondjuk van egy általános AccountRepositoryException, ami az Exception-ből származik, és van egy AccountNotActivatedException, ami az AccountRepositoryException-ből származik, és akkor dobja a rendszer, ha a felhasználó nem aktivált fiókkal próbál belépni. Lehet, hogy te is így szoktad csinálni, csak kevés volt rá az időd.

Illetve olyan apróságok, mint hogy használhattál volna PDO, a prepared statement-ek miatt.

Esetleg lehet, hogy érdemes lenne kicsit elmenned statikus típusosság felé, megadni a paraméterek típusát, a visszatérési értékek típusát.

Nálam pl. az account entity osztályban, ami egy accountot reprezentál, a propertyk inicializációja, így néz ki:

private ?int $id = null;

private ?string $username = null;

private ?string $email = null;

private ?string $newemail = null;

private ?int $regdate = null;

private ?int $lastlogin = null;

private ?string $password = null;

private ?int $status = null;

private ?string $activationcode = null;

private ?string $passwordrecoverycode = null;

private ?string $role = null;

Továbbá lehetne használni a PHP újabb szolgáltatásait, pl. a constructor property promotiont.

Amúgy mennyi időt adtak erre a feladatra?

A JS, CSS és akár a View részt rakhatnád Assets, vagy Resources mappába.

Ajánlom figyelmedbe, hogy nézd meg pl a Laravel mappa szerkezetét.

Igen ami nekem is feltűnt, mint a fenti válaszolónak, hogy Service-be raktad amit, amúgy a modellbe kellett volna.