• 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

0 głosów
216 wizyt
pytanie zadane 19 sierpnia 2021 w Nasze projekty przez Layoutowiec Mądrala (5,470 p.)
zamknięte 4 lutego 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 (222,610 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 (222,610 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.
4
komentarz 19 sierpnia 2021 przez ScriptyChris Mędrzec (180,980 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 Maniak (68,580 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

+3 głosów
odpowiedź 19 sierpnia 2021 przez Milesq Nałogowiec (30,930 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 (30,930 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ę :)
+3 głosów
odpowiedź 19 sierpnia 2021 przez ScriptyChris Mędrzec (180,980 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 157 wizyt
+1 głos
0 odpowiedzi 111 wizyt
pytanie zadane 9 kwietnia 2021 w PHP przez Lopus Początkujący (360 p.)
+3 głosów
2 odpowiedzi 189 wizyt
pytanie zadane 17 października 2020 w JavaScript przez FlamerX Nowicjusz (150 p.)

88,408 zapytań

137,019 odpowiedzi

305,805 komentarzy

58,661 pasjonatów

Motyw:

Akcja Pajacyk

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

Sklep oferujący ćwiczenia JavaScript, PHP, rozmowy rekrutacyjne dla programistów i inne materiały

Oto dwie polecane książki warte uwagi. Pełną listę znajdziesz tutaj.

...