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 ответов

28 просмотров

Мб @staticmethod

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

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

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

Маша
Мб @staticmethod

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

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

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

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

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

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

30500 за редактор? )
Владимир
47
а через ESC-код ?
Alexey Kulakov
29
Чёт не понял, я ж правильной функцией воспользовался чтобы вывести отладочную информацию? но что-то она не ловится
notme
18
Добрый день! Скажите пожалуйста, а какие программы вы бы рекомендовали написать для того, чтобы научиться управлять памятью? Можно написать динамический массив, можно связный ...
Филипп
7
У меня есть функция где происходит это: write_bit(buffer, 1); write_bit(buffer, 0); write_bit(buffer, 1); write_bit(buffer, 1); write_bit(buffer, 1); w...
~
14
Недавно Google Project Zero нашёл багу в SQLite с помощью LLM, о чём достаточно было шумно в определённых интернетах, которые сопровождались рассказами, что скоро всех "ибешни...
Alex Sherbakov
5
Ребят в СИ можно реализовать ООП?
Николай
33
https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_h_common.erl#L174 https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_olp.erl#L76 15 лет назад...
Maksim Lapshin
20
Как передать управляющий символ в открытую через CreateProcess консоль? Собсна, есть процедура: procedure TRedirectThread.WriteData(Data: OEMString); var Written: Cardinal;...
Serjone
6
Всем привет! Имеется функция: function IsValidChar(ch: UTF8Char): Boolean; var i: Integer; ValidChars: AnsiString; begin ValidChars := 'abcdefghijklmnopqrstuvwxyzABCDE...
Евгений
44
Карта сайта