Hirdetés

Keresés

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

  • Jim-Y

    veterán

    válasz Sk8erPeter #3128 üzenetére

    Ezt meg annyival egeszitenem ki, ami kimaradt Sk8erPeter amugy remek osszefoglalojabol, hogy a ; (semicolon) rakasokat sem hasznalod tul kovetkezetesen. Neha kiteszed, neha nem. A best practice az, ha mindig kiteszed. A masik a dangling else.. Tegyuk mar ki a { (curly braces) jeleket!

    if ($(this).hasClass('active')) $(this).find('#loginLogo').html('▲')
    else $(this).find('#loginLogo').html('▼')

    helyett

    if ($target.hasClass('active')) {
    $loginLogo.html('▲')
    } else {
    $loginLogo.html('▼')
    }

    Sot, mivel ez meg mindig nem tul DRY, ezert ami ennel is jobb practice

    var chevron = $target.hasClass('active') ? '▲' : '▼';

    $loginLogo.html(chevron);

    Illetve en biztos, hogy nem hasznalnam a click(), change() stb.. shorthand functionoket event handlingre, mert ha helyette:

    $topLevelElem.on(CLICK, 'eventDelegation', handler);

    Igy hasznalod az esemenykezeloket, akkor vele jar elonyokre teszel szert:

    - egyreszt a CLICK valtozot egyszer fogod letrehozni amit addot esetben le tudsz majd konnyen cserelni TOUCH_END-re peldaul.
    - event delegation

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