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

• #7887 cattus őstag btz #7886

  Új Válasz 2020-05-19 00:25:52 #7887

  Ú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

  

  cattus

  őstag

  válasz btz #7886 üzenetére

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

  Do the thing!

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