Search
Write a publication
Pull to refresh
1
0

User

Send message

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

Спасибо за статью. Не понравился тип возвращаемых данных для метода load. Думаю, уместнее будет указать тип Self или TypeVar из модуля typing (или typing_extensions, в зависимости от вашей версии питона).

Есть же tinydb с кучей модулей. Зачем заново все придумывать?

зачем тянуть requests, если среди зависимостей уже есть httpx?

Впервые вижу вижу, что кто-то создает виртуальное окружение средствами ide :)

Вы храните статистику в словаре, а значит в памяти. При падении или перезапуске вся статистика обнулится, поэтому можно подключить простую tinydb.

В нескольких хендлерах вы проверяете статус юзера на админа ифами, нарушаете DRY :) может вынести в отдельную функцию и использовать оператор in?

Переменная duration с параметром по умолчанию намекает на то, что это константа, как и bad_words, можно вынести отдельно.

В хендлерк mute_user сно использовать моржовый оператор, где проверка args.

Что-то "пет-IT проект" глаза режет. Может лучше "IT пет-проект"?

Спасибо за статью и ваш труд, как раз недавно начал заниматься пет-проектами, чтобы навык не терять, а теперь ещё и причешу их немного :)

Доброго времени суток. Подскажите, а какая была цель создания данного проекта и, разве, не удобнее работать с тасками чернз телеграмм / приложение?

Information

Rating
Does not participate
Registered
Activity