Ukryj ukryty element ... Jak źle?

Question

Jak źle zrobić ukryć ukryty element?

Załóżmy, że element #myelement jest ukryty, a ja wykonuję numer $('#element').hide(). Czy to złe? Czy powinienem najpierw sprawdzić, czy jest on widoczny?

Dzięki!

Answer 1

jQuery sprawdza czy element jest widoczny i ukrywa tylko wtedy, gdy jest to widoczne: https://github.com/jquery/jquery/blob/master/src/css.js#L78

} else if (!values[ index ] && !isHidden(elem)) { 
     jQuery._data(elem, "olddisplay", curCSS(elem, "display")); 
    } 

więc nie trzeba przeprowadzić dodatkowe kontrole. Każda kontrola ręczna pogorszyłaby wydajność (i czytelność).

Answer 2

Jeśli ukryjesz element, który jest już ukryty, nic się nie wydarzy.

Jedynym sposobem, że jest "zły" jest to, że wykonuje kod, który nie musi być wykonany.

Można sprawdzić, czy jest ona ukryta użyciu jest():

if(!$("my-element").is(":visible")) { 
    $("my-element").hide(); 
}) 

Answer 3

Tylko hiding, element nie trzeba by sprawdzić, czy element jest już hidden lub nie, ale jeśli masz sprawę jak wtedy, gdy element jest następnie pokazać jeszcze zrobić coś innego, a następnie można użyć

if ($('your-element').is(':hidden')) { 
$('your-element').show(); 
} else { 
//do something else 
} 

Answer 4

Jeżeli istnieje wybór pomiędzy

$("my-element").hide(); 

i

if(!$("my-element").is(":visible")) { 
    $("my-element").hide(); 
}) 

jest o wiele prostsze (lepiej?), Aby po prostu mieć pierwszą linię dla czytelności sake. jQuery zrobi to, czy to zrobisz, czy nie.

Answer 5

Lepiej sprawdzić widoczność elementu przed jego ukryciem.

$('#element').hide(); 

spowoduje to pewne problemy w przeglądarce Internet Explorer (IE9). Strona czasu nie ładuje się prawidłowo. Niektóre treści zostaną pominięte na stronie.

Poniższy kod jest znacznie lepszy.

if(!$("#myelement").is(":visible")) { 
    $("#myelement").hide(); 
})