Телеграмм чат группы aiogram

Size: a a a

aiogram [ru]

2020 July 24

мне не нравится что у тебя классы без создания экземпляров

помню когда с классовыми фильтрами только познакомился, недоумевал почему мидлвари регистрируются инстансами, а фильтры классами 🤔

dispatcher.filters_factory.bind(SomeFilter)
dispatcher.middleware.setup(SomeMiddleware())

источник

03:09пожаловаться #1

Tishka17 in aiogram [ru]

Вообще непонятно что он сэкономил на таком классе, как будто только более текста надо писать

источник

03:09пожаловаться #2

Tishka17 in aiogram [ru]

Технопёс

dispatcher.filters_factory.bind(SomeFilter)
dispatcher.middleware.setup(SomeMiddleware())

Я классовые фильтры не юзаю. Вообще не понимаю зачем они

источник

03:09пожаловаться #3

Технопёс in aiogram [ru]

Tishka17

Я классовые фильтры не юзаю. Вообще не понимаю зачем они

юзаешь

источник

03:10пожаловаться #4

Технопёс in aiogram [ru]

кварг-фильтры же классовые

источник

03:10пожаловаться #5

Tishka17 in aiogram [ru]

Технопёс

юзаешь

Ну имею ввиду свои не делаю

источник

03:10пожаловаться #6

Технопёс in aiogram [ru]

std::mpa

укажи какие хочешь

а зачем мне их указывать? Чтобы что?

источник

03:10пожаловаться #7

std::mpa in aiogram [ru]

Технопёс

тип обновления можно указать только один? А что если мне надо сделать одно и тоже действие при разных событиях? Например, произвести возврат в какое-то главное меню при reply/inline кнопке cancel? Класс-бейзд хэндлеры спокойно относятся к ветвлению isinstance? Если да, то зачем тогда вооюще указывать тип?

мне в этом проекте только такие нужны были.

источник

03:10пожаловаться #8

Tishka17 in aiogram [ru]

Технопёс

dispatcher.filters_factory.bind(SomeFilter)
dispatcher.middleware.setup(SomeMiddleware())

И да, непонятно, почему нельзя было сделать мидлварю на это всё

источник

03:10пожаловаться #9

std::mpa in aiogram [ru]

Технопёс

а зачем мне их указывать? Чтобы что?

эээ, а что ты хочешь? на все типы уведомления хендлер повесить?

источник

03:11пожаловаться #10

Tishka17 in aiogram [ru]

std::mpa

мне в этом проекте только такие нужны были.

А в чем профит?

источник

03:11пожаловаться #11

Технопёс in aiogram [ru]

std::mpa

мне в этом проекте только такие нужны были.

else:
    continue

🤔

источник

03:11пожаловаться #12

std::mpa in aiogram [ru]

Технопёс

else:
    continue

🤔

+ :)
там пока ничего другого и нет.

источник

03:11пожаловаться #13

Технопёс in aiogram [ru]

std::mpa

эээ, а что ты хочешь? на все типы уведомления хендлер повесить?

а, вот оно зачем

источник

03:12пожаловаться #14

std::mpa in aiogram [ru]

Tishka17

А в чем профит?

удобство и DI в некоторых местах :D

источник

03:12пожаловаться #15

Tishka17 in aiogram [ru]

std::mpa

удобство и DI в некоторых местах :D

Какое DI? У тебя инстансы даже не создаются

источник

03:12пожаловаться #16

Tishka17 in aiogram [ru]

std::mpa

удобство и DI в некоторых местах :D

А удобство в чем? Я вижу что кода столько же, только он какой-то неявный

источник

03:13пожаловаться #17

std::mpa in aiogram [ru]

всё просто. в 3.х такой CBH, кстати.

источник

03:13пожаловаться #18

std::mpa in aiogram [ru]

Tishka17

Какое DI? У тебя инстансы даже не создаются

создаются, когда успешно проходит фильтр. я же показал инит.

источник

03:13пожаловаться #19

Tishka17 in aiogram [ru]

std::mpa

создаются, когда успешно проходит фильтр. я же показал инит.

А смысл? Это же так же в хэндлер обычный прилетает

источник

03:14пожаловаться #20