А кто должен по твоему? я не особо спорю с

этим т.к. не уврен в этом подходе

16 ответов

18 просмотров

никто не должен. у тебя есть ивент "что-то произошло" - тот кто ивент слушает решает что с ним надо делать

давай по правильному — какую задачу решаешь?

izucken- Автор вопроса
Sergey P
никто не должен. у тебя есть ивент "что-то произош...

Тогда в горизонтальном модуле будет для сотни эвентов написано что с ними делать

izucken
Тогда в горизонтальном модуле будет для сотни эвен...

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

izucken- Автор вопроса
Sergey P
да, для сотни ивентов будет написано что с ними де...

одинаково вывести кастомное сообщение для каждого типа, но это не просто "лог"

izucken- Автор вопроса
izucken
одинаково вывести кастомное сообщение для каждого ...

ну вот ты уточняй что такого непростого в этом "логе"

izucken
Для просто лога и так есть декоратор

скажи что делаешь? какая задача? что такое горизонатльный модуль?

izucken- Автор вопроса
Максим Федоров
скажи что делаешь? какая задача? что такое горизон...

Не претендую на корректность наименований, не могу более подходяшего слова вспомнить. Типо модуль не имеющий прямых зависимостей, но прогоняющий множество других через себя посредством интерфейсов, типо в который можно плагиниться через реализацию его интерфейсов. Кроме эвентов там и другие есть, типо опрос для снмп - каждый модуль для него реализует у себя адаптер если есть что опубликовать.

izucken
Не претендую на корректность наименований, не могу...

ну и обогащай этим модулем свои события в своих модулях... зависимость стабильная, судя по всему это нормально и хоорошо по дизайну

izucken- Автор вопроса
Максим Федоров
ну и обогащай этим модулем свои события в своих мо...

Я не уверен что правильно понимаю. Например ты предлагаешь для аудитов заинжектить логгер в каждый модуль где это нужно?

izucken
Я не уверен что правильно понимаю. Например ты пре...

А понял кажется, ты хочешь события свои закрыть интерфейсом И чтобы все события этого интерфейса модуль переваривал ? Я бы сделал так, чтобы в событие была мета, что событие «logged” или интерфейс повесил И на уровне приложения сделал бы обогатитель, который бы превращал события в новые для твоего лагер-модуля

izucken- Автор вопроса
Максим Федоров
А понял кажется, ты хочешь события свои закрыть ин...

Нюанс в том что для многих таких концернов обогатитель фактически не нужен т.к. все уже есть в евенте

izucken
Нюанс в том что для многих таких концернов обогати...

Ну сделай мету настраиваемой #[Logged(needRich: true)] Или #[Logged(needRich: false)] сквозная логика она такая

izucken- Автор вопроса
Максим Федоров
Ну сделай мету настраиваемой #[Logged(needRich: tr...

Ну грубо говоря тоже самое только без интерфейсов а на атрибутах

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

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

Добрый вечер, Пока не совсем понимаю как наладить общение между телеграм ботом и ПО для работы с сим боксом. По самому боту так понял: - Нужен некий баланс, который можно поп...
Magic
6
сделал сайт, прикрутил в боте сайт, и виджет логина. как автоматически логинить пользователя в аккаунт(телеграм), при входе с бота?
Александра Чернивецкая
5
Объясните, пожалуйста, почему компилятор ругается на использование в условии неинициализированной переменной: int x; Task.Run(async () => { x = await somefunc(); }).Wait...
Александр
5
Ребят, подскажите, пожалуйста, почему в префиксе к ассетам, которые генерируются через фильтр | theme в шаблоне, стал вдруг появляться index.php? Вот так выглядит ссылка на а...
Виталий
1
Всем привет. Ребята, подскажите, пожалуйста. у ботов есть ограничение на отправку сообщений - 30 сообщений в секунду, эти ограничения накладываются на все сообщения? или на со...
Artem Stormageddon
4
Блин, ребята, сори за тупые вопросы. А можно ли как-то открыть вебапку по нажатию на кнопку в меню(которое появляется слева, команды)?
Artem Stormageddon
3
а плаксы из-под питона умеют только в комфортных условиях что-то выдавить из себя?)
Lencore
9
Но, может, есть уже проверенная? Наши требования такие: 1. Сообщения должны приходить из Инста в CRM оду 2. Должна быть возможность подключить несколько экаунтов Инстаграм. Р...
Alexander Sharoiko MSE / Александр Шаройко
13
Это может быть все-таки не флудвейт? у меня ботфазер принимает изменения и отображает даже что они изменились, на видео видно что он прислал якобы уже измененное описание, н...
OVERLINK
13
Коллеги, может знает кто, можно ли цвет бейджа счётчика в BackendMenu менять без бубнов?
Alex Blaze
3
Карта сайта