0

我注意到我的视图的渲染函数被调用了 2 次。这是我的代码：

视图，它是一个集合：

define([
  'jquery',
  'underscore',
  'backbone',
  'mustache',
  'icanhaz',
  'views/spots/Spot',
  'collections/Spots',
  'text!../../../../templates/spots/spots.mustache!strip',
], function($,
            _,
            Backbone,
            mustache,
            ich,
            SpotView,
            Spots,
            SpotsTemplate){
  var SpotsView = Backbone.View.extend({

    initialize: function(){
       var ich = window['ich'],
          spots = ich.addTemplate('spots',SpotsTemplate);

          spots = ich['spots'];

          this.template = spots;

      _.bindAll(this,'render'); 
      var self = this;
      this.collection.bind("all", function() { self.render(); }, this);
      this.collection.fetch(); 
    },
    events: {
        "change": "render"
    },
    render: function(){
      window.counter = window.counter +1;
      console.log('inside render for the ' + window.counter + ' times!');

      this.el = this.template();

      this.collection.each(function (spot) {

        $(this.el).append(new SpotView({model:spot}).render().el);
      }, this);

      console.log($(this.el).children().length);

      return this;
    }
  });
  // Returning instantiated views can be quite useful for having "state"
  return SpotsView;
});

app.js 中的代码，当我尝试显示时

   var  spots = new Spots({model: Spot});

    window.counter = 0 + 0;

    var spots_view = new SpotsView({collection: spots});
    $('#spots').html(spots_view.render().el);

我的输出是：

inside render for the 1 times! 
1 
inside render for the 2 times! 
6

在玩不同的东西时，我注意到它甚至被调用了 3 次。我究竟做错了什么？显然，当结果从服务器带到渲染函数时，这一行：

$('#spots').html(spots_view.render().el);

已经过去了

多谢

4

1 回答 1

2

你的观点是initialize这样说的：

this.collection.bind("all", function() { self.render(); }, this);
this.collection.fetch();

并将fetch重置集合：

当模型数据从服务器返回时，集合将重置。

重置集合将：

[触发] 最后的单个“重置”事件

通过绑定到"all"，集合上的任何事件都将触发render调用。因此，当您明确表示时，您的视图将呈现一次，spots_view.render()并在fetch调用从服务器返回某些内容时再次呈现。

顺便说一句，你有这个：

_.bindAll(this,'render');

所以你不需要使用self和self.render()或提供上下文参数bind，你可以简单地说：

_.bindAll(this, 'render');
this.collection.bind("all", this.render);

你也在你的render：

this.el = this.template();

这绝不是一个好主意。setElement如果您需要更改视图，您应该使用this.el; 这将负责重新绑定事件和更新this.$el。this.el但是，如果您已经放入DOM ，那将无济于事。而不是完全替换el，你应该把你需要的一切都放在里面this.el：

var $content = $(this.template());
this.collection.each(function (spot) {
    var spot = new SpotView({ model: spot });
    $content.append(spot.render().el);
});
this.$el.html($content);

然后您可以清空它并重新渲染它以响应事件而不会出现任何问题。

于 2012-07-23T20:53:10.053 回答