Fts 童鞋的思路是很nice的，简单粗暴，但是真正的问题并不是打开首页性能会差一点，而是这样做会导致数据库紊乱，categories表中的数据和movies表中的数据不能一一对应

@Fts你这里不用遍历category中的数组，直接pop(id)，可以吗？pop不是直接吧最后一个元素pop出去吗？

//index
exports.index = function (req, res) {
    Category.find({}, function (err, categories) {
        if (err) {
            console.log(err)
        }

        var map = {}
        for (i in categories) {
            map[categories[i]._id] = i
            categories[i].movies=[]
        }
        
        Movie.find({}, function (err, movies) {
            for (i in movies) {
                categories[map[movies[i].category]].movies.push(movies[i])
            }
            res.render('index', {
                title: '',
                categories: categories
            })
        })
    })
}

在index中可以这样写，这样就可以把category中的movies字段删掉，保存电影的时候也不需要修改category表，声明map是为了减少遍历次数，我这样的写法，只需要将categories和movies各遍历一遍即可。这样写的缺点就是打开首页的性能会差一点，优点是更新和删除电影的时候不需要对category表进行任何操作（尤其是更新电影，非常麻烦）

问题并不出在update函数里面.而在save函数里面，因为修改已存在movie的category属性时，你并没有在新的分类中添加这部电影。同时也存在一个问题，你没有从原来的分类中删除这部电影，这导致这部电影依旧存在原来的分类中。