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

question-closed Code review strony internetowej

VPS Starter Arubacloud
0 głosów
434 wizyt
pytanie zadane 19 sierpnia 2021 w Nasze projekty przez Layoutowiec Mądrala (5,470 p.)
zamknięte 4 lutego 2022 przez Layoutowiec
Witam, prosiłbym o ocenę kodu napisanej jakiś czas przeze mnie strony internetowej: co jest dobrze napisane, co poprawić oraz ewentualnie jakie nawyki byłoby dobrze wprowadzić :)

Miłego dnia,
Pozdrawiam
komentarz zamknięcia: Otrzymałem odpowiedź
komentarz 19 sierpnia 2021 przez Wiciorny Ekspert (269,120 p.)
No nie żeby coś, ale code-review ... to robią ludzie odpłatnie.  To jest poświęcanie swojego czasu, na rzecz czyjejś korzyści.

Rozumiem jakiś fragment kodu, problemu, optymalizacje, ale nie projektu strony.
Zresztą sory, ale ciężko coś tu ocenić skoro to jest strona napisana w HTML, CSS tylko i wyłącznie, generalnie to jest nie strona a utworzenie szablonu... graficznego.
komentarz 19 sierpnia 2021 przez Layoutowiec Mądrala (5,470 p.)
Rozumiem, zależałoby mi na w miarę ogólnym spojrzeniu na całość i ocenieniu czy nie pojawiają się jakieś większe błędy
komentarz 19 sierpnia 2021 przez Wiciorny Ekspert (269,120 p.)
tylko ze dla projektu w którym piszesz tylko w SCSS, CSS, HTML i importujesz zdjęcia, bardzo trudno mówić o błędach, bo tu wartości twórczej "twojej" jest bardzo mało.
3
komentarz 19 sierpnia 2021 przez ScriptyChris Mędrzec (190,190 p.)

No nie żeby coś, ale code-review ... to robią ludzie odpłatnie.  To jest poświęcanie swojego czasu, na rzecz czyjejś korzyści.

@Wiciorny, a czy pomaganie odpowiadając na pytania nie jest poświęcaniem swojego czasu na rzecz czyjejś korzyści? Jaka jest różnica (oprócz ilości poświęconego czasu) czy pomożesz komuś w formie zrobienia code review, czy odpowiesz na pytanie lub podasz wskazówki do rozwiązania problemu? Poza tym, na forum pomoc jest dobrowolna i jeśli nie chcesz poświęcać komuś czasu, to nie musisz.

Zresztą sory, ale ciężko coś tu ocenić skoro to jest strona napisana w HTML, CSS tylko i wyłącznie, generalnie to jest nie strona a utworzenie szablonu... graficznego.

Przecież strona internetowa nie musi zawierać nie wiadomo jak skomplikowanego kodu, czy nawet JavaScriptu. I to, że strona jest prosta, nie znaczy że nie ma tam czego oceniać.


@Layoutowiec, czy mógłbyś tą stronę pokazać w formie live? Choćby na GitHub Pages, jeśli nie masz własnego hostingu.

komentarz 19 sierpnia 2021 przez Layoutowiec Mądrala (5,470 p.)

@Layoutowiec, czy mógłbyś tą stronę pokazać w formie live? Choćby na GitHub Pages, jeśli nie masz własnego hostingu.

Jasne, dodałem do treści posta 

komentarz 19 sierpnia 2021 przez SzkolnyAdmin Szeryf (86,280 p.)

@Layoutowiec, strona z poziomym paskiem przewijania (na ekranie o 1366 px szerokości) to nie jest to, co tygrysy lubią najbardziej.

komentarz 19 sierpnia 2021 przez Layoutowiec Mądrala (5,470 p.)
Racja, dziękuję za czujność :)

2 odpowiedzi

+2 głosów
odpowiedź 19 sierpnia 2021 przez Milesq Nałogowiec (32,020 p.)
Przede wszystkim masz tylko 1 commit, musisz zacząć lepiej używać git'a. Po drugie pierwszy plik scss'owy do którego zajrzałem ma prawie 1000 lini kodu. To za dużo. Masz tam dużo powtarzających się fragmentów kodu, co scss powinien eliminować
komentarz 19 sierpnia 2021 przez Milesq Nałogowiec (32,020 p.)
po trzecie, niepotrzebnie trzymasz pliki .map i main.css w repo... używaj .gitignore.

Po czwarte, zerknij na narzędzia takie jak prettier i styleline
komentarz 19 sierpnia 2021 przez Layoutowiec Mądrala (5,470 p.)
Dzięki za pomoc, na pewno poczytam i zastosuję :)
+2 głosów
odpowiedź 19 sierpnia 2021 przez ScriptyChris Mędrzec (190,190 p.)
  • brakuje sekcji: headermain
  • brakuje nagłówka <h1>
  • obrazki o klasach v-line i h-line nie wydają się istotne dla zrozumienia treści strony, więc można je wstawić jako background w CSS zamiast jako <img>
  • linki w nawigacji nie prowadzą do odpowiadających im sekcji (w href jest sam #, bez podanego identyfikatora sekcji)

Podobne pytania

+3 głosów
0 odpowiedzi 220 wizyt
+3 głosów
0 odpowiedzi 236 wizyt
pytanie zadane 9 września 2022 w C# przez Artur Koniec Gaduła (3,670 p.)
+1 głos
0 odpowiedzi 260 wizyt
pytanie zadane 9 kwietnia 2021 w PHP przez Lopus Początkujący (360 p.)

92,451 zapytań

141,261 odpowiedzi

319,073 komentarzy

61,853 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!

...