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

Привет, у меня есть тг бот на 1тыс+ строк, кто

может посмотреть как более опытный разраб, какие то советы подкинуть?
Стек telebot psycopg2

16 ответов

15 просмотров

1. раздели на файлы 2. не используй глобальные переменные 3. разделяй код, где написан sql и тот что реализует логику

Tishka17
1. раздели на файлы 2. не используй глобальные пер...

кстати, beginners вопрос под третьим пунктом обычно понимают создание функций(методов) типа thing.get_all_users(), thing.get_user_data(id) и тд?

Notai- Автор вопроса

1. У меня есть главный файл bot.py в котором строится логика telebot 2. Есть файл bot_logic в который я вынес функции и классы которые строят внутреннюю логику приложения. 3. есть bot_config в котором все клавиатуры, списки с наименованием продуктов и тп 4. И собсна postgres.py в котором есть функции относящиеся только с работой в БД и главная функция make_query() с помощью которой я обращаюсь к БД в главном файле. Из глобальных переменных только константы Что скажешь?

Notai
1. У меня есть главный файл bot.py в котором строи...

1. клавиатуры не тянут на конфиг, неудачное имя 2. make_query звучит сомнительно, сделай много разных функций, которые делают кокнретные вещи.

Notai- Автор вопроса
Tishka17
1. клавиатуры не тянут на конфиг, неудачное имя 2....

Клавиатуры вынести в keyboards.py например? а make_query() вот таким образом работает - def make_query(query, fetch=None): connection = psycopg2.connect(user='postgres', password='5432', host='127.0.0.1', port=5432, database='dairyproject') with connection.cursor() as cursor: cursor.execute(query) if fetch == 'one': return cursor.fetchone() elif fetch == 'all': return cursor.fetchall() connection.commit() она вызывается около 70 раз по всему проекты в двух файлах (в основном в главном) не знаю как сделать тогда

Notai
Клавиатуры вынести в keyboards.py например? а make...

а вот что вместо глобалов, магические константы

Notai
Клавиатуры вынести в keyboards.py например? а make...

у тебя адрес БД и пароли должны браться из конфигурационного файла/переменных окружения, а не валяться в коде

Notai
Клавиатуры вынести в keyboards.py например? а make...

откуда ты спер этот код? у латанда?

Notai- Автор вопроса
Tishka17
у тебя адрес БД и пароли должны браться из конфигу...

Данные бд тянутся с postgres_config.py там как раз данные для коннекта, и мои внутренние заметки

Notai- Автор вопроса
Tishka17
откуда ты спер этот код? у латанда?

Сам написал 😂 не понимаю о чем ты

Notai
Данные бд тянутся с postgres_config.py там как раз...

то есть глобальные переменные все таки?

Notai- Автор вопроса
Tishka17
то есть глобальные переменные все таки?

Мной инициализированных нет, если ты об этом

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

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

30500 за редактор? )
Владимир
47
а через ESC-код ?
Alexey Kulakov
29
Чёт не понял, я ж правильной функцией воспользовался чтобы вывести отладочную информацию? но что-то она не ловится
notme
18
Добрый день! Скажите пожалуйста, а какие программы вы бы рекомендовали написать для того, чтобы научиться управлять памятью? Можно написать динамический массив, можно связный ...
Филипп
7
У меня есть функция где происходит это: write_bit(buffer, 1); write_bit(buffer, 0); write_bit(buffer, 1); write_bit(buffer, 1); write_bit(buffer, 1); w...
~
14
Недавно 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
Как передать управляющий символ в открытую через CreateProcess консоль? Собсна, есть процедура: procedure TRedirectThread.WriteData(Data: OEMString); var Written: Cardinal;...
Serjone
6
Всем привет! Имеется функция: function IsValidChar(ch: UTF8Char): Boolean; var i: Integer; ValidChars: AnsiString; begin ValidChars := 'abcdefghijklmnopqrstuvwxyzABCDE...
Евгений
44
Карта сайта