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

• btz addikt

  #7888

  #7887 cattus

  2020-05-19 23:50:57

  Ú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

  

  btz

  addikt

  #7887 cattus

  #7887 cattus

  Engedelmeddel írnék néhány feedbacket a kódhoz:
  HTML:
  A `big` HTML tag obsolete, nem javasolt a használata
  <input type="button" /> helyett inkább használjunk <button>-t.
  Nem szerencsés, ha a `name` attribútom ugyanaz több input elemnél is, mivel esetleges szerveroldali feldolgozásnál ez azonosítja az egyes elemeket.

  JS:
  var helyett ajánlott let vagy const használata
  Személyes preferencia, de szerintem nem mindegy, hogy JS-ből milyen módon kérdezed le a DOM elemeket. Pl. te class alapján teszed, én ezt akkor tenném, ha pl. a class-t akarom módosítani. Ebben az esetben inkább document.querySelectorAll('input[type="checkbox"]')-ot használnék (így pl. már eleve ki van szűrve minden input, ami nem checkbox).

  A for ciklus helyett megcsináltam array műveletekkel: fiddle

  +1: érdemes jól formázottan tartani a kódot, sokat segít az olvashatóságon (a JSFiddle tudja automatikusan a jobb felül lévő Tidy gombbal).

  Természetesen épp ezért raktam fel, hogy lássak egy jobb megvalósítást, mert az előző példákból már látszott, hogy ez is sokkal rövidebben, szebben, modernebben megoldható. És jól sejtettem...
  Köszönöm a kódot. Ha másnak is van alternatívája, azt szívesen fogadom 🙂
  

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