2016-05-06 5 views
-1

Ich verstehe noch nicht viel über die Leistung von Javascript. Ich habe my_var verwendet und überschreibe es immer wieder in meiner Funktion mit einer each() Schleife wie unten.Ist es die schlechte Übung oder nicht für die Verwendung von Override-Variable in Javascript

function lookupObjs(obj) { 
    var my_var = jQuery; //or $ 
    if (Object.keys(obj).length != 0) { 
     return my_var.each(obj, function (ins, val) { 
      my_var = ins; 
      if (my_var === 'holiday') {  
       console.log("holiday : " + my_var); 
      } 
      if (my_var === 'work') {  
       console.log("work : " + my_var); 
      } 
     }); 
    } else { 
     console.log(obj)  
    } 
} 
+3

Lesbarkeit ist sehr schlecht! – gurvinder372

+0

Oh mein Gott, wie arm ist es? –

+0

Freunde Bitte helfen Sie mir zu erklären, wie man diese Funktion besser macht als das? –

Antwort

2

Sie brauchen nicht my_var in der Schleife außer Kraft zu setzen, können Sie nur verwenden ins:

return my_var.each(obj, function (ins, val) { 
    if (ins === 'holiday') { 
     console.log("holiday : " + ins); 
    } 
    if (ins === 'work') { 
     console.log("work : " + ins); 
    } 
}); 

Es ist auch keine gute Praxis Variablen auf diese Weise außer Kraft zu setzen - es, wenn Sie viel klarer ist nur Erzeuge neue Variablen mit offensichtlichen Namen; Sobald sie außer Reichweite sind, werden sie Müll gesammelt.

+0

Tatsächlich wird my_var gar nicht benötigt. Sie können stattdessen einfach die Variable namens jQuery verwenden. –

Verwandte Themen