2016-04-30 105 views
0

我已经看过所有关于这个问题的答案,但似乎没有任何工作。req.body undefined in expressjs4.x

这里是我的app.js

var express = require('express'); 
var path = require('path'); 
var favicon = require('serve-favicon'); 
var logger = require('morgan'); 
var cookieParser = require('cookie-parser'); 
var bodyParser = require('body-parser');  

var about = require('./routes/about'); 

var app = express(); 

// view engine setup 
app.set('views', path.join(__dirname, 'views')); 
app.set('view engine', 'jade'); 

// uncomment after placing your favicon in /public 
//app.use(favicon(path.join(__dirname, 'public', 'favicon.ico'))); 
app.use(logger('dev')); 
app.use(bodyParser.json()); 
app.use(bodyParser.urlencoded({ extended: false })); 
app.use(cookieParser()); 
app.use(express.static(path.join(__dirname, 'public'))); 

app.use('/about', about); 

// catch 404 and forward to error handler 
app.use(function(req, res, next) { 
    var err = new Error('Not Found'); 
    err.status = 404; 
    next(err); 
}); 

// error handlers 

// development error handler 
// will print stacktrace 
if (app.get('env') === 'development') { 
    app.locals.pretty = true; 
    app.use(function(err, req, res, next) { 
    res.status(err.status || 500); 
    res.render('error', { 
     message: err.message, 
     error: err 
    }); 
    }); 
} 

// production error handler 
// no stacktraces leaked to user 
app.use(function(err, req, res, next) { 
    res.status(err.status || 500); 
    res.render('error', { 
    message: err.message, 
    error: {} 
    }); 
}); 


module.exports = app; 

这里是我的about.js

var express = require('express'); 
var router = express.Router(); 

router.get('/', function(req, res) { 
    res.render('about');  
}); 

router.post('/', function(req, res) { 
    console.log("Called"); 
    console.log(req.body); // this is always undefined 

    //below variables are also undefined 
var name = req.body.contactName, 
     email = req.body.contactEmail, 
     message = req.body.contactMessage, 
     subj = req.body.contactSub; 

}); 


module.exports = router; 

我花了相当长的时间周围app.js文件中的声明运动在许多建议SO答案,但我认为我有只是幸运。我当然做错了什么,但找不到它。请注意,我是nodejs世界的相对初学者。

这里是req.headers

{ host: 'debian:3000', 
    connection: 'keep-alive', 
    'content-length': '0', 
    'cache-control': 'max-age=0', 
    accept: 'text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,*/*;q=0.8', 
    origin: 'http://debian:3000', 
    'upgrade-insecure-requests': '1', 
    'user-agent': 'Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/49.0.2623.112 Safari/537.36', 
    'content-type': 'application/x-www-form-urlencoded', 
    referer: 'http://debian:3000/about', 
    'accept-encoding': 'gzip, deflate', 
    'accept-language': 'en-US,en;q=0.8' } 

更新 我错过了加入name我的形式,这引起了这个问题。感谢@ t.niese提出这个建议。

+1

您正在显示标题,但在哪里做你的文章? –

+1

你的代码看起来很好,但请求的'content-length'是'0',你确定你的表单中有任何命名的输入元素吗?你可以使用'curl -i -d“param1 = value1&param2 = value2”http:// localhost:3000/about“直接测试你的代码,并检查控制台是否正确输出params参数。{{param1:'value1',param2: 'value2'}'。 –

+0

请使用提交给您的路由器的表单发布您的视图。 – roflmyeggo

回答

1

content-length标头的值为0的情况告诉您客户端未在请求的正文内发送任何数据。因此,您需要在客户端查找问题,因为您希望正文中有一些数据。

可能有一个不同的原因 - 而且在评论中这实际上是问题 - 是你忘记给表单元素一个名字。