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

Ocena kodu [HTML,CSS,JS]

VPS Starter Arubacloud
+2 głosów
373 wizyt
pytanie zadane 15 lutego 2016 w JavaScript przez radek024 Szeryf (77,180 p.)

Witam ;)

Przed chwilą skończyłem dopracowywanie gry(znana jako kółko i krzyżyk). Napisałem ją ze względu na to, że w JSie stawiam swoje pierwsze kroki i chciałem się sprawdzić.

Teraz chciałbym się dowiedzieć od Was, co sądzicie o jej kodzie - myślę, że nie jest on nawet dobry, ale to jak zawsze z początku ;). Wiem, że strona nie jest responsywna, myślę też że nie do końca dobrze zastosowałem tagi do umieszczania elementów. Nie jestem pewny także nazewnictwa klas z użyciem myślników.

Link do samej gierki: [KLIK]

Link do kodu: [KLIK]

Dziękuje za opinie z góry i pozdrawiam :).

3 odpowiedzi

+3 głosów
odpowiedź 15 lutego 2016 przez Comandeer Guru (599,730 p.)

Brakuje przycisku "Restart".

var tytul = document.getElementById("duzy").innerHTML;
tytul = tytul.toUpperCase();
document.getElementById("duzy").innerHTML=tytul;
//to było trudne XD

text-transform: uppercase w CSS.

Zamiast nadawania wszystkiemu [onclick] warto zastosować po prostu event delegation – wówczas wszystkimi zdarzeniami na planszy będziesz kierował z jednego miejsca.

No i przede wszystkim: gry nie mogą być w globalnym scope, bo to pozwala na oszukiwanie ;)

komentarz 15 lutego 2016 przez radek024 Szeryf (77,180 p.)
Wiem o takiej właściwości, chciałem po prostu zrobić to JSie. Dzięki za opinie.
+1 głos
odpowiedź 15 lutego 2016 przez Vizooo Początkujący (380 p.)
jestem też takim początkujacym ale rzuca mi się w oczy jedna rzecz, mianowicie

<aside id="description" class="description-css">

po co używasz diva i klasy na jednym elemencie? Po co? nie lepiej napisać

<aside class="description">?
komentarz 15 lutego 2016 przez radek024 Szeryf (77,180 p.)
Myślę że chodzi Ci o id, a nie diva. Poprzez id odnoszę się do rzeczy w JSie, a w klasach styluję element. Przy klasach miałbym też utrudniony dostęp do rzeczy, mógłbym zapomnieć dopisać [0] i trudno byłoby się połapać czemu jest błąd.
komentarz 15 lutego 2016 przez Vizooo Początkujący (380 p.)
dzięki,ogarnąłem mniej więcej po co to :P
+1 głos
odpowiedź 15 lutego 2016 przez jpacanowski VIP (101,940 p.)

Taka uwaga, że u mnie na Firefoksie, nie wyświetla się
http://pasteboard.co/1yHi59yr.png

komentarz 15 lutego 2016 przez radek024 Szeryf (77,180 p.)
Obstawiam, że nie masz czcionki Consolas ani Segoe UI - zaraz dodam coś od google'a. Dzięki opinię.
komentarz 15 lutego 2016 przez radek024 Szeryf (77,180 p.)
Możesz spróbować teraz?
komentarz 15 lutego 2016 przez jpacanowski VIP (101,940 p.)
Nadal to samo.
komentarz 15 lutego 2016 przez radek024 Szeryf (77,180 p.)
Poprawione - okazuje się, że Mozilla nie obsługuje innerText.

Podobne pytania

+1 głos
1 odpowiedź 252 wizyt
pytanie zadane 20 marca 2022 w Rozwój zawodowy, nauka, praca przez Martita Bywalec (2,470 p.)
0 głosów
2 odpowiedzi 377 wizyt
0 głosów
2 odpowiedzi 162 wizyt
pytanie zadane 4 marca 2017 w HTML i CSS przez lghjufbjd Początkujący (370 p.)

92,452 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!

...