Проверка формы Node.js дает Невозможно прочитать свойство 'profileimage' из неопределенного - PullRequest
1 голос
/ 03 апреля 2019

Я пытаюсь создать форму в Node.js, используя Express, следуя инструкциям.Я пишу логику проверки, которая должна показывать сообщения об ошибках, если форма отправляется пустой, однако вместо этого она выдает мне эту ошибку в строке 33, и я не могу понять, почему:

Cannot read property 'profileimage' of undefined

Iотладил это, но не мог понять, что является причиной этого.Что я делаю не так, как я могу это исправить?

Вот код формы:

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

/* GET users listing. */
router.get('/', function(req, res, next) {
  res.send('respond with a resource');
});

router.get('/register', function(req, res, next) {
  res.render('register', {
      'title': 'Register'
  });
});

router.get('/login', function(req, res, next) {
  res.render('login', {
      'title': 'Log In'
  });
});

router.post('/register', function(req, res, next) {
  var name = req.body.name;
  var email = req.body.email;
  var username = req.body.username;
  var password = req.body.password;
  var password2 =  req.body.password2;


  // Check for Image Field
  if(req.files.profileimage){
      console.log('uploading File...');

      // File Info
      var profileImageOriginalName = req.files.profileimage.originalname;
      var profileImageName = req.files.profileimage.name;

      var profileImageMime = req.files.profileimage.mimetype;
      var profileImagePath = req.files.profileimage.path;
      var profileImageExt = req.files.profileimage.extension;
      var profileImageSize = req.files.profileimage.size;
  } else {
      // Set a Default Image
      var profileImageName = 'noimage.png';
  }

    // Form Validation

    req.checkBody('name','Name field is required').notEmpty();
    req.checkBody('email','Email field is required').notEmpty();
    req.checkBody('email','Email not valid').isEmail();
    req.checkBody('username','Username field is required').notEmpty();
    req.checkBody('password','Password field is required').notEmpty();
    req.checkBody('password2','Password do not match').equals(req.body.password);

    // Check for errors
    var errors = req.validationErrors();

    if(errors){
        res.render('register', {
            errors: errors,
            name: name,
            email: email,
            username: username,
            password: password,
            password2: password2
        });
    } else {
        var newUser = new User({
            name: name,
            email: email,
            username: username,
            password: password,
            profileImage: profileImageName
        });

            // Create User
            User.createUser(newUser, function(err, user){
                if(err)throw err;
                console.log(user);
            });

            //Success Message
            req.flash('success', 'You are now registered and may log in');

            res.location('/');
            res.redirect('/');
    }
});

module.exports = router;

и мой app.js:

var express = require('express');
var path = require('path');
var logger = require('morgan');
var expressValidator = require('express-validator');
var cookieParser = require('cookie-parser');

var session = require('express-session');
var passport = require('passport');
var LocalStrategy = require('passport-local').Strategy;
var bodyParser = require('body-parser');
var multer = require('multer');
var flash = require('connect-flash');
var mongo = require('mongodb');
var mongoose = require('mongoose');
var db = mongoose.connection;


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

var app = express();

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


// Handle file uploads
var multer = require('multer');
var upload = multer({ dest: './uploads' });

// 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 }));

// Handle Express Sessions
app.use(session({
    secret:'secret',
    saveUninitialized: true,
    resave: true
}));

// passport
app.use(passport.initialize());
app.use(passport.session());

// Validator
app.use(expressValidator({
  errorFormatter: function(param, msg, value) {
      var namespace = param.split('.')
      , root    = namespace.shift()
      , formParam = root;

    while(namespace.length) {
      formParam += '[' + namespace.shift() + ']';
    }
    return {
      param : formParam,
      msg   : msg,
      value : value
    };
  }
}));

app.use(cookieParser());
app.use(express.static(path.join(__dirname, 'public')));


app.use(flash());
app.use(function (req, res, next) {
  res.locals.messages = require('express-messages')(req, res);
  next();
});

app.get('*', function(req, res, next){
    res.locals.user = req.user || null;
    next();
});

app.use('/', routes);
app.use('/users', users);

// 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.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;

Ответы [ 2 ]

1 голос
/ 03 апреля 2019

Прежде всего files должен быть массив . Если вы уверены, что отправляете только один файл, попробуйте этот блок кода:

  // Check for Image Field
  if(req.files && req.files[0] && req.files[0].profileimage){
      console.log('uploading File...');

      // File Info
      var profileImageOriginalName = req.files[0].profileimage.originalname;
      var profileImageName = req.files[0].profileimage.name;

      var profileImageMime = req.files[0].profileimage.mimetype;
      var profileImagePath = req.files[0].profileimage.path;
      var profileImageExt = req.files[0].profileimage.extension;
      var profileImageSize = req.files[0].profileimage.size;
  } else {
      // Set a Default Image
      var profileImageName = 'noimage.png';
  }
0 голосов
/ 03 апреля 2019

В вашем коде есть следующая строка:

if(req.files.profileimage){

когда files не определено, что означает, что req не имеет свойства files (может быть, файл не был отправлен по этому запросу?), Тогда попытка доступа к чему-либо внутри него приведет к ошибке (и аварийному завершению ваш узел, если вы его не ловите)

Итак, сначала вы должны добавить безопасность типов следующим образом:

if(req.files && req.files.profileimage) {

или, еще строже, вот так:

if(typeof req.files === 'object' && req.files.profileimage) {

(Примечание: вы должны выполнять проверку безопасности типа тогда и только тогда, когда другая часть вашего кода еще не гарантировала, что она безопасна, например, проверка безопасности предыдущего типа, или вы явно указали свойство в коде)

Я также вижу, что вы используете multer для получения объекта files в вашем req объекте. Согласно его документам, files может быть массивом или объектом строк в массивах, со страницы npm "multer":

Если используется так:

app.post('/photos/upload', upload.array('photos', 12), function (req, res, next) {

Это будет массив:

req.files - это массив photos файлов

req.body будет содержать текстовые поля, если они были

Или, если используется так:

var cpUpload = upload.fields([{ name: 'avatar', maxCount: 1 }, { name: 'gallery', maxCount: 8 }])
app.post('/cool-profile', cpUpload, function (req, res, next) {

тогда это будет объект (String -> Array):

req.files - это объект (String -> Array), где fieldname является ключом, а значение является массивом файлов

, например

req.files ['avatar'] [0] -> Файл

req.files ['gallery'] -> Array

Так что вы должны соответствующим образом изменить свой код. Например, если вы выберете второй метод, вам следует перейти к следующему:

if(typeof req.files === 'object' && req.files.profileimage) {
  req.files.profileimage.forEach((profileimage) => {
    //do anything you previously did on req.files.profileimage, on profileimage
    let profileImageMime = profileimage.mimetype;
    //... more things and code...
  })
}
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...