Ist es sinnvoll, if-Anweisungen zu verwenden, um seitenabhängigen Code mit jQuery zu trennen?

Question

wandert dir vor, wenn es sinnvoll ist, zu verwenden, wenn Aussagen wie unten in jquery Dokument-Ready-Funktion wird nur, wenn auf einer bestimmten Seite gefeuert Skript, um sicherzustellen, ..

if(window.location.href.indexOf("our-team")>= 0){ 
      if(window.location.hash) { 
       var hash = window.location.hash; 
       $('#teamMembers div:not('+hash+')').hide(); 
       $("#teamMenu .imgHolder").removeClass('active'); 
       $('a[href='+hash+']').closest(".imgHolder").addClass('active'); 
      } 
      else { 
       $('#teamMembers div:not(#member1)').hide(); 
      } 
     } 

Dies ist wahrscheinlich nicht sehr effizient, aber die if-Anweisung erforderlich, um das Ausführen des Skripts zu stoppen, wenn die aktuelle Seite unser Team nicht in der URL enthält.

Irgendwelche Ideen Leute?

Answer 1

Sie erhalten die beste Leistung, wenn Sie auf Seiten, auf denen diese Elemente nicht vorhanden sind, nicht viele Selektoroperationen ausführen. Wenn es also leicht ist, diese Selektoroperationen durch Überprüfen der URL oder einer anderen schnellen Überprüfung herauszufiltern, kann dies aus Leistungsgründen wünschenswert sein.

Persönlich bevorzuge ich Seitenarten mit einem Klassennamen auf dem Body-Tag zu identifizieren, anstatt die URL zu untersuchen. Das bedeutet, dass Sie eher nach dem erwarteten Inhalt als nach dem Inhalt suchen. Wenn jemand, der die Website verwaltet, beschließt, das URL-Layout oder die URLs zu ändern, hängt Ihr Code nicht davon ab.

if ($(document.body).hasClass("ourteam")) { 
    if (window.location.hash) { 
     var hash = window.location.hash; 
     $('#teamMembers div:not('+hash+')').hide(); 
     $("#teamMenu .imgHolder").removeClass('active'); 
     $('a[href='+hash+']').closest(".imgHolder").addClass('active'); 
    } else { 
     $('#teamMembers div:not(#member1)').hide(); 
    } 
} 

Technisch gesehen, die erste if-Anweisung ist nicht erforderlich, wenn keiner der Wähler Sie in Seiten existieren verwenden, die Sie nicht wollen, zu verarbeiten. Es ist jedoch nicht effizient, 4-5 zusätzliche Selektoroperationen auf vielen Seiten auszuführen, auf die sie nicht angewendet werden. Und wenn jede Seite diese Art von Code im allgemeinen JS hat und somit 4-5 Operationen * 50 Seiten hat, dann wäre das eine Menge verschwendeter Selektoroperationen und könnte die Seitenleistung wirklich beeinflussen. Je mehr Seiten Sie haben, desto größer ist die Wahrscheinlichkeit, dass Sie einen Selektor auf einer anderen Seite wiederholen und feststellen, dass sich der JS von einer Seite aus auch auf eine andere Seite auswirkt.