ANTICHAT - Бот для телеграмма Moderator

ANTICHAT (https://forum.antichat.xyz/index.php)

- Python (https://forum.antichat.xyz/forumdisplay.php?f=205)

- - Бот для телеграмма Moderator (https://forum.antichat.xyz/showthread.php?t=1544753)

Бот Moderator с управлением чата, участников.

Выдача рангов администрации взято под основу как у Iris

Установка бота

moder.py откройте в блокноте замените токен на свой!

Где взять токен?

Перейдите в бота @BotFather, создайте бота в нем и скопируйте ваш токен.

Запуск бота!

Распакуйте архив в любое для вас удобное место, установите последнию версию Python

Установите зависимости для бота в командной строке

pip install python-telegram-bot

пароль от архива: 123123

Цитата:

Сообщение от MrApostolAngels

https://forum.antichat.xyz/attachmen...785f24afb1.png

Цитата:

Сообщение от dask

Цитата:

Сообщение от dask

Если ты про то что там канал то пиши @useryoufi1

Цитата:

Сообщение от MrApostolAngels

?

Если ты про то что там канал то пиши @useryoufi1

я про то что ты не указал :)

Цитата:

Сообщение от dask

я про то что ты не указал :)

А зачем мне указывать это мой телеграм

Цитата:

Сообщение от MrApostolAngels

А зачем мне указывать это мой телеграм

ladno

Отличный пример говнокода. Всё сделано через одно место.

Во-первых, вся логика и обработка команд свалены в один гигантский файл. Вместо нормального разделения на 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. пишу с телефона, поэтому мне лень расставлять табы в блоках с кодом

Цитата:

Сообщение от vindarix

Отличный пример говнокода. Всё сделано через одно место.
Во-первых, вся логика и обработка команд свалены в один гигантский файл. Вместо нормального разделения на 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

)

:

Код:

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

)

Код:

with

open

(

DATA_FILE

,

'w'

)

as

f

:

json

.

dump

(

data

,

f

)

Код:

RANKS

=

[

'Helper'

,

'Moderator'

,

'Admin'

,

'Owner'

]

TOKEN

=

'123456:ABC-DEF'

такое нельзя держать в коде, нужно выносить в переменные окружения.

Отдельно отмечу, что python-telegram-bot — не лучший выбор для такого бота. Он громоздкий, неудобен для масштабных ботов и плохо справляется с асинхронностью. Тут лучше юзать aiogram.

В итоге имеем:
1. Монолитный код без адекватной структуры
2. JSON вместо СУБД
3. Дублирование функций
4. Блокирующие вызовы в async-коде
5. Хардкод токенов и рангов
Проще будет реально переписать с нуля: сделать отдельный модуль для БД, разбить проект на файлы и настроить нормальные хендлеры.

P.s. пишу с телефона, поэтому мне лень расставлять табы в блоках с кодом

Спасибо за предоставленную информацию! Учту следующий раз.

Цитата:

Сообщение от MrApostolAngels

А зачем токен бота выставлять в открытый доступ?

Цитата:

Сообщение от MEMER_FAUD

А зачем токен бота выставлять в открытый доступ?

Это для примера