Ocena kodu aplikacji

0

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.

0

Ale, że co? Dajesz cudzą aplikację do oceny bo chcesz ją skopiować i nie wiesz czy jest dobrze napisana?

2
  1. 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.

  2. 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.

  3. 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.

  4. Unikaj inline CSS, zwłaszcza, gdy powtarzasz te same style w kilku elementach. Wrzuć to do .klasy.

  5. 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.

  6. 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ę.

1

Na szybko co rzuciło mi się na oczy:

  • brak migracji i seederów: nie odtworze tej aplikacji bo nie wiem nic o bazie danych
  • puste modele
  • brak relacji między modelami a za to w serwisach popisane właśnie funkcje, które można by oprzeć o ORM

1 użytkowników online, w tym zalogowanych: 0, gości: 1