流星/ 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()
  }
});