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

• #1806 Sk8erPeter nagyúr trisztan94 #1805

  Új Válasz 2013-01-21 21:09:23 #1806

  Ú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

  

  Sk8erPeter

  nagyúr

  válasz trisztan94 #1805 üzenetére

  if($('#SpecifyCodeSnippetLanguage').css('display', 'none'))
  Ez a feltétel eleve értelmetlen, mivel itt a display-nek none értéket adsz, és vizsgálgatod a visszatérési értékét.

  Így lenne értelmes:

  if($('#SpecifyCodeSnippetLanguage').css('display') === 'none')

  Ezenkívül inline-block jelenleg a szövegmező, tehát a sima inline-ra való vizsgálgatás nem lenne jó.

  Még valami, simán rossz kódolási szokással kapcsolatos megjegyzés/javaslat:

  $('#SpecifyCodeSnippetLanguage').css('display')
  ezt egy függvényen belül nem érdemes többször hívogatni, mivel túl sok overheadet jelent.

  Így lehet elkerülni az overheadet:

  var $customCodeSnippetLanguageElement = $('#SpecifyCodeSnippetLanguage');
  var $customCodeSnippetLanguageElementVisibility = $customCodeSnippetLanguageElement.css('display');
  
  if($customCodeSnippetLanguageElementVisibility === 'none'){
  // ...
  } else if($customCodeSnippetLanguageElementVisibility === 'inline-block'){
  // ...
  }

  így nem lesz túl sok felesleges függvényhívásod.

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