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

Ocena kodu JavaScript

Cloud VPS
0 głosów
481 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 (607,100 p.)
komentarz 18 lutego 2017 przez Benek Szeryf (93,190 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 (607,100 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 266 wizyt
pytanie zadane 8 lutego 2020 w JavaScript przez Dev26 Nowicjusz (130 p.)
+3 głosów
1 odpowiedź 602 wizyt
pytanie zadane 13 sierpnia 2017 w Java przez Wi_ktos Bywalec (2,950 p.)
0 głosów
2 odpowiedzi 338 wizyt
pytanie zadane 29 sierpnia 2017 w HTML i CSS przez DraveS Początkujący (300 p.)

93,487 zapytań

142,423 odpowiedzi

322,773 komentarzy

62,910 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

Kursy INF.02 i INF.03
...