NodeJS :: TypeError: Cannot read property 'first_name' of undefined

I studied the MEAN stack from the tutorial. When I tried on my localhost I got an error.

TypeError: Cannot read property 'first_name' is undefined

at router.post (/var/www/html/mean/contactlist/routes/route.js:17:28)

I found some similar questions on the internet. But I haven't found the right solution.

Here is my app.js file

//importing modules
var express =  require('express');
var mongoose = require('mongoose');
var bodyparser = require('body-parser');
var cors = require('cors');
var path = require('path'); //core module 


// calling express method
var app = express(); 


//connect to mongodb
mongoose.connect('mongodb://localhost/27017/contactlist');

//on connection
mongoose.connection.on('connected', () => {
    console.log("connected to database database mongodb @ 27017 ");
});

mongoose.connection.on('error', (err) => {

    if(err){
        console.log('Error in Database connection : ' + err);
    }
});

//adding middleware cors
app.use(cors());

//adding body parser
app.use(bodyparser.json());

//adding static files
app.use(express.static(path.join(__dirname, 'public')));

//setting port no 
const port = 3000;


//routing
var route = require('./routes/route'); 

//using the route
app.use('/api', route); 


//testing server
app.get('/', (req, res)=>{

    res.send('foobar');

});

//binding the server with port no (callback)

app.listen(port,() =>{
    console.log('Server Started at Port : '+ port);


});

      

In the stackOverflow solution I found,

I have to use the following line before routing

app.use(bodyparser.json());

      

So I changed it.

And my ./routes/route.js

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

const Contact = require('../models/contacts');

//Retrieving contacts
router.get('/contacts', (res, req, next) => {
    contact.find(function(err,contacts){
        res.json(contacts);
    })

});

//Add contact
router.post('/contact', (res, req, next) => {
    let newContact = new Contact({
        first_name:req.body.first_name,
        last_name:req.body.last_name,
        phone:req.body.phone
    });



    newContact.save((err,contact) => {

        if(err){
            res.json({msg : 'Failed to add contact'});
        }
        else{
           res.json({msg : 'Contact added successfully'}); 
        }

    });
});


//Deleting Contact
router.delete('/contact/:id', (res, req, next) => {
    contact.remove({_id: req.params.id }, function(err, result){

        if(err){
            res.json(err);
        }
        else{
            res.json(result);
        }

    });
});


module.exports = router;

      

Depending on package.json

"dependencies": {
    "body-parser": "^1.17.1",
    "cors": "^2.8.3",
    "express": "^4.15.2",
    "mongoose": "^4.9.8"
  }

      

And the nodejs version is

v7.10.0

      

I used Postman to test the API

So, I tested the POST method and the following content type.

 {"Content-Type":"application/x-www-form-urlencoded"}

      

This was my example input

{ 
   "first_name" : "RENJITH",
   "last_name"  : "VR",
   "phone" :  "1234567890"
}

      

Is this a version issue? Please suggest me the correct way to code.

+4


source to share


2 answers


Your Content Type {"Content-Type":"application/x-www-form-urlencoded"}

To support URL-encoded data bodies, you should use this:

app.use(bodyparser.urlencoded({     // to support URL-encoded bodies
  extended: true
}));

      

What did you use for JSON encoded data like POST: {"name":"foo","color":"red"}

EDIT:



The order of your route parameters is incorrect. Is notrouter.post('/contact', (res, req, next)

In fact router.post('/contact', (req, res, next)

The first parameter is the request, the second is the response.

+2


source


Just move the body-parse lines to the top before the route (app.js)



app.use(bodyparser.json());

app.use('/api',route);

      

+1


source







All Articles