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

Ocena kodu JavaScript

Aruba Cloud PRO i VPS, Openstack, VMWare, MS Hyper-V
0 głosów
307 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 (191,600 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 (591,060 p.)
komentarz 18 lutego 2017 przez Benek Szeryf (90,700 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 (591,060 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

+2 głosów
2 odpowiedzi 175 wizyt
pytanie zadane 8 lutego 2020 w JavaScript przez Dev26 Nowicjusz (140 p.)
+3 głosów
1 odpowiedź 368 wizyt
pytanie zadane 13 sierpnia 2017 w Java przez Wi_ktos Bywalec (2,950 p.)
0 głosów
2 odpowiedzi 208 wizyt
pytanie zadane 29 sierpnia 2017 w HTML i CSS przez DraveS Początkujący (300 p.)

91,286 zapytań

139,951 odpowiedzi

315,179 komentarzy

60,736 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.

...