Где правильно разместить валидатор POST?

Question

[alex4answ]

Добрый день, использую express-validator, не могу решить где лучше разместить валидатор.
Использую такой подход:

Router

middleware уже не встроить.

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

router.route('/')
  .get(userController.getAll)
  .post(userController.create);

Controller:

exports.create = async (req, res, next) => {
  const validator = require('./validators/userCreate'); // Чтобы грузить только при создании
  await validator(req, res);

  try {
    const user = await userService.create(req.body);
    res.status(201).json(user);
  } catch(err) {
    res.status(422).json({ message: err.message });
  }
};

Чтобы валидатор загружался только при создании, я его запрашиваю (require) в конкретном экшене контроллера.

Сам валидатор

const { body, validationResult } = require('express-validator');
modules.exports = validate([ body('email').isEmail(), body('name').isString() ]);

const validate = (validators) => { // хранится в utils, тут для наглядности
  return async (req, res) => {
    await Promise.all(validators.map((validator) => validator.run(req)));

    const errors = validationResult();
    if (!errors.isEmpty) {
      res.status(422).send({ errors: errors.array() });
    } 
     // будь это полноценным middleware, тут было бы next()
  }
};

Все сомнения сводятся к тому, что мой валидатор это вроде и middleware, но разместить его в цепочке я не могу, потому что валидаторы бывают очень объемными и хочется загружать их только при нужном экшене.

На сколько удачно мое решение, или как подобные вещи реализуются?

Comments

[Михаил]

Асинхронная загрузка модулей в ноде оч страннвя идея,да и после 1 require модуль останется в кэше

[alex4answ]

Михаил, почему странная ?, я по быстрому накидал, подразумевалась такая конструкция:

const createValidator = require('./validators/userCreate');
await createValidator(req, res);

Да, после 1 require модуль в кеше, я вкурсе, но зачем мне он в кеше изначально, если еще не был вызван метод

[Михаил]

alex4answ, в идеале задача валидатора это проверка данных и уж никак не ответы на запросы

Я б запихал в middleware и не парился,

А то что вы делаете это больше похоже на преждевременную оптимизацию

[alex4answ]

Михаил,

А то что вы делаете это больше похоже на преждевременную оптимизаци

Согласен, откажусь от этого.

в идеале задача валидатора это проверка данных и уж никак не ответы на запросы

Да, я понимаю, но так же не хочу повторяться постоянно в:

if(validator.hasError) {
  res.status(422).json({ errors: validator.errors }}
}

А почему бы вы запихали в middleware и не парились бы?
если задача валидатора проверка данных, но не ответы на запросы, это же нарушает принцип единой ответственности

Я просто в процессе обучения, и хочу разобраться

[Михаил]

alex4answ, насколько я помню если в next передать ошибку она идёт в конец очереди

app.get()
app.get() -> тут ошибка
app.get()

app.use(errorHandler); <- придёт сюда

проверяете что за ошибка и отправляете клиенту

[Михаил]

alex4answ,

const express = require('express');
const app = express();

const auth = (rules) => (req, res, next) => {
  setTimeout(() => {
    next(new Error('Validation fail'));
  }, 1000)
}

app.get('/', (req, res, next) => {
  console.log('-------')
  console.log('1');
  next()
})
app.get('/', auth(['admin']), (req, res, next) => {
  console.log('2')
  next();
})
app.get('/', (req, res, next) => {
  console.log('3')
  next()
})

app.use((err, req, res, next) => {
  console.log(err)
  res.send('Server / User error');
})

app.listen(3000);

https://expressjs.com/en/guide/using-middleware.ht...

[alex4answ]

Михаил, да, спасибо, создам свою ошибку и в error handler, если она instanceof ValidationError, то отдам 422 с ошибками

[Дмитрий Беляев]

Михаил, alex4answ,

А то что вы делаете это больше похоже на преждевременную оптимизацию

Скорее на деоптимизацию, require блокирующая штука содержащая в себе чтение файла, его разбор и выполнение, при котором снова могут встретится несколько require.