Sprawdzenie skryptu php

Question

Witam, czy był by ktoś chętny i przejrzał mój kod. Z góry dziekuje :)

https://github.com/zawiszaty/planj_z

http://zawiszaty.ct8.pl/planj_z/

http://zawiszaty.ct8.pl/planj_z/zaloguj.php

dane do logowania admin admin

Answer 1

Witam.

• Kod nie jest obiektowy
• Mieszasz kod HTML z kodem PHP. Po co? Użyj jakiegoś systemu szablonów (np. twig), lub napisz własny, np. w PHP :). Zobaczysz, wyjdzie Ci to na dobre.
• Plik send_edit.php. Po co tyle ifów? Nie lepiej po prostu wysłać hurtem wszystkie dni (oczywiście po wcześniejszym sprawdzeniu, czy dane zmienne sesyjne na podstawie których ustalamy dzień, wtedy tylko jedna pętelka i sprawa załatwiona)?

Ogólnie spoko. Można się też przyczepić do wyglądu, ale to kwestia gustu. Działa? Działa :)

Comments

Wygląd będzie poprawiany ten jest tylko tak na próbę. Dziękuje za rady na pewno uzyje ich w następnym projekcie.

Answer 2

Kod niezły ale zwracaj uwagę na czytelność.

I właściwie jedna uwaga do całej strony ,zwykle zawartość strony zamyka się w div'ie który się trochę środkuje. Możesz np. Poprawić formularz logowania tak żeby inputy nie dotykały tak bardzo marginesów bo na to wystarczy dużo mniej miejsca:) po prostu chodzi o to żeby zachować pewien odstęp od krawędzi. Zobacz na 4. Odcinek CSS na kanale Mirosława Zelenta tam formularz jest po środku , otoczony cieniem a reszta ma inny kolor.

Ale poza tym strona dobrze wykonana a wiadomo że potem będzie więcej treści:)

Answer 3

Sytax error przy dodawaniu.  Stosuj. Camel case, kod ma być po angielsku.  + za pdo.

Comments

jak zauważyłem tu error wyskakuje jak naciśniesz dodanie klasy a input z nazwa będzie pusty albo jak 2 klasy beda miały taka sama nazwę. Czy u ciebie wyskoczyło coś innego jeszcze ?

---

kliknąłem dodaje , towrzy się button boostrapowy i dostaje syntaxa.,