Нет, не нужно дублировать их между списками. Я не очень

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

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

Если все команды могут повторяться, то хранить все команды в сабмодуле.

Ещё этот механизм со списком вынести в тот же сабмодуль, чтобы в каждом проекте только список был.

Чем ваш вариант отличается?

1 ответов

19 просмотров

ну все правильно. Вам не кажется этот вариант избыточным? Создавать два файла просто чтобы разделить функционал для субмодуля и для файлов с изменениями? Не создаст ли это лишние зависимости, как вы говорили "При обработке команд мы смотрим сначала в локальном списке, а потом в списке сабодуля."?

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

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

Добрый вечер, Пока не совсем понимаю как наладить общение между телеграм ботом и ПО для работы с сим боксом. По самому боту так понял: - Нужен некий баланс, который можно поп...
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
Карта сайта