1

我写了以下内容，由于某种原因，当我尝试从集合中删除项目时，它为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;

});
4

1 回答 1

7

模型没有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名为. windowself仅此就足够了：

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

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

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

我更喜欢使用_thisorthat代替，因为如果你忘记了in或者你不小心忘记了声明，可能会导致self有趣的错误。是的，我很难学到这一点。window.selfvarvar self;self

于 2012-10-28T20:25:20.720 回答