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

• Sk8erPeter nagyúr

  #3154

  #3153 klambi

  2013-01-15 15:25:55

  Ú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

  

  Sk8erPeter

  nagyúr

  #3153 klambi

  A páros, páratlan számok tömbjének kiíratásáig volt kedvem foglalkozni vele:

  http://jsfiddle.net/KpJpK/1/

  Kerüld az onclick()-et, inkább oldd meg úgy, ahogy írtam, kerüld a document.forms.satöbbibaromságokat, inkább használj id-t az elemekhez, és azonosítsd őket egyértelműen, vagy class alapján, vagy ilyesmi.
  Meg használd a JavaScript tömbjeinek metódusait, kényelmesebbé tehetik a kódodat.
  Ezenkívül kerüld az ilyen t, t1, t2 és hasonló fantáziadús változóneveket, később fingod sem lesz róla, az a változó az mit csinál, mi a szerepe, tehát nyugodtan írj jó hosszú, egyértelmű neveket, sokkal hasznosabb, legalább tudod, mire való.
  Kikommenteztem a kódban direkt egy-két részt, hogy lásd, mi változott.
  Ja, és a W3C validátorát is érdemes használni a kódodra, mert elég sok hibát tartalmazott.
  Remélem, innentől menni fog, ne kérd senkitől, hogy csinálja meg teljesen a kódodat, ha elakadtál valamiben, kérdezz rá, mi nem megy, és akkor abban szívesen segítünk. Mondjuk szerintem így is már sok segítség volt a fenti.

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