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

14 просмотров

Мб @staticmethod

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

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

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

Маша
Мб @staticmethod

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

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

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

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

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

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

Всем привет, написал код ниже, но он выдает сегфолт, в чем причина? #include <stdio.h> #include <stdlib.h> #include <string.h> struct product { char *name; float price; };...
buzz базз
75
База данных не поможет. Шифрование не поможет. Какие там ещё варианты? Накидывайте.
КТ315
20
А табстоп это сообщение от окна или от элемента управления?
The Bird of Hermes
18
А как лучше конвертировать физический адрес в виртуальный при маппинге? В случае ядра у меня, например, direct mapping, первые 768МБ я как есть мапплю в higher half, а остальн...
Evg Resh
26
Открыл свой двухкилобайтный экзешник в x32dbg, а тут какая-то хрень. Смущает кнопка "выполнить до пользовательского кода", а что ещё может быть в файле помимо него ?
НѣкъиⰘижєжєиꙁъвьсєсвѣтьноѣсѣтиѥсть•
11
Мне были интересны дишные хаки и я нашёл любопытный способ на форуме через __traits, что-то вроде int delegate(int) fac = (int n) => n == 0 ? 1 : n * __traits(parent, {})(n - ...
Constantin F.
1
Вопрос тем кто смотрит видео и слушает подкасты - как вы потом ищете нужную вам информацию? Вот статью я прочитал, потом могу искать нужную мне часть банальным поиском. Пропус...
Aleksandr Druzhinin
4
Всем привет, подскажите/посоветуйте пожалуйста. Фаердак компоненты, имею одно место где бизнес хочет видеть при открытии формы список всех клиентов, это порядка 30к. Мои дово...
Sasha Sch
14
Ребят, если кто в курсе - скажите, а в загранке такое же засилье маркетплейсов? или там простые сермяжные интернет-магазины живут попроще?
Андрей [aharito] Харитонов
14
Коллеги, доброе утро. Запустил на удаленном хосте приложение (ручками зашел туда по ssh и запустил, не командой удаленно). Создал потом ssh-туннель, и с моей машины приложение...
Δημήτηρ
9
Карта сайта