• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Code Review projektu serwisu - PHP I JS

VPS Starter Arubacloud
0 głosów
376 wizyt
pytanie zadane 3 lipca 2016 w Nasze projekty przez niezalogowany

Witam, będę bardzo wdzięczny jeśli ktoś bardziej doświadczony ode mnie przejrzy mój kod PHP i JS tej strony: http://whowasbetter-projekt.pl/ (Zdjęcia piłkarzy są dodane tylko do jednego meczu, bo nie chciałem sobie zaśmiecać serwera). 

Jestem osobą początkującą i cały czas się uczę, więc wytknięcie mi konkretnych błędów byłoby dla mnie bardzo pomocne. Chciałbym jednak abyście skupili się na kodzie PHP I JS. Kodu HTML (CSS) nie ma sensu oceniać, bo pisałem go jakiś czas temu i zdaję sobie sprawę, że responsywność strony wręcz nie istnieje i jest tam popełnionych wiele błędów. Będę projektował jednak nowy wygląd i wtedy zrobię to od nowa.

Cały kod znajduje się tutaj: 

https://github.com/GeneralXavi/wsbprojekt

Struktura tabeli związanej z porównaniami piłkarzy wygląda tak: http://scr.hu/2ja7/gnh5r Każde porównanie ma swoją własną tabelę.

Każda konstruktywna krytyka bardzo mile widziana. 

3 odpowiedzi

+1 głos
odpowiedź 3 lipca 2016 przez xandros Nałogowiec (29,450 p.)
wybrane 10 lipca 2016
 
Najlepsza

Na razie tylko zerknalem i będe aktualizował posta:

Baza:

- Ja z reguly używam utf8mb4_unicode_ci jako collation, no i z reguły używa się jakiegoś utf8

- Glówny klucz powinien mieć nazwe id. Nie jest ci potrzebne 'porownania', czyż nie?

- Jak się dobrze przyjrzysz, to zobaczysz pewne powtórzenia.  https://i.imgur.com/ddgaUml.png​ 
mozesz to wrzucić do osobnej tabeli (czyli to z np. zielonego bez cyferek + dodatkowe pole na identifikator) i relacja wiele do wielu :P

(Wrzucone na gita: https://github.com/GeneralXavi/wsbprojekt/issues/1)

Front:

- Ktos zauwazyl, ze grafiki dlugo sie laduja. Jedno to zmniejszyc rozmiar, ale możesz dodatkowo użyć jakiegoś preloadera, który ładuje grafiki przez jsa. Powinno to działać tak: póki grafikia się nie załadowała, tło jest czarne. Gdy js już załaduje daną grafike, powinna pojawiać się z 0 opacity do 100. Tak czy inaczej zmniejsz trochę ich rozmiar.

komentarz 4 lipca 2016 przez niezalogowany
Numer 1 oznacza zawodnika po lewej stronie, a numer 2 po prawej stronie strony. To nie są powtórzenia. :P
komentarz 4 lipca 2016 przez xandros Nałogowiec (29,450 p.)
Dodałem na gicie, bo w komentarzach i postach wygladają mocno średnio: https://github.com/GeneralXavi/wsbprojekt/issues/1
komentarz 4 lipca 2016 przez niezalogowany
Rozumiem, na pewno sobie poprawię. Dziękuję :)
+1 głos
odpowiedź 3 lipca 2016 przez jpacanowski VIP (101,940 p.)
  • nazwy plików, katalogów i tabel w bazie proponowałbym po angielsku nazywać
  • zoptymalizuj zdjęcia na stronie, każde tło ma po parę MB! https://tinyjpg.com/
  • nie mieszaj kodu HTML z PHP (sam właśnie w tej chwili piszę CMSa i używam Twiga)
  • kod JS w plikach .html przenieś do osobnych plików .js
komentarz 3 lipca 2016 przez jpacanowski VIP (101,940 p.)
Ten kto dał minusa może napisać za co?
komentarz 3 lipca 2016 przez xandros Nałogowiec (29,450 p.)
pewnie za twiga :V
komentarz 3 lipca 2016 przez jpacanowski VIP (101,940 p.)
Co masz na myśli? ;v
komentarz 3 lipca 2016 przez GaCeL Dyskutant (7,500 p.)
Za darmo nie wiesz za co :D
0 głosów
odpowiedź 4 lipca 2016 przez manjaro Nałogowiec (37,390 p.)
Ja się tylko doczepię do fragmentu "Przebiegnięte KM"

KM to są konie mechaniczne ;)

Kilometry z małych liter.

Pozdrawiam

Podobne pytania

0 głosów
2 odpowiedzi 307 wizyt
pytanie zadane 11 kwietnia 2019 w C# przez Shimeo7 Obywatel (1,910 p.)
+4 głosów
4 odpowiedzi 328 wizyt
pytanie zadane 9 marca 2019 w HTML i CSS przez smokolisz Mądrala (6,340 p.)
+1 głos
1 odpowiedź 156 wizyt
pytanie zadane 13 grudnia 2020 w HTML i CSS przez DominikPie Użytkownik (770 p.)

92,453 zapytań

141,262 odpowiedzi

319,085 komentarzy

61,854 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...