Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Сервисный слой убежал #1077

Open
celestora opened this issue Jan 1, 2024 · 0 comments
Open

Сервисный слой убежал #1077

celestora opened this issue Jan 1, 2024 · 0 comments

Comments

@celestora
Copy link
Member

Не помню, обсуждали это ещё или нет, но в опенвк отсутствует сервисный слой, вся бизнес-логика находится в контроллерах и API-контроллерах, где она разумеется дублируется.

Предлагаю отрефакторить и вынести всю бизнес-логику из контроллеров в сервисный слой, который в этих контроллерах подёргивать. АПИ станет проще поддерживать, уберётся путаница в коде. Ещё, вероятно, стоит переписать шаблоны, но это уже другая тас очка будет. Минусы? Мне кажется at this point рефакторинг займёт пару лет и вряд-ли целесообразен, но у нас весь проект нецелесообразен, так что мало ли, может кому будет цiкаво над этим поработать когда-нибудь через 20 лет в тернополе.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant