209 похожих чатов

Всем привет. Вопрос достаточно странный, но все-же нужен совет. Попался один проект

на несте и заметил немного странный подход , а может и нет.
В проекте примерно такая структура:
- user
1. user.controller.ts
2. user.service.ts
3. user.module.ts
- category
1. category.controller.ts
2. category.module.ts
- app.module.ts

и такого около 20 папок и в некоторых просто контроллер и модуль.
Так вот почему бы просто в app.module.ts не добавлять контроллеры и сервисы ?
Какая цель такого подхода?
Дает ли это какую-то производительность или нет?

6 ответов

6 просмотров
Alex-👑 Автор вопроса

Если например создается какая-то либа, то тут как бы логично создавать модуль и потом его экспортировать

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

Alex-👑 Автор вопроса
Alx
Это вообще очень странно, так как модульная систем...

Логика описана в репозиториях , сервисах

Alex 👑
Логика описана в репозиториях , сервисах

Перенеси сервисы и репозиторий, которые отвечают за конкретные вещи в модули что бы в модулях было все что му нужно, а если понадобится экспортнешь их потом

Alx
Перенеси сервисы и репозиторий, которые отвечают з...

Мне так кажется более логичным )) А там да некоторые (очень тупые) контролеры можешь и прям в апп модуле прописать

Похожие вопросы

Обсуждают сегодня

Ready for some fun AND a chance to win TKO Tokens? Join us for exciting minigames in our Telegram group! 🕒 Don’t miss out—games start on today 25 October 2024, at 8 PM! Ge...
Milkyway | Tokocrypto
255
VIP-392 [zkSync] New native USDC market in the Core pool Summary If passed, following the Community proposal “Support native USDC on Venus Core Pool of ZKSync Era” and the a...
Venus Announcements
1
is that okay?
Samurai 🇯🇵
21
How about the project bro Likes the community not that active ?
🅿️abby_FX
19
Guys, Donald Trump or Kamala Harris? It does not matter for Bitcoin in the long run!! 2020 Election: After Joe Biden's victory in November 2020, Bitcoin steadily rose from...
Sabah Shabu
3
checking for Berkeley DB C++ headers... no configure: error: libdb_cxx headers missing, Litecoin Core requires this library for wallet functionality (--disable-wallet to disab...
GNU/Линуксяша-libre (";("
5
@brucezli Any info regarding ETA on updates?
Old Man Tom
10
------------------------------------------------------------------------------ 📢 MAJOR ANNOUNCEMENT: HSUITE ECOSYSTEM UPDATE 📢 @everyone Dear HbarSuite Community, After month...
Big Stones
3
Is like having a company with great talents, early investors & workers who joined a company but saw that the vision was getting lost so they pushed for a change from within. T...
Randall Roland
2
error: Provider exception on decrypt DDO. Status: Request failed with status code 400 error: INDEXER: Error processMetadataEvents: Error: Provider exception on decrypt...
Bordo Tor
6
Карта сайта