Ocena kodu aplikacji

Odpowiedz Nowy wątek
2020-06-17 15:52

Rejestracja: 5 lat temu

Ostatnio: 15 godzin temu

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.

Pozostało 580 znaków

2020-06-17 20:28

Rejestracja: 1 rok temu

Ostatnio: 7 godzin temu

0

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

To jest moja aplikacja. - pol90 2020-06-17 21:29

Pozostało 580 znaków

2020-06-18 14:25

Rejestracja: 11 miesięcy temu

Ostatnio: 11 godzin temu

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

Pozostało 580 znaków

2020-06-18 17:18

Rejestracja: 3 lata temu

Ostatnio: 23 minuty temu

0

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

Pozostało 580 znaków

Odpowiedz

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