169 похожих чатов

Https://Github.Com/Vsecoder/Decent-Catbot/ Кто может провести ревью и порекомендовать по коду?

33 ответов

29 просмотров
〘 Всеволод 〙-⒗ Автор вопроса

@decent_catbot сам бот

BOT_TOKEN = BOT["token"] две ошибки сразу. Глобальные переенные вместо конфига и словарь вместо объекта в коде

〘 Всеволод 〙 ⒗
@decent_catbot сам бот

ты не написал какие команды у него есть

from handlers import * не надо импортировать *

〘 Всеволод 〙 ⒗
там дока в описании

всё же, лучше всего сделать это

Wakatime 7h Не верю

codecs.open(in_file, 'r', 'utf-8') - проcто open(in_file, "r", encoding="utf-8")

⛩️Camhowe⛩️
А лучше with open

там with уже есть, речь про другое

https://github.com/vsecoder/decent-catbot/blob/master/handlers/mute.py#L11 слишком сложная функция, разбить на несколько

〘 Всеволод 〙 ⒗
8

Всё равно маловато

〘 Всеволод 〙-⒗ Автор вопроса
Tishka17
да

там это проект другой, я туда даже не лез)

〘 Всеволод 〙-⒗ Автор вопроса
Evgen Fil // hehe
Всё равно маловато

Мало? проект вчера после обеда был придуман...

〘 Всеволод 〙 ⒗
забей, что ещё?

нет, не забью. Если это другой проект - не копипасть к себе, а подключай как либу. Если скопипастил - разбирайся и поддерживай

〘 Всеволод 〙 ⒗
забей, что ещё?

https://github.com/vsecoder/decent-catbot/blob/master/handlers/setprotect.py#L6 опять какие-то глобалы вместо конфига и словари. Или это не конфиг?

〘 Всеволод 〙 ⒗
забей, что ещё?

https://github.com/vsecoder/decent-catbot/blob/master/handlers/notes.py#L32 HTML экранировать надо

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

〘 Всеволод 〙 ⒗
Всмысле?

в прямом, вот будет в name там <blabl> и всё сломается

〘 Всеволод 〙-⒗ Автор вопроса
Tishka17
в прямом, вот будет в name там <blabl> и всё слома...

А, м, в utils функция есть, юзну, понял

Tishka17
в прямом, вот будет в name там <blabl> и всё слома...

О дааа, я раньше сталкивался с такой проблемой, пользователя просто не записовало в бд так как код думал что это HTML разметка

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

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

Господа, а что сейчас вообще с рынком труда на делфи происходит? Какова ситуация?
Rꙮman Yankꙮvsky
29
А вообще, что может смущать в самой Julia - бы сказал, что нет единого стандартного подхода по многим моментам, поэтому многое выглядит как "хаки" и произвол. Короче говоря, с...
Viktor G.
2
30500 за редактор? )
Владимир
47
а через ESC-код ?
Alexey Kulakov
29
Чёт не понял, я ж правильной функцией воспользовался чтобы вывести отладочную информацию? но что-то она не ловится
notme
18
У меня есть функция где происходит это: write_bit(buffer, 1); write_bit(buffer, 0); write_bit(buffer, 1); write_bit(buffer, 1); write_bit(buffer, 1); w...
~
14
Добрый день! Скажите пожалуйста, а какие программы вы бы рекомендовали написать для того, чтобы научиться управлять памятью? Можно написать динамический массив, можно связный ...
Филипп
7
Недавно 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
Карта сайта