3

我有一个页面,其中的路线GET /team正在加载团队列表,并且DEL /team正在从/team/:key. 因此,您导航到团队的个人资料页面并从那里删除它们,删除时它应该将您重定向到该/team页面。我已将日志放入控制台,它成功地删除了team并且奇怪的是,它说它正在加载/team但浏览器没有加载它。我把我的代码放在下面,有什么想法吗?

路线:

  app.get('/team'/*, lim("Must be logged in to see teams")*/, getAllTeams, function(req, res){
    util.log('Serving request for url [GET] ' + req.route.path);
    // Pass it the list of all Teams
    res.render('team', {'teamsList' : req.teamsList} );
  });

  app.get('/team/:key', function(req, res) {
    util.log('Serving request for url [GET] ' + req.route.path);
    Team.findByKey(req.params.key, function(err, teamData){
      if(!err && teamData){
        teamData = teamData;
        res.render('teamDetails', { 'teamData' : teamData } );
      } else {
        util.log('Error in fetching Team by key : ' + req.params.key);
        res.json({
          'retStatus' : 'failure',
          'msg' : 'Error in fetching Team by key ' + req.params.key
        });
      }
    });
  });

  /**
    * DEL /team/:key
    * Delete Team by key
    */
  app.del('/team/:key', getAllTeams, function(req, res) {
    util.log('Serving request for url [DEL] ' + req.route.path);    
    Team.remove({key : req.params.key}, function(err){
      var message = '';
      var retStatus = '';
      if (!err) {
        util.log('Successfully deleting Team with key: ' + req.params.key);
        message = 'Successfully deleting Team with key: ' + req.params.key;
        retStatus = 'Success';
        res.redirect('/team');
      } else {
        util.log('Error deleting Team with key: ' + req.params.key + 'Error: ' + util.inspect(err));
        res.json({
          'retStatus' : 'failure',
          'msg' : 'Error in fetching Team with key ' + req.params.key
        });
      }
    });
  });

JavaScript + HTML 模板:

        button#teamDelete.btn.btn-danger.btn-mini(type="submit", value="Delete Team") Delete
        script(type='text/javascript')
            $('#teamDelete').live('click',function(){ 
                var teamId = #{teamData.key};
                $.post('/team/' + teamId, { _method : 'delete' }, function(response) {
                    console.log(response);
                    if(response.retStatus === 'Success') {
                        if('/team' && '/team' !== "") {
                            window.location = '/team';
                        }
                    }
                });
            });

控制台日志:

10 Mar 11:52:01 - Serving request for url [GET] /team
10 Mar 11:52:02 - Serving request for url [GET] /team/:key
10 Mar 11:52:03 - Serving request for url [DEL] /team/:key
10 Mar 11:52:03 - Successfully deleting Team with key: 1362855941128
10 Mar 11:52:03 - Serving request for url [GET] /team

获取所有团队:

var getAllTeams = function(req, res, next){
  Team.getAll(function(err, teamsList){
    if(!err && teamsList){
      req.teamsList = teamsList;
      return next();
    }
  });
};

Team.getAll(团队架构)

Team.statics.getAll = function(cb){
    var query = this.find({});
    query.sort({key : -1});
    return query.exec(cb);
  };
4

2 回答 2

2

为了快速解决方法,只需将重定向 url 添加到响应中,然后在客户端执行以下操作:

if (redirectUrl && redirectUrl !== "")
    window.location = redirectUrl;
于 2013-03-10T14:08:09.100 回答
2

你的请求是 POST ($.post),你的路由是 app.del,所以它永远不会到达 app.del 路由中的 res.redirect。

为什么不使用 app.post?

更新:

假设 $.post 在这里发送 HTTP DEL 请求发生了什么:服务器发送没有数据的 302 响应,但浏览器从不向 GET 路由发送另一个请求,因为服务器指示它(或者 jQuery 是否也处理重定向?不确定)。res.redirect() 是实际的 HTTP 响应,而不是一些内部服务器端指令,用于将请求重新路由到另一个路由,就像您在 ASP.NET 中可以做的那样(实际上这是错误的)......路由旨在接收请求,回复回复并忘记它。您需要将路由与处理它们的实际函数分开,然后您将能够调用该函数而不是发送重定向。

代码建议

在 app.del('/team/:key' ...

...
    retStatus = 'Success';
    // res.redirect('/team');
    res.send({
      retStatus : retStatus,
      redirectTo: '/team',
      msg : 'Just go there please' // this should help
    });
...

客户端在 $.post('/team/' ...

...
    $.post('/team/' + teamId, { _method : 'delete' }, function(response) {
        console.log(response);
        if(response.retStatus === 'Success') {
            // not sure what did you mean by ('/team' && '/team' !== "")
            // if('/team' && '/team' !== "") {
            if (response.redirectTo && response.msg == 'Just go there please') {
                window.location = response.redirectTo;
            }
        }
    });
...

不确定它是否会起作用,因为我不明白你的getAllTeams工作以及你为什么存储teamList在 req.xml 中。如果你想存储在会话中,而不是假设中间件配置正确,你需要使用 req.session。如果您只需要在请求中存储它并且您的 getAllTeams 准备了此团队列表,则最好将其存储在 res.locals(如 res.locals.teamList)中。并确保您的 getAllTeams 接下来调用。所以基本上你的 getAllTeams 应该是这样的:

function getAllTeams (req, res, next) {
    res.locals.teamList = [/* whatever */];
    next();
}

然后你可以在你的路由处理程序中使用 res.locals.teamList 而不是 req.teamList。

res.render('team', {teamsList : res.locals.teamsList} );

而“团队”模板也可能有问题......

表达建议:)

此外,您使用 Express 的方式也使得扩展/管理应用程序变得非常困难。我不记得确切的位置,但是他们在文档中的某个地方写道,Express 应该用作您的应用程序框架的基础,而不是像大多数 PHP 框架那样作为一个完整的框架。它为您提供了强大的功能和灵活性,但也使您有必要提前考虑您的应用程序架构。

express 最强大的功能是您可以让许多特定于路由的处理程序/中间件处理任何路由,并通过 next() 将控制权传递给彼此。我有一个静态表,它定义了在每条路由上使用哪些处理程序,允许在一页上查看整个应用程序,其中包含 30 条左右的路由。并且该表用于在应用程序启动时动态组装路由。它带来了很大的灵活性、可管理性(我可以在路由之间移动/复制粘贴处理程序 - 每个处理程序都表示为一个单词)和代码重用。我还在客户端中使用相同的路由定义散列进行客户端路由。

于 2013-03-10T17:53:44.367 回答