Отличный пример говнокода. Всё сделано через одно место.
Во-первых, вся логика и обработка команд свалены в один гигантский файл. Вместо нормального разделения на handlers.py, database.py, utils.py — там просто огромная плеяда десятков функций подряд, например:
Python:
Код:
async
def
show_admins
(
update
,
context
)
:
async
def
promote_user
(
update
,
context
)
:
async
def
demote_user
(
update
,
context
)
:
Каждая из этих функций делает почти одно и то же, но код повторяется слово в слово.
Во-вторых, вместо СУБД используется JSON:
Python:
Код:
DATA_FILE
=
'admin_system_data.json'
def
load_data
(
)
:
with
open
(
DATA_FILE
,
'r'
)
as
f
:
return
json
.
load
(
f
)
и потом в десятках мест:
Python:
Код:
data
=
load_data
(
)
d
ata
[
'admins'
]
.
append
(
user_id
)
save_data
(
data
)
При параллельных записях файл запросто повредится. Надо юзать SQLite или PostgreSQL.
Третье — асинхронность нарушена: код объявлен как async, но внутри есть блокирующие операции:
Python:
Код:
with
open
(
DATA_FILE
,
'w'
)
as
f
:
json
.
dump
(
data
,
f
)
Получаем блокировку event loop'а и торможение бота при любой записи.
К этому добавляем глобал переменные и хардкод по всему коду:
Python:
Код:
RANKS
=
[
'Helper'
,
'Moderator'
,
'Admin'
,
'Owner'
]
TOKEN
=
'123456:ABC-DEF'
такое нельзя держать в коде, нужно выносить в переменные окружения.
Отдельно отмечу, что python-telegram-bot — не лучший выбор для такого бота. Он громоздкий, неудобен для масштабных ботов и плохо справляется с асинхронностью. Тут лучше юзать aiogram.
В итоге имеем:
1. Монолитный код без адекватной структуры
2. JSON вместо СУБД
3. Дублирование функций
4. Блокирующие вызовы в async-коде
5. Хардкод токенов и рангов
Проще будет реально переписать с нуля: сделать отдельный модуль для БД, разбить проект на файлы и настроить нормальные хендлеры.
P.s. пишу с телефона, поэтому мне лень расставлять табы в блоках с кодом