Есть бот, у которого команды представляются объектами класса Command, каждая

Question

Есть бот, у которого команды представляются объектами класса Command, каждая

команда определена в своём модуле.
При запуске бота делается os.listdir('commands') и каждый модуль запускается в цикле.
А объекты класса Command при инициализации закидываются в глобальный список команд (это прямо в конструкторе прописано).

Хочется выпилить эти глобальный список и нечистый конструктор.

Можно импортнуть каждую команду вручную вместо этого. Но можно ли сделать как-то лучше?

Есть идея ввести __commands__ = (...) в каждом модуле, и всё так же пробегаться по модулям команд и закидывать оттуда команды, лежащие в __commands__. Насколько это уместно и приемлемо?

#backend #programming #python #russian

0

23.04.2021

9 ответов

19 просмотров

Tishka17

Это очень хрупко

0

23.04.2021

Test Case

Возможно стоит создать некий класс CommandManager, сделать его Singleton"ом, реализовать метод register(command: Command), который будет класть комманды в аттрибут self._commands = [] и сделать getter на _commands. Затем оттуда, откуда нужно забрать все команды вызвать CommandManager и забрать все зарегестрированные.

0

23.04.2021

Tishka17

Test Case
Возможно стоит создать некий класс CommandManager,...

Забудьте про синглтоны

0

23.04.2021

Michael Pokrovsky

Tishka17
Забудьте про синглтоны

аргументации не будет?

0

23.04.2021

Test Case

Michael Pokrovsky
аргументации не будет?

Гуглится в лоб "Antipattern Singleton", узнал некоторые новые детали.

0

23.04.2021

Tishka17

Michael Pokrovsky
аргументации не будет?

Просто синглтон имеет непонятный жизненный цикл. Синглтон с доступом к экземпляру через gwtInctace() или аналог имеет все те же проблемы что глобальные переменные. В том числе, сильное сцепление программы

0

23.04.2021

🌿UnorderedSequence🌿 Shindov 🍀 Автор вопроса

Tishka17
Это очень хрупко

А как сделать норм? Только импортнуть каждую команду вручную?

0

23.04.2021

Tishka17

🌿UnorderedSequence🌿 Shindov 🍀
А как сделать норм? Только импортнуть каждую коман...

Да. Это 1. Просто 2. Понятно 3. Очень гибко

0

23.04.2021