2017-04-10 138 views
1

我对所有这些(包括Javascript回调和ES6)有点新。我正在使用NodeJS + Express + MongoDB。

我打电话给一个Ajax函数来更新一个项目,并且成功的Ajax调用从未完成。

这里是我的Ajax调用(从阵营调用)

editBug : function(bug){ 

    console.log('about to edit bug with these values',bug); 
    $.ajax({ 
     url:'/api/bugs', 
     method: 'PUT', 
     data:bug 
    }) 
    .done((jqxhr) => { 
     console.log('succcess while editing the bug'); 
     this.setState({successVisible : true}); 
    }) 
    .fail((jqxhr) => { 
     console.log('error : ' + jqxhr); 
    }) 
}, 

这里是我的API函数:

app.put('/api/bugs',function(req,res){ 

    //console.log('req',req); 
    console.log('query string : ',req.query); 
    console.log('query params : ',req.params); 
    console.log('query body: ',req.body); 
    let id = new ObjectID(req.body._id); 
    req.body._id = new ObjectID(req.body._id); 

    db.collection('bugs').replaceOne(
     {_id:id}, 
     req.body, 
     function(err,result){ 
      assert.equal(err,null); 
      console.log('Successfull replace!'); 
      res.status(200); 
     } 
    ); 
}); 

Successfull replace!日志在服务器端正确显示。 正面显示about to edit bug with these values正面。但succcess while editing the bug日志不显示在前端,似乎.done调用永远不会执行。

+1

您不提及是否调用了fail()。 –

+0

你使用的是什么版本的jquery? Promise被添加到jquery 1.6 –

+0

@ freedomn -m'失败()'不被调用。 – nicolasdaudin

回答

3

问题是您没有向节点端的浏览器发回任何回应。尝试以下代码段,你应该很好去

另外,我想指出,你应该处理错误。如果出现错误,更新bugs,最好的做法是通知浏览器500状态代码,指示预期的操作失败。我在下面的截图中添加了这方面的内容

app.put('/api/bugs', function(req, res) { 

    //console.log('req',req); 
    console.log('query string : ', req.query); 
    console.log('query params : ', req.params); 
    console.log('query body: ', req.body); 
    let id = new ObjectID(req.body._id); 
    req.body._id = new ObjectID(req.body._id); 

    db.collection('bugs').replaceOne({ 
     _id: id 
    }, 
    req.body, 
    function(err, result) { 
     if (err) { 
     console.log('Failed replace'); 
     res.status(500).end(); // <- We set the response status code and end the request 
     } else { 
     assert.equal(err, null); 
     console.log('Successfull replace!'); 
     res.status(200).end(); // <- We set the response status code and end the request 
     } 
    } 
); 
}); 
+0

太棒了!这工作。我选择了你的答案作为正确答案,因为它比另一个更详细(也是正确的)。非常感谢您的快速帮助!关于处理错误的一点。其实我正在跟着一个教程,这就是为什么我只做'快速'的东西。但你是完全正确的 – nicolasdaudin

3

你不需要在Node.js端结束你的响应对象吗? 尝试添加res.end();或对响应对象的任何回应。另外,你可以使用chrome(或任何其他浏览器)的网络标签来真正看到你的AJAX请求是如何结束的,以查看它们是否挂起或完成。

+0

谢谢。 res.end()确实有效!我没有想到,因为在我使用res.json(doc)之前,它似乎做了一切(关闭流)。 – nicolasdaudin