2

以下代码显示了一些与我预期不同的行为。


我的期望:

GET /--> 显示“欢迎”并关闭连接

POST /pages--> 增加/记录计数器;显示“在 POST 函数中”,并关闭连接

GET /someRandomPath --> 增加/记录计数器;显示 404 消息


我观察到的:

GET /--> 显示“欢迎”并关闭连接

POST /pages-->没有计数器的增量/日志;显示“在 POST 函数中”,并关闭连接

GET /someRandomPath --> 增加/记录计数器;显示 404 消息


代码:

var express = require('express');
var request_counter = 0;

var app = express()

    .use(express.basicAuth('test', 'test'))

    //serve the root (welcome)
    .get('/', function(req, resp, next) {
        resp.end('welcome');  
    })  

    //  count/log the requests
    .use(function(req, resp, next) {
        console.log('request# ' + (++request_counter));
        next();
    })

    //  serve "/pages"
    .post('/pages', function (req, resp, next) {
        console.log('in the POST function');
        resp.end('in the POST function');
    })

    //  serve 404
    .use(function (req, resp) {
        resp
            .status(404)
            .end('BB: not found')
        ;
    })
;

module.exports = app;

为什么我打电话时计数器不增加/记录POST /pages

我注意到的一件事是,如果我注释掉该//serve the root部分,我会得到我期望的行为。

4

1 回答 1

1

看起来好像您应该在开始定义路线之前定义所有中间位置,如this answer中所述。

您没有显式使用app.use(app.router),但是当您使用app.get时会自动调用它。

知道了这一点,我很可能会将您的代码更改为类似于以下内容:

var express = require('express');
var request_counter = 0;

var app = express()

app.use(express.basicAuth('test', 'test'))

//  count/log the requests for all except '/'
app.use(function(req, resp, next) {

    if (req.path != '/') {
        console.log('request# ' + (++request_counter));
    }

    next();
})

//serve the root (welcome)
app.get('/', function(req, resp, next) {
    resp.end('welcome');  
})  

//  serve "/pages"
app.post('/pages', function (req, resp, next) {
    console.log('in the POST function');
    resp.end('in the POST function');
})

//  serve 404 for all the rest
app.all('*', (function (req, resp) {
    resp
        .status(404)
        .end('BB: not found')
    ;
}))

app.listen(1234);
于 2013-10-29T02:12:54.570 回答