Как надстроить общение между классами REST API?

Question

[Антон]

Привет. Я новичок в js, уже 2 дня не могу найти ответ.
Есть два контроллера. Как в одном контроллере обратится к другому и получить нормальный ответ?
Первый:

class categoryController {
    async activ(req, res) {
        db.query(
            `SELECT * FROM chat_cat WHERE chat_id = ?`,
            [req.query.id],
            (err, data) => {
                if (err) return res.json(err)
                return res.json(data)
            })
    }
}

Второй обращаюсь к первому

async groupQuestionIdd(req, res) {
        // const menu = await menuController.main()
        const category = await categoryController.activ(req, (err, data) => {
            if (err) return res.json(err)
            return res.json(data)
        })
        // const categor2 = await fetch(`http://localhost:3000/api/category/groupActiv?id=${req.query.id}`)
        // const resaCategory = await category.json();
        // const resaCategory2 = await category2.json();
        // console.log(resaCategory)
        // console.log(resaCategory2)
        // console.log(menu)
        
        return res.json('data')
    }

Уже по всякому пробовал, получается только через fetch, но это же бред
(В комментах как пробовал, на фото постоянная ошибка)

Подскажите, пожалуйста, хотя бы где копать

Comments

[Дмитрий]

Потому что это плохая идея вызывать методы контроллера из других контроллеров и методов. Ваш activ надо вынести в модель в сервис - куда угодно. Собственно и правило - если возникают какие то трудности в базовых вещах, то скорее всего эти базовые вещи вы используете не правильно

[szQocks]

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

твой метод в классе

async activ(req, res) {
        db.query(
            `SELECT * FROM chat_cat WHERE chat_id = ?`,
            [req.query.id],
            (err, data) => {
                if (err) return res.json(err)
                return res.json(data)
            })
    }

ничего не возвращает, так что что ты ожидаешь в category - большой вопрос

плюс так же ты вызываешь res.json - тут тоже вопросы... но да фиг с ними, надо изучить Js, потом писать, а не наоборот

[Антон]

szQocks, как его изучать то, теорию только чтоли, вот я и изучаю на практике

[szQocks]

Антон, ну на практике изучать - это хорошо, но что бы применять тот или инной функционал js - нужно ведь понимать что он делает ( а что бы понимать что он делает - надо как минимум прочитать о нём, скопировать из документации, потыкаться посмотреть как будет исполняться на какие-то изменения )

например изучить return и что возвращает ( синхронная/асинхронная ) функция по умолчанию, что будет если из функции вернуть там какое-то странно значение например

const f = () => (5,6); // понять что тут вернет, и почему

res.json is not function - базовая ошибка основ, как бы если ты основы не изучил

Answer 1

[Игорь Дмитриев]

Добрый день.
Начну с того, что контроллеры Rest Api одного приложения не должны между собой общаться. У них одно предназначение: предоставить внешним системам интерфейс сервиса через http запросы. Судя по первому контроллеру вы обращаетесь к БД за данными. Поэтому, непосредственная работа с данными должна выполняться отдельным сервисом. А вот уже этот сервис должны использовать rest-контроллеры. Структура в простом виде выглядит так: абстрактный интерфейс - реализация интерфейса - rest контроллер для внешних систем.

Comments

[Антон]

спасибо, я познакомился с nest js и всё стало на свои места