Question

我写了以下内容，由于某种原因，当我尝试从集合中删除该项时，它会为removeItem函数中的项返回undefined：

Todos = (function(){

//////////////////////////
// 
//  MODEL
// 
//////////////////////////

var TodoModel = Backbone.Model.extend({

    defaults: {
        id: null,
        item: null
    }

});

//////////////////////////
// 
//  COLLECTION
// 
//////////////////////////

var TodoCollection = Backbone.Collection.extend({

    model: TodoModel

});

//////////////////////////
// 
//  VIEW
// 
//////////////////////////

var TodoView = Backbone.View.extend({

    el: $('#todos'),

    itemField: $('#new-item'),

    initialize: function(){
        this.el = $(this.el);
    },

    events: {
        'submit form': 'addItem',
        'click .remove-item': 'removeItem',
        // Debug
        'click #print-collection': 'printCollection'
    },

    template: $('#item-template').html(),

    render: function(item) {
        var templ = _.template(this.template);
        var id = _.uniqueId('todo_');
        this.el.children('ul').append(templ({id: id,item: item}));
    },

    addItem: function(e) {
        e.preventDefault();
        item = this.itemField.val();
        // Call render
        this.render(item);
        // Clear field
        this.itemField
            .val('')
            .focus();
        // Add to collection
        var newItem = new TodoModel({
            item: item
        });
        this.collection.add(newItem);
    },

    removeItem: function(e) {
        var thisid = this.$(e.currentTarget).parent('li').data("id");
        var thisitem = this.collection.get(thisid);
        thisitem.remove();
        // Remove from DOM
        $(e.target).parent('li')
            .fadeOut(300,function() {
                $(this).remove();
            });
    },

    printCollection: function(){
        this.collection.each(function(item) {
            console.log(item.get('item'));
        });
    }

});

//////////////////////////
// 
//  SELF
// 
//////////////////////////

self = {};
self.start = function(){
    new TodoView({collection: new TodoCollection()});
};
return self;

});

Answer 1

模型没有remove方法（除非你自己添加了一个），所以这不起作用：

var thisitem = this.collection.get(thisid);
thisitem.remove(); // <------ this goes boom!

模型确实有destroy方法，所以你可以：

thisitem.destroy();

这将告诉服务器模型已消失，它触发的"destroy"事件将通知集合模型已消失。如果您不想与服务器通信，那么您可以将集合告诉remove模型：

this.collection.remove(thisitem);

这将在不打扰服务器的情况下将其从集合中删除。

切换为this.collection.remove有效：http://jsfiddle.net/ambiguous/8chHf/

虽然我在这里，但你有一个隐藏的问题：

self = {};

当您可能想要分配给名为self的局部变量时，您正在分配全局self（实际上是standard property of window）。这就足够了：

return {
    start: function() {
        new TodoView({collection: new TodoCollection()});
    }
};

如果您愿意，也可以这样做：

var self = {};
self.start = function(){
    new TodoView({collection: new TodoCollection()});
};
return self;

我更喜欢使用_this或that代替self，因为window.self如果忘记var中的var self;会导致有趣的错误或者如果你不小心忘记完全声明self。是的，我很难学到这一点。

Backbone无法从集合中删除项目

1 个答案: