Не удается прочитать свойство 'id' из неопределенного при публикации в локальном API

Question

Это мой первый раз, когда я создаю API, и у меня, похоже, возникла проблема с запросами post / put. Получить работает, как ожидалось. Когда я пытаюсь POST к API, я получаю и ошибка, которая говорит

TypeError: Невозможно прочитать свойство 'id' из неопределенного в Object.addUser (D: \ github \ DC-API \ models \ user.js: 11: 58)

Вот мой код.

user.js

var db = require('../dbconnection');

var User = {  
  getAllUsers: function(callback) {  
    return db.query("select * from users", callback);  
  },  
  getUserById: function(id, callback) {  
    return db.query("select * from users where Id=?", [id], callback);
  },  
  addUser: function(User, callback) {  
    return db.query("Insert into users values(?,?,?)", [User.id, User.username, User.avatar], callback);  
  },  
  deleteUser: function(id, callback) {  
    return db.query("delete from users where Id=?", [id], callback);  
  },  
  updateUser: function(id, User, callback) {  
    return db.query("update users set username=?,avatar=? where Id=?", [User.username, User.avatar, id], callback);  
  }  
};  
module.exports = User;

users.js

var express = require('express');  
var router = express.Router();  
var User = require('../models/user');

router.get('/:id?', function(req, res, next) {  
  if (req.params.id) {  
    User.getUserById(req.params.id, function(err, rows) {  
      if (err) {  
        res.json(err);  
      } else {  
        res.json(rows);  
      }  
    });  
  } else {  
    User.getAllUsers(function(err, rows) {  
      if (err) {  
        res.json(err);  
      } else {  
        res.json(rows);  
      }  
    });  
  }  
});  
router.post('/', function(req, res, next) {  
  User.addUser(req.body, function(err, count) {  
    if (err) {  
      res.json(err);  
    } else {  
      res.json(req.body); //or return count for 1 & 0  
    }  
  });  
});  
router.delete('/:id', function(req, res, next) {  
  User.deleteUser(req.params.id, function(err, count) {  
    if (err) {  
      res.json(err);  
    } else {  
      res.json(count);  
    }  
  });  
});  
router.put('/:id', function(req, res, next) {  
  User.updateUser(req.params.id, req.body, function(err, rows) {  
    if (err) {  
      res.json(err);  
    } else {  
      res.json(rows);  
    }  
  });  
});  
module.exports = router;

app.js

var createError = require('http-errors');
var express = require('express');
var path = require('path');
var cookieParser = require('cookie-parser');
var logger = require('morgan');

var indexRouter = require('./routes/index');
var usersRouter = require('./routes/users');

var app = express();
var cors=require('cors');
var Users=require('./routes/users');

app.use(cors());
app.use('/users',usersRouter);

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

app.use(logger('dev'));
app.use(express.json());
app.use(express.urlencoded({ extended: false }));
app.use(cookieParser());
app.use(express.static(path.join(__dirname, 'public')));

app.use('/', indexRouter);
app.use('/users', usersRouter);

// catch 404 and forward to error handler
app.use(function(req, res, next) {
  next(createError(404));
});

// error handler
app.use(function(err, req, res, next) {
  // set locals, only providing error in development
  res.locals.message = err.message;
  res.locals.error = req.app.get('env') === 'development' ? err : {};

  // render the error page
  res.status(err.status || 500);
  res.render('error');
});

module.exports = app;

Я использую Postman для проверки API, и я отправляю данные через тело как с form-data, так и с необработанным json. Как я уже говорил, это первый раз, когда я создаю API, так что это вполне может быть простым упущением. Заранее спасибо за помощь.

Answer 1

Ваша проблема в том, что req.body не определено всякий раз, когда вы передаете его User.createUser. В свою очередь, ваш параметр User в createUser также не определен, поэтому всякий раз, когда вы пытаетесь получить доступ к id внутри параметра User, вы получаете ошибку Cannot read property 'id' of undefined, поскольку вы не можете получить доступ к свойствам объекта для неопределенных значений .

Чтобы исправить эту проблему, добавьте это в конфигурацию вашего приложения:

var bodyParser = require('body-parser');
app.use(bodyParser.urlencoded({ extended: true }));
app.use(bodyParser.json())

Также, чтобы избежать коллизий типов, я предлагаю сделать все ваши параметры User строчными (user).

Как только вы измените свои параметры на user с User и добавите конфигурацию приложения выше, ваше приложение должно работать правильно.

---

Редактировать: Делая все ваши User параметры строчными, я имею в виду только параметры функции. Вот пример:

 addUser: function(User, callback) {  
   return db.query("Insert into users values(?,?,?)", [User.id, User.username, User.avatar], callback);  
 }

На это:

 addUser: function(user, callback) {  
   return db.query("Insert into users values(?,?,?)", [user.id, user.username, user.avatar], callback);  
 }