Из-за чего у меня в коде ошибка (node:54008) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'query' of undefined?

Question

[Руслан]

Здравствуйте!
Пытаюсь выбрать людей из БД postgresql таким образом:

const Router = require('express')
const router = new Router()
const mainController = require('../controllers/MainController')

module.exports = function (app) {
    app.get('/', (req, res) => {
        let as = mainController.getAll;
        // console.log(as)
        res.end(as);
    });
}

Код контроллера такой:

const {Worker} = require('../models/models')

class MainController {
    async create(req, res, next) {
        let {fio, department, position, phonenumber} = req.body

        const worker = await Worker.create({fio, department, position, phonenumber})

        return res.json(worker)

    }

    async getAll(req, res) {
        let {fio, department, position, phonenumber} = req.query
        let workers;
        workers = await Worker.findAll()

        return res.json(workers)
        // const workers = await Worker.findAll()
        // return res.json(workers)
    }

}

module.exports = new MainController()

Но вылетает ошибка (node:54008) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'query' of undefined

Comments

[Lynn «Кофеман»]

В этом коде getAll вообще не вызывается.
Так что показывайте настоящий код.

[Руслан]

Lynn «Кофеман», вот этой строчкой же она вызывается - let as = mainController.getAll;

[Lynn «Кофеман»]

Руслан, нет.
Этой строчкой вы присвоили в `as` функцию. Но не вызвали.

[Руслан]

Lynn «Кофеман», поставил вызов let as = mainController.getAll(); В браузер теперь выдается пустой {}, а ошибка в консоли так и осталась

[Lynn «Кофеман»]

Руслан, программируете наугад и в темноте?

Ваша функция getAll ожидает два аргумента, а вы не передали ни одного.

Answer 1

[GeminiX]

res.end(as) -> res.send(as)

Answer 2

[Олег]

Так как с реакот сам знаком только 2 недели отвечу на простой вопрос.
Функции могут возврашать обешание сделать что-то в будущем. Promise.
У промиса два исхода. успех или ошибка.
в полном синтаксисе пишется через some_function_return_promise().then(). catch()

получается запутанная лапша. Внутри async и только в таких функция (не на главном потоке) можно использовать
await. Это сахар, который возмет успешный результат от асинхронной функции (промиса) и будет ждать пока она не завершит работу.
ошибку в этом случае можно ловить через try {. код где есть await } catch () { }

Так как Вы не обернули в подобный код, то в случае ошибки и получаете
UnhandledPromiseRejectionWarning

в вашем примере использовано 2 авайта и вижу две функции с модификатором async
это еще один сахар, который делает неявно из функции промис

В ошибке еще написано
Cannot read property 'query' of undefined

смотрим где же query ? как поле у req значит выше, а где вызывали getAll что передали и т.д.
пока не найдете где забыли правильно проинициализировать