Hirdetés

Keresés

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

  • btz
    addikt

    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 🙂
    :R

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