Meine JQuery existieren Funktion: macht es Sinn oder nicht?

Question

Ich bin ein bisschen verwirrt die letzten paar Tage. Ich benutze meine JQUERY-Selektoren wann immer ich möchte ... aber ich habe nicht überprüft, ob ein Selektor existiert oder nicht, stattdessen habe ich die .each-Funktion benutzt.

var exist = function(obj){ 
    var returnObject ={}; 
    for(var key in obj){ 
     if(obj[key].length){ 
     returnObject[key] = obj[key]; 
     }else { 
     return false; 
     } 
    } 
    return returnObject; 
    } 
    //define all your selectors that would be needed for core functionality. 
    var activeSelectors = exist({ 
    selList : $('div.selectorone'), 
    selTag : $('a#tagtwo'), 
    selFloat : $(div.float) /*etc etc*/ 

    }) 

    if (activeSelectors) { 
    console.log('all my core selectors are here!'); 
    /* do Stuff*/ 
    } 

Ich weiß, das viel ein wenig aussieht, vor allem, wenn Sie nur einem Wähler brauchen, aber ich kann keine bessere Art und Weise (mit Ausnahme eines lame if-Anweisung in jedem Wähler) herauszufinden. Ich sah Menschen

mit

$('div#mySelector').each(function(){ /* do stuff*/});

aber ich bin nicht einverstanden, dass es schön ist. Beachten Sie, dass #mySelector (weil es eine ID ist) nur einmal erlaubt ist.

Ich würde das Feedback lieben. Bitte beachten Sie die Leistung vs Nizza Programmierung.

für weitere Informationen, bitte kommentieren Sie unten oder kontaktieren Sie mich!

Answer 1

Wenn ich wirklich nur mit einem einfachen if Aussage vermeiden wollte, dann würde ich wahrscheinlich mit einer einfachen Funktion wie diese gehen nur:

var exists = function() 
{ 
    for (var i in arguments) 
    { 
     if ($(arguments[i]).length == 0) 
     { 
      return false; 
     } 
    } 

    return true; 
} 

Und rufen Sie es wie folgt aus:

var list = $('div.selectorone'); 
var tag = $('a#tagtwo'); 
var float = $('div.float'); 
if (exists(list, tag, float)) 
{ 
    // Do some stuff. 
} 

Oder:

if (exists('div.selectorone', 'a#tagtwo', 'div.float')) 
{ 
    // Do some stuff. 
} 

ich glaube, Sie Over-Engineering, das Problem aber. Alles, was Sie wirklich tun müssen, ist die Eigenschaft length für jede der von Ihnen getroffenen Auswahlen (d. H. Die Variablen list, tag, float) zu überprüfen.

Auch Leistung ist ein komplettes Nicht-Problem hier; ist die Methode zu überprüfen, ob die Elemente tatsächlich die Benutzererfahrung für Ihre Website beeinflussen? Als Donald Knuth sagte: vorzeitige Optimierung ist die Wurzel aller Übel:

Wir sollten etwa 97% der Zeit über kleine Effizienz, sagen vergessen.