Ocena kodu https://github.com/tomi0001/nastroj__2019
Chcę żebyście ocenili, ponieważ zamierzam poprawić daną aplikację zrobię jej kopie i dodam kilka funkcji, więc ocena mi się przyda.
Ocena kodu https://github.com/tomi0001/nastroj__2019
Chcę żebyście ocenili, ponieważ zamierzam poprawić daną aplikację zrobię jej kopie i dodam kilka funkcji, więc ocena mi się przyda.
Ale, że co? Dajesz cudzą aplikację do oceny bo chcesz ją skopiować i nie wiesz czy jest dobrze napisana?
Puste commit messages, albo takie, które mówią tyle, co nic. Dobre opisy commitów pozwalają wyszukać miejsce, w którym zaszły interesujące nas zmiany. W jednoosobowym projekcie to może i nawet spamiętasz, ale w pracy zespołowej wiele osób pracuje nad wieloma rzeczami naraz - wtedy dobre opisy są nieocenione.
Taka tutaj funkcja: link. Po pierwsze - do porównywania stringów w PHP zawsze używaj ===
. Po drugie - rozważyłbym użycie tutaj switch
.
Zostawiasz dużo wykomentowanego kodu. Jeżeli to tylko tymczasowe, to można tak zrobić, tylko dać jakiś komentarz mówiący, co i dlaczego; w przeciwnym wypadku po prostu usuń i napisz opis do commita, który w razie potrzeby pozwoli łatwo odnaleźć, kiedy to zostało usunięte.
Unikaj inline CSS, zwłaszcza, gdy powtarzasz te same style w kilku elementach. Wrzuć to do .klasy
.
W jednym miejscu sklejasz URLe ręcznie. Co w wypadku, gdy przekazany do tej funkcji url
już zawiera jakiś ?param=wartosc
? Możesz zamiast tego użyć dostępnej w przeglądarkach klasy URL.
Te wiersze wyglądają jak gołe wartości, a nie coś, co miało być brane z env()
.
Na razie tyle, później popatrzę może jeszcze trochę.
Na szybko co rzuciło mi się na oczy:
Zarejestruj się i dołącz do największej społeczności programistów w Polsce.
Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.