Ocena kodu JavaScript

Question

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)

Comments

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

 

Answer 1

• Przycisk "Stop" ma jednostosunkowe opóźnienie.

• Nie stosuj [on…] w kodzie tylko addEventListener.

• Czemu licznik nie jest w main?

• Czemu skrypt nie jest dołączony na końcu body albo z atrybutem [defer]?

• Przyjętą konwencją jest, żeby wszystkie nazwy były po angielsku w kodzie.

• Wszystko siedzi w globalnym scope. Polecam poczytać o IIFE a potem o modułach.

• Formatuj kod po ludzku!

• Poczytaj o konwencjach nazewniczych i ogólnie o codestyle.

• Nie daje się stringa do setTimeout.

Comments

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

 

---

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).

---

Dziękuję za opinię :)

Answer 2

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 :)

Comments

Dziękuję za opinię :)