2016-04-30 13 views
0

私はこれに関するすべての答えを読んだようですが、何も動作していないようです。ここでexpressjs4.xでreq.bodyが定義されていません

は私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; 

である多くのSOの回答で提案されているように私はapp.jsファイルで宣言動き回るかなりの時間を費やしてきたが、私は私が持っていると思いますただ幸運だった。確かに何か間違っているが、見つけられない。私は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

あなたのaboutルータに提出するフォームであなたのビューを投稿してください。 – roflmyeggo

答えて

1

content-lengthヘッダーの値が0の状況では、クライアントは要求の本文内にデータを送信していないことがわかります。その時点で、あなたはクライアント側で問題を探す必要があります。なぜなら、あなたは身体内にいくつかのデータがあると予想しているからです。

これは異なる理由があります。これは実際には問題であったコメントの中で、フォーム要素に名前を付けるのを忘れたことです。

関連する問題