将函数传递给 Backbone.View.extend

Question

最近我和一些同事因为一些我不知道的事情发生争执 发现不正确。 我们在大型应用程序中使用 Backbone，我创建视图的方法是 “标准”骨干方式：

var MyView = Backbone.View.extend({ 
  className: 'foo', 
  initialize: function() { 
    _.bindAll(this, 'render' /* ... more stuff */); 
  }, 
  render: function() { 
    /* ... render, usually 
      using _.template and passing 
      in this.model.toJSON()... */ 
    return this; 
  } 
}); 

但团队中的某人最近决定这样做：

var MyView = Backbone.View.extend( (function() { 
  /* 'private stuff' */ 
  function bindMethods(view) { 
    _.bindAll(view, /* ... more stuff */); 
  }; 
  function render(view) { 
     /* ... render, usually 
        using _.template and passing 
        in view.model.toJSON()... */ 
  }; 
  return { 
    className: 'foo', 
    initialize: function() { 
      bindMethods(this); 
      render(this);   
    } 
  }; 
}()); 

这就是伪代码的想法。 阅读了 BB 源代码并阅读了教程、文章后，我发现这是一个 不好的做法（对我来说这没有意义），但我希望得到一些反馈 其他 Backbone 开发者/用户

提前致谢

Answer 1

我看到使用闭包的一个好处是为您不希望从视图外部的代码访问的变量和函数提供私有范围。

即便如此，我还没有看到很多 Backbone 应用程序使用闭包来定义视图/模型/集合等。

这里还有一封来自 Jeremy Ashkenas 的电子邮件，涉及这个问题。

是的，在 JavaScript 中可以使用闭包创建具有私有变量的对象实例。但这是一种不好的做法，应该避免。这与 Backbone 特别无关；这是 JavaScript 中 OOP 的本质。

如果您使用闭包模式（也称为“模块”模式），您将为您创建的每个实例创建每个函数的新副本。这完全忽略了原型，并且在速度方面，尤其是在内存使用方面，效率非常低。如果您制作 10,000 个模型，您还将拥有每个成员函数的 10,000 个副本。使用原型（使用 Backbone.Model.extend），即使该类有 10,000 个实例，您也只会拥有每个成员函数的一个副本。

Answer 2

我完全同意保罗的观点。有时您可能会发现有必要定义私有的并且不能从外部弄乱的方法和属性。我认为这取决于您的班级是否需要这种范围界定机制。就您对课程的要求而言，混合两种方法不会那么糟糕，不是吗？