如何使用Express和Mongoose发布和获取每个登录用户的唯一数据?

时间:2013-04-27 19:45:14

标签: javascript node.js mongodb express mongoose

我目前正在开发一个小型单页应用,允许用户使用PassportJs和Mongoose登录。

我尝试做的其中一件事是允许用户登录,每个用户都有一个唯一的待办事项/任务列表,这些是与该用户相关联的项目。

我已经能够完成第一部分...用户可以使用jade#{user.username}登录和表达/护照会话,因此登录用户时请参阅“欢迎,[user.username]”

现在我添加一个表单(用户登录时可访问),表单显示为undefined。我不确定它是我的Mongoose架构设计还是引起问题的路由。感谢您阅读本文,这是我的代码:

Mongoose Schema

mongoose.connect('mongodb://localhost/poplivecore')
var Schema = mongoose.Schema;
var ObjectId = Schema.ObjectId;

var user = new Schema({
username: String,
password: String,
email: String,
todos: [Todo]
});

var Todo = new Schema({
name: {type: String, default : ''},
user: {type: Schema.ObjectId, ref: 'user'},
createdAt  : {type : Date, default : Date.now}

})


var Todo = mongoose.model('Todo', Todo);
var user = mongoose.model('user', user);

以下是我的快速路线:     //WORKING....这条路线是登录用户看到的路线,用

形成帖子
app.get('/home', ensureAuthenticated ,function(req, res){
res.render('home', { user: req.user});
});

// WORKING ...此路线允许用户发布/提交登录

app.post('/login', 
passport.authenticate('local', { failureRedirect: '/login', failureFlash: true }),
function(req, res) {
res.redirect('/home');
});


//WORKING....This route allows user to create a user/account    

app.post('/create', function(req, res, next){
var user = new user({
"username": req.body.username, 
"password" : req.body.password,
"email" : req.body.email});


user.save(function (err) {
if (!err) {
  res.redirect('/home');
}
else {
  res.redirect('/');
  }
 });
});



**//NOT WORKING..Post used in the form inside the logged in Area, that adds a 'todo'** 

app.post('/todo', function(req, res){
var todo = new todo(req.body.name);

todo.save(function (err) {
if (!err) {
  res.redirect('/home');
}
else {
  res.redirect('/fail');
  }
 });
});

Jade Form,用于添加待办事项

enter code here
form(method='post', action='/todo')
 //input(type='hidden', value= user._id)#userId
 fieldset
  label Todo
   div.input
    input(name='todo.name', type='todo.name', class='xlarge')
   div.actions
    input(type='submit', value='Save', class='btn primary')
    button(type='reset', class='btn') Cancel

如果您需要查看更多代码,我可以在github上发帖...谢谢。

根据'numbers1311407'建议更新 * todo的新发布路线,也在模式和路线中将待办事项改为“Todo” *

app.post('/todo', function(req, res){
var todo = new Todo({name : req.body["Todo.name"]});


 todo.save(function (err) {
 if (!err) {
  res.redirect('/home');
 }
 else {
  res.redirect('/fail');
 }
 });
});

1 个答案:

答案 0 :(得分:2)

这里至少有两个问题导致这种问题无效:

  1. 表单传递的输入名称为todo.name,您在路线中将其引用为req.body.name

  2. mongoose模型使用属性对象进行实例化,但是你只是给它一个字符串(实际上,由于第一个问题,它实际上是null)。

  3. 因此,对于您的工作路线,它看起来更像是这样:

    app.post("/todo", function (req, res) {
      var todo = new Todo({name: req.body["todo.name"]});
      todo.user = req.user._id;
      // ...
    });
    

    如果要将todo属性作为参数对象传递,则需要使用括号todo[name]命名它们,而不是点。这将导致todo属性在req.body上的对象上,例如:

    app.post("/todo", function (req, res) {
      console.log(req.body.todo); //=> { name: "whatever" }
      // ... which means you could do
      var todo = new Todo(req.body.todo);
      todo.user = req.user._id;
      // ...
    });
    

    您可能想要更改的其他一些事项:

    1. 正如@NilsH指出的那样,您不希望在表单中传递用户ID,因为只要知道他们的ID,任何人都可以为其他人做出todo。而是因为您使用的是护照,请在会话中使用该用户。您应该通过护照确定的用户访问用户ID,例如req.user._id。我在上面的两个例子中都加了这个。

    2. 表单输入的typetodo.name。它应该是text(这就是浏览器无论如何都要对它进行处理)。

    3. 不一定是错误,但模型名称通常是大写的。这也解决了您的代码所遇到的问题,因为当您说todo时,您正在重新定义var todo = new todo(...)