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

• #3129 Jim-Y veterán Sk8erPeter #3128

  Új Válasz 2015-03-05 15:30:06 #3129

  Ú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

  

  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