Ребзя, я вернулся спустя несколько месяцев и апдейтнул своего бота,

который для чтения xkcd.

1) Добавил русский UI.
2) Пошерстил по интернетам русские переводы, теперь их ~+300 по сравнению с прошлой версией (а этом боте самая большая коллекция переводов в одном месте (если знаете, где еще надыбать, то напишите мне, буду премного благодарен)).
3) При выборе русского объяснение буду автоматически переводиться (причем прилично).
4) Админу можно написать, вотъ.
5) Хуева гора мелочей всяких..

Конечно же ж вылизал код (хоть @Forevka в меня и не верил). Снабдил там все комментами.

Короче хочу наклянчить у вас повторное код-ревью (сойду за джуна?), те советы были самые толковые)))

Бот: https://t.me/telexkcdbot
Код: https://github.com/coryphoenixxx/telexkcdbot

не сочти за придирку но регистрация хэндлеров внизу кода - неуважение к читающим )

всяко лучше чем диспатчеры

Да нет, с чего бы. Это новый стайл написания кода

базару нет. яж написал ИМХО ) мне просто не нравится такой подход.

?__?

self.pool.execute удачи сделать нескольк запросов в одной транзкции

Нормальные формы? А что это? async def update_bookmarks(self, user_id: int, new_bookmarks: list[int]): query = """UPDATE users SET bookmarks = $1 WHERE user_id = $2;""" upd_bookmarks_json = json.dumps(new_bookmarks) await self.pool.execute(query, upd_bookmarks_json, user_id)

tuple[int] зачем тебе возвращать тупл из одного числа? За что ты так с людьми?

пиздеж, говоришь что тупл из одного числа, возвращаешь тупл из хз какого числа чисел async def get_all_comics_ids(self) -> tuple[int]: query = """SELECT array_agg(comic_id) FROM comics;""" res = await self.pool.fetchval(query) return tuple(res) if res else ()

он подразумевал тупл состоящий только из интов

хуево подразумевал

Это работает не так

Я не пойму, типа List[int] возвращать корректно? Судя по stackoverflow так делают, подразумевая именно список чисел

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

Ну дык я так получаю просто список чисел от бд, а так мне компрехеншн фигачить, зачем?

Ну... Окей, принимается

Нормальные специалисты (aka Груша с Алексом да Тишкой) хуесосят за такие вещи, но я использую, и нормально.

Ну если серьезно в разработку идтить, то ведь знание logging более универсально, особенно если легаси

Это и так очевидно. В своих ботах я кидаю в логуру весь вывод логгинга и дело с концом

только логуру не нужен

Pybabel. Не должна она пропадать, какая кнопка "перевести"?

Пубабел

ShiroNoHaga
Заметил, что при старте оно снова просит язык ввес...

можно узнать язык пользователя и менять автоматом на него💁‍♂