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

Друзья, подскажите по ООП, совсем запутался. Есть работающая программа, но со

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

Выделил основную часть - в ней описывается подключение к базе, к API сторонней программы, к API Telegram, основные основные функции работы с базой. Сделал класс A.
Остальные функции начал выносить в отдельные файлы и классы. Как пример - слушалка и обработка событий из сторонней программы, с последующей обработкой и отправкой в телеграм (класс EventsListener)
При необходимости я могу подключать эти дополнительные функции, кастомизировать для различных инстансов и так далее.

Пытаюсь сделать так:
class A:
def __init__():
self.telebot = telebot.Telebot(TOKEN)
self.listener = EventsListener()
……..

def do_smth(self, msg):
self.listener.process_message(msg)

Но дело в том, что внутри process_message нужно обращение к базе (которое описано в классе A), чтобы правильно составить список тех, кому отправлять сообщения в Telegram. Да и сам telebot нужен, чтобы отправить сообщение через send_message (но его я хотя бы могу передать в момент инциализации self.listener).

Натолкните плиз на мысль, как все это разорвать и правильно сделать?

7 ответов

27 просмотров

Мб @staticmethod

Вынеси базу в отдельный класс, передай ее в конструктор А и лисенеру, там сохрани и используй

Alexander-Fomin Автор вопроса
Denis Kalinochkin
Вынеси базу в отдельный класс, передай ее в констр...

Ага, вот тоже до этого дошёл и вынес, спасибо!

Маша
Мб @staticmethod

мб тебе пока еще учиться самой надо, а не советы раздавать?

Киану Ривз
А тебе уже пора советы раздавать?

да, могу. а тебе уже мамка спать идти велела.

Нужно понимать фловы в программе хотя бы, чтобы давать советы - здесь не понятно, какое отношение между ивент листенером и A. Как уже сказали, обычно есть врепер или несколько, в которых определяют методы для работы с БД - остальные классы изолированы от написания SQL запросов. Если здесь есть event listener, то странно, что он создаётся внутри А - чаще всего это делают через dependency injection, где главный класс инициализирует компоненты - тогда А бы получал listener как параметр, класс БД бы настраивался и передавался куда нужно тоже в главном классе... Примеры dependency injection - Spring(Java), oatpp(C++). В классе EventListener не должно быть логики по отправки месседжей в телеграмм, потому что он реализует publisher/subscriber паттерн. Обработчик подписывается на ивент, а затем обрабатывает месседж(собирает получателей с базы и отправляет, куда нужно)... Но насколько я понял, то тут только название EventListener, а по факту какой-то медиатор.

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

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

Мужики и девушки, привет) в Вelphi xe7 в настройках во вкладке "Editor Options" далее " Color" есть список: "Elements", открыв который мы можем настраивать отображение разных...
Kraszx
14
Добрый вечер. Есть вопрос, а может и предложение. Был у меня диалог в другой группе о делфи и я задался вопросом: "А нельзя ли в делфи цвет //коментария и {комментария} сде...
Kraszx
24
Всем привет! Подскажи, пожалуйста, как передать в TComboBox сразу значение и id записи. На Delphi я делал так: ComboBox1.Items.AddObject('Какое-то значение', Pointer(id запис...
Евгений
13
А вот это что за конструкция? Вернее, она тут нафига?
Serjone
10
Привет. Подскажите, как правильно сматчить лист фиксированного размера, чтобы компилятор не говорил мне о неполном паттерне? Допустим что-то такое [x', y'] = sort [x, y]?
Arseny
8
Мдя, прикол, боевая сборка запускается (именно под отладчиком) после F9 примерно полторы минуты (97 секунд если быть точным). Начал копать - проблема детектится сразу - зависа...
Александр (Rouse_) Багель
38
Здравствуйте, вопрос по структурам данных. Были у вас случаи, когда пришлось писать деревья или двунаправленные списки?
/ /
50
Мужики. привет) в Вelphi xe7 в настройках во вкладке "Editor Options" далее " Color" есть список: "Elements", открыв который мы можем настраивать отображение разных элементов...
Kraszx
2
Товарищи, кто работа с iphelper? Или может я в самой логике ошибки фигачу, не пойму.... var ifTable : PMIB_IFTABLE; size, corSize: DWORD; Buffer ...
Warfarellen
4
я так понимаю, я так подозреваю, что создание такого плагина для человека, кто умеет писать плагины для делфи потребует минут 5-10 времени. но это мое подозрение. хотелось бы ...
Kraszx
7
Карта сайта