流星/ Javascript - 這是不好的做法？ - 聲明var

Question

在javascript/Meteor中，如果我有多個使用相同類的方法，那麼在方法外部聲明它並重用該類是不好的做法還是很好的做法？

這兩個選項中的哪一個實際上是編碼的正確方法？

即選項1

Meteor.methods({ 
    do1: function() { 
    var facebook = facebook('facebook id'); 
    facebook.something1() 
    }, 
    do2: function() { 
    var facebook = facebook('facebook id'); 
    facebook.something2() 
    } 
}) 

即選項2

facebook = facebook('facebook id'); 

Meteor.methods({ 
    do1: function() { 
    facebook.something1() 
    }, 
    do2: function() { 
    facebook.something2() 
    } 
}) 

Answer 1

是的，這是一個很好的做法，把它放在外面（這樣你不要重複自己），但它是一個不好的做法不要實際宣佈它。

var facebook = facebook('facebook id'); /* 
^^^ use var! */  
Meteor.methods({ 
    do1: function() { 
    facebook.something1() 
    }, 
    do2: function() { 
    facebook.something2() 
    } 
});