Új hozzászólás Aktív témák

• #4631 Sk8erPeter nagyúr adam_ #4630

  Új Válasz 2014-12-08 19:36:01 #4631

  Új hozzászólás

  Összes hozzászólása itt Válaszok az összes hozzászólására itt Válaszok erre a hozzászólásra

  Privát üzenet küldése

  

  Sk8erPeter

  nagyúr

  válasz adam_ #4630 üzenetére

  Ami miatt egyáltalán nem fut le a kód, hogy ezt írtad:
  console.log("Thank you! We should race at the next concert!);

  nincs lezárva a string, tehát ez helyesen:
  console.log("Thank you! We should race at the next concert!");

  Az összehasonlítás if (feedback > "8") helyett első körben:
  if (parseInt(feedback, 10) > 8)
  Persze itt semmi értelmes ellenőrzés nincs, normális validáció során megnézed azt is, hogy mondjuk a feedback változó, ami egy string, nem tartalmaz-e nem megengedett karaktereket (pl. betűket, amikor csak számok megengedettek; jelenleg mondjuk ha beírod, hogy "9abc", akkor azt is parse-olni fogja 9-re, de mégsem ellenőrizted, hogy a felhasználó nem gépelt-e be általad nem elfogadott karaktereket, pedig illik, ennek megoldását rádbízom).
  Működik az explicit parse-olás nélkül is, ha > "8" helyett > 8-at írsz, DE szerintem sokkal szebb és kezelhetőbb, ha egyértelműen jelzed a kódban, hogy mi is történik, tehát hogy egy stringből kotorsz ki egy egészszám-értéket.

  Szerk.: egyébként ha rákattintasz a JSHint gombra a jsFiddle-felületen, akkor segíteni is fog, hogy hol van jelenleg elrontva a kódod, érdemes használni, mert így nem akadsz el ilyeneken, hogy egy stringet elfelejtettél lezárni. Meg érdemes figyelni a szintaktika-kiemelésre, mondjuk jelen esetben könnyű volt elsiklani felette.

Új hozzászólás Aktív témák