2016-06-30 9 views
0

Ich hatte eine if-Anweisung in einer Bildlauffunktion, um zu überprüfen, ob die div #homepage-flag oder div #landingspage-flag existiert. Es war so etwas wie dieses:If-Anweisung funktioniert nicht nach Minimierung

$(window).scroll(function(e){ 
    if($('#homepage-flag').length > 0 || $('#landingspage-flag').length > 0){ 
     // code 
    } 
}); 

ich minimierte den Code, aber jetzt ist die if-Anweisung funktioniert nur für die div #landingspage-flag:

$("#homepage-flag").length > 0 || $("#landingspage-flag").length > 0 && $(window).scroll(function(l) { 
    // code 
}) 
+3

Was minifier verwenden Sie? Das sieht völlig falsch aus. – Pointy

+1

Warte, du sagst die verkleinerte Version ist das zweite Code-Snipplet? Weil das aufgebockt ist. Randnotiz: 'if ($ ('# homepage-flag, # landingspage-flag'). Length)' würde für einen if statt zwei Bedingungen funktionieren. Es sollte jedoch auch gesagt werden, dass Sie DOM-Lookups in einem Scroll-Event-Handler wirklich vermeiden sollten, da dieser ALOT auslöst. – Taplar

+0

Ich benutzte http://jscompress.com/. Ich habe es öfter benutzt, und jedes Mal, wenn es gut war, bis jetzt –

Antwort

1

Der minimierte Code völlig falsch ist.

$(window).scroll(function(g){$("#homepage-flag").length>0||$("#landingspage-flag").length>0}); 
+0

Oh verdammt, sehr schlampig in der Tat zu minimieren. Eine Frage, wo lege ich meinen Code ab? –

+0

Der gleiche Ort, den Sie es nennen würden, wenn es nicht minimiert wurde. –

+1

$ (window) .on ('scroll', Funktion (e) {if ($ ('# homepage-flag'). Länge> 0 || $ ('# landingspage-flag'). Length> 0) {//Ihr Code geht hier}}); –

0
if($('#homepage-flag').length > 0 || $('#landingspage-flag').length > 0){ 
    // code 
} 

wird: Die minimierte Version sollte so etwas wie

if($("#homepage-flag").length>0||$("#landingspage-flag").length>0)//code; 
Verwandte Themen