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

• Sk8erPeter nagyúr

  #4121

  #4116 Kommy

  2014-04-30 17:29:13

  Ú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

  

  Sk8erPeter

  nagyúr

  #4116 Kommy

  Szerintem az van, amit Karma említett, hogy az .on()-t kellene használnod, a selectek szülőelemére "aggatva", a selectekre szűrve az .on()-nak paraméterként megadott selectorral.
  De amit SOHA többé ne csinálj ( mert egészségkárosító hatású), az a (#4117) azonosítójú hsz.-edben látható, és ROSSZ, NAGYON ROSSZ:
  var value=$(this).children('option:selected').val(); // BORZALMAS
  var valueid=$(this).children(':selected').attr("id"); // NE CSINÁLD
  var valuechamp=$(this).children(':selected').attr("champ"); // ROSSZ
  Tudod, hogy van, amikor egy ilyet leírsz, akkor az egy kismacska életébe kerül.

  Totálisan feleslegesen futtatod le újra és újra ugyanazt a metódust, ráadásul egymás alatt:
  $(this).children(':selected')
  --> ennek az értékét tárold el egy VÁLTOZÓBAN, arra találták ki. Ugyanarra az elemre hivatkozol.
  Tehát a fentit cseréld le valahogy így:
  var $self = $(this);
  var $selectedElement = $self.children('option:selected')
  var value=$selectedElement.val();
  var valueid=$selectedElement.attr("id");
  var valuechamp=$selectedElement.attr("champ");

  a lényeg: a többször használt elemeket tárold el változó(k)ban, ne kérd le őket újra és újra, mert erőforrás-igényes, még ha nem is veszed észre.

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