Ocena kodu - JS I PHP

Question

Witam. Stworzyłem prostą gierkę przeglądarkowa. Można zagrać tutaj. Na stronie jest też link do GitHuba.  W webówce jestem raczej początkujący. Proszę o ocenę PHP i JS i wytknięcie ewentualnych błędów - z góry dzięki za review 

---

 

Answer 1

Po wpisaniu pseudo danych i wybraniu sign up a potem Play, dostałem takie coś:

Fatal error: Call to undefined method PDOStatement::fetch_assoc() in /home/endnote/domains/jozef-pilsudski.pl/public_html/game.php on line 125

W game.php w jednym miejscu (linia 19) tworzysz obiekt połączenia mysqli, a w linii 86 pod tą samą zmienną tworzysz obiekt pdo. Zdecyduj się.

Po drugie nie łącz php z htmlem, staraj się trzymać to całkiem osobno. Możesz użyć jakiegoś systemu szablonów, twig czy smarty.

Ponadto lepiej by było, gdybyś utworzył sobie klasę obsługującą łączenie z bazą danych.

W pliku index.php odnosisz się do pliku connect.php, który nie istnieje, przynajmniej w repo go nie ma.

Wszystkie nazwy plików po angielsku prócz jednego - podsumowanie.php, trochę oczy bolą :P

Tytuł commitów "no message" pozostawia wiele do życzenia ;)

Na Twoim miejscu przerobiłbym to też na obiektowe podejście. Rozumiem, że to bardzo prosta apka, ale trzeba na czymś się tego uczyć.

Poczytaj o PSR i staraj się też tego trzymać, na tę chwilę głównie chodzi o styl pisania.

JS'a nie oceniam, bo nie czuję ni trochę kompetentny w tym temacie. Logiki też na razie nie sprawdzałem, bo przerwa w pracy jest za krótka :D

Comments

Connect php jest w ignorowany przez gita i to celowy zabieg. A z tym PDO to zaraz poprawie XD