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

Ocena kodu JavaScript

VPS Starter Arubacloud
0 głosów
348 wizyt
pytanie zadane 18 lutego 2017 w JavaScript przez Szymon213 Nowicjusz (160 p.)
Witam , uczę się JavaScriptu od niedawna. Stworzyłem prosty licznik w JS w celach edukacyjnych, prosiłbym o jego ocenę i sugestię co można by w nim poprawić.

link: http://polskiesymbolenarodowe.hol.es/Strona-glowna (Nazwa jest niezgodna ponieważ została stworzona  na inną pracę dodatkową na lekcję informatyki)
komentarz 18 lutego 2017 przez ScriptyChris Mędrzec (190,190 p.)

Stosując if korzystaj z potrójnego operatora porównania, gdyż np. dla warunku w funkcji CzyNaPewno z fragmentu Twojego kodu:

 var Niezepsuj=0;
  
  function CzyNaPewno(){ 
  if(Niezepsuj==0){Niezepsuj=1; licznik();}

Jeśli zmienna Niezepsuj będzie mieć jakimś cudem - a z tym różnie bywa, gdyż w JS możesz zmienić typ zmiennej w kodzie - zmienić typ z number na np. boolean lub string, wtedy wyniki instrukcji if mogą sprawiać wrażenie "dziwnych":

false == 0 // true
'' == 0 // true
00 == 0 // true
'00' == 0 // true

 

2 odpowiedzi

+2 głosów
odpowiedź 18 lutego 2017 przez Comandeer Guru (599,730 p.)
komentarz 18 lutego 2017 przez Benek Szeryf (90,690 p.)

Poczytaj o konwencjach nazewniczych i ogólnie o codestyle.

Głupie pytanie, ale czy to rzeczywiście tyczy się wszystkich języków, przykładowo PHP? A co jak mnie się łatwiej czyta:

if ($state)
   $var++;

zamiast:

if ($state) $var++;

https://github.com/airbnb/javascript#blocks

 

1
komentarz 18 lutego 2017 przez Comandeer Guru (599,730 p.)

Głupie pytanie, ale czy to rzeczywiście tyczy się wszystkich języków, przykładowo PHP?

Każdy język ma własny codestyle. W PHP jest praktycznie tylko jeden, PSR-2.

Co do podanego przykładu: akurat w tym wypadku najcześciej pisze się z rozbiciem na linie:

if ( state ) {
    other++;
}

W PHP, zgodnie z PSR-2, jest:

 if ($state) {
    $var++;
}

Ogólnie to zależy od projektu i wybranego codestyle'u (chociaż w PHP praktycznie każdy stosuje PSR-2).

komentarz 18 lutego 2017 przez Szymon213 Nowicjusz (160 p.)
Dziękuję za opinię :)
+1 głos
odpowiedź 18 lutego 2017 przez Dexterim Dyskutant (8,370 p.)
Myślę, że można by dodać 0 z przodu dla wartości mniejszych niż  10 i dodatkowo przycisk reset by można było zacząc od nowa bez odświeżania strony :)
komentarz 18 lutego 2017 przez Szymon213 Nowicjusz (160 p.)
Dziękuję za opinię :)

Podobne pytania

+1 głos
2 odpowiedzi 198 wizyt
pytanie zadane 8 lutego 2020 w JavaScript przez Dev26 Nowicjusz (130 p.)
+3 głosów
1 odpowiedź 413 wizyt
pytanie zadane 13 sierpnia 2017 w Java przez Wi_ktos Bywalec (2,950 p.)
0 głosów
2 odpowiedzi 220 wizyt
pytanie zadane 29 sierpnia 2017 w HTML i CSS przez DraveS Początkujący (300 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!

...