render: function(person) {
var that = this;
this.$el.empty();
this.collection.each(function(person) {
var personView = new PersonView({model: person});
that.$el.append(personView.render().el);
});
return this;
}
Collection#each это всего лишь закаррированный _.each, который третьим аргументом принимает контекст для итератора. То есть можно переписать так:
render: function(person) {
this.$el.empty();
this.collection.each(function(person) {
var personView = new PersonView({model: person});
this.$el.append(personView.render().el);
}, this);
return this;
}
А еще этот кусок будет тормозить тем сильнее, чем больше элементов будет в списке.
Я обычно не делаю по вьюхе на каждый элемент, хватает одной вьюхи на весь список.
Ну по мелочи — отступы, всегда использовать {}, определиться, двойные или одинарные кавычки, не использовать однобуквенные переменные и параметры функций (что это за e — error, exception, event?).
В остальном криминала нет:)
