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

• adam_ senior tag

  #4633

  #4631 Sk8erPeter

  2014-12-08 19:47:09

  Új Válasz

  Ú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

  

  adam_

  senior tag

  #4631 Sk8erPeter

  #4631 Sk8erPeter

  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.

  Sk8erPeter, Jim-Y Köszönöm, így már minden világos.

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