Как сократить данный код?

Question

DarkLight_723 @DarkLight_723

Python

Как сократить данный код?

from requests import get
import telebot 
from telebot import types # для создания конопок!

bot = telebot.TeleBot(token='') 



@bot.message_handler(commands=['start', 'старт', 'Старт', 'Start']) # отслеживаем команды (/start, /старт)
def start_message(message):
    markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
    btn1 = types.KeyboardButton('Записаться')
    markup.add(btn1)

    mess = f' Здравствуйте <b>{message.from_user.first_name}</b>, нажмите на кнопку "Записаться" для продолжения' 
    bot.send_message(message.chat.id, mess, parse_mode= 'html', reply_markup = markup) #


@bot.message_handler(content_types=['text'])
def bot_message(message):

    if message.text == 'Записаться':
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        bn1 = types.KeyboardButton('26.01')
        bn2 = types.KeyboardButton('27.01')
        bn3 = types.KeyboardButton('28.01')
        bn4 = types.KeyboardButton('29.01')
        bn5 = types.KeyboardButton('30.01')
        bn6 = types.KeyboardButton('31.01')
        back1 = types.KeyboardButton('Назад')
        markup.add(bn1, bn2, bn3, bn4, bn5, bn6, back1)
        bot.send_message(message.chat.id, text='Выберите удобную вам дату для записи', reply_markup = markup)


    if message.text == '26.01':
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        bt1 = types.KeyboardButton('8:00')
        bt2 = types.KeyboardButton('9:00')
        bt3 = types.KeyboardButton('10:00')
        bt4 = types.KeyboardButton('11:00')
        bt5 = types.KeyboardButton('12:00')
        bt6 = types.KeyboardButton('13:00')
        bt7 = types.KeyboardButton('14:00')
        bt8 = types.KeyboardButton('15:00')
        bt9 = types.KeyboardButton('16:00')
        bt10 = types.KeyboardButton('17:00')
        bt11 = types.KeyboardButton('18:00')
        bt12 = types.KeyboardButton('19:00')
        bt13 = types.KeyboardButton('20:00')
        bt14 = types.KeyboardButton('21:00')
        back11 = types.KeyboardButton('Назад')
        markup.add(bt1, bt2, bt3, bt4, bt5, bt6, bt7, bt8, bt9, bt10, bt11, bt12, bt13, bt14, back11)
        bot.send_message(message.chat.id, text='Выберите удобную вам дату для записи', reply_markup = markup)

    if message.text == '27.01':
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        bt1 = types.KeyboardButton('8:00')
        bt2 = types.KeyboardButton('9:00')
        bt3 = types.KeyboardButton('10:00')
        bt4 = types.KeyboardButton('11:00')
        bt5 = types.KeyboardButton('12:00')
        bt6 = types.KeyboardButton('13:00')
        bt7 = types.KeyboardButton('14:00')
        bt8 = types.KeyboardButton('15:00')
        bt9 = types.KeyboardButton('16:00')
        bt10 = types.KeyboardButton('17:00')
        bt11 = types.KeyboardButton('18:00')
        bt12 = types.KeyboardButton('19:00')
        bt13 = types.KeyboardButton('20:00')
        bt14 = types.KeyboardButton('21:00')
        back11 = types.KeyboardButton('Назад')
        markup.add(bt1, bt2, bt3, bt4, bt5, bt6, bt7, bt8, bt9, bt10, bt11, bt12, bt13, bt14, back11)
        bot.send_message(message.chat.id, text='Выберите удобную вам дату для записи', reply_markup = markup)
    
    if message.text == '28.01':
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        bt1 = types.KeyboardButton('8:00')
        bt2 = types.KeyboardButton('9:00')
        bt3 = types.KeyboardButton('10:00')
        bt4 = types.KeyboardButton('11:00')
        bt5 = types.KeyboardButton('12:00')
        bt6 = types.KeyboardButton('13:00')
        bt7 = types.KeyboardButton('14:00')
        bt8 = types.KeyboardButton('15:00')
        bt9 = types.KeyboardButton('16:00')
        bt10 = types.KeyboardButton('17:00')
        bt11 = types.KeyboardButton('18:00')
        bt12 = types.KeyboardButton('19:00')
        bt13 = types.KeyboardButton('20:00')
        bt14 = types.KeyboardButton('21:00')
        back11 = types.KeyboardButton('Назад')
        markup.add(bt1, bt2, bt3, bt4, bt5, bt6, bt7, bt8, bt9, bt10, bt11, bt12, bt13, bt14, back11)
        bot.send_message(message.chat.id, text='Выберите удобную вам дату для записи', reply_markup = markup)

    if message.text == '29.01':
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        bt1 = types.KeyboardButton('8:00')
        bt2 = types.KeyboardButton('9:00')
        bt3 = types.KeyboardButton('10:00')
        bt4 = types.KeyboardButton('11:00')
        bt5 = types.KeyboardButton('12:00')
        bt6 = types.KeyboardButton('13:00')
        bt7 = types.KeyboardButton('14:00')
        bt8 = types.KeyboardButton('15:00')
        bt9 = types.KeyboardButton('16:00')
        bt10 = types.KeyboardButton('17:00')
        bt11 = types.KeyboardButton('18:00')
        bt12 = types.KeyboardButton('19:00')
        bt13 = types.KeyboardButton('20:00')
        bt14 = types.KeyboardButton('21:00')
        back11 = types.KeyboardButton('Назад')
        markup.add(bt1, bt2, bt3, bt4, bt5, bt6, bt7, bt8, bt9, bt10, bt11, bt12, bt13, bt14, back11)
        bot.send_message(message.chat.id, text='Выберите удобную вам дату для записи', reply_markup = markup)

    if message.text == '30.01':
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        bt1 = types.KeyboardButton('8:00')
        bt2 = types.KeyboardButton('9:00')
        bt3 = types.KeyboardButton('10:00')
        bt4 = types.KeyboardButton('11:00')
        bt5 = types.KeyboardButton('12:00')
        bt6 = types.KeyboardButton('13:00')
        bt7 = types.KeyboardButton('14:00')
        bt8 = types.KeyboardButton('15:00')
        bt9 = types.KeyboardButton('16:00')
        bt10 = types.KeyboardButton('17:00')
        bt11 = types.KeyboardButton('18:00')
        bt12 = types.KeyboardButton('19:00')
        bt13 = types.KeyboardButton('20:00')
        bt14 = types.KeyboardButton('21:00')
        back11 = types.KeyboardButton('Назад')
        markup.add(bt1, bt2, bt3, bt4, bt5, bt6, bt7, bt8, bt9, bt10, bt11, bt12, bt13, bt14, back11)
        bot.send_message(message.chat.id, text='Выберите удобную вам дату для записи', reply_markup = markup)

    if message.text == '31.01':
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        bt1 = types.KeyboardButton('8:00')
        bt2 = types.KeyboardButton('9:00')
        bt3 = types.KeyboardButton('10:00')
        bt4 = types.KeyboardButton('11:00')
        bt5 = types.KeyboardButton('12:00')
        bt6 = types.KeyboardButton('13:00')
        bt7 = types.KeyboardButton('14:00')
        bt8 = types.KeyboardButton('15:00')
        bt9 = types.KeyboardButton('16:00')
        bt10 = types.KeyboardButton('17:00')
        bt11 = types.KeyboardButton('18:00')
        bt12 = types.KeyboardButton('19:00')
        bt13 = types.KeyboardButton('20:00')
        bt14 = types.KeyboardButton('21:00')
        back11 = types.KeyboardButton('Назад')
        markup.add(bt1, bt2, bt3, bt4, bt5, bt6, bt7, bt8, bt9, bt10, bt11, bt12, bt13, bt14, back11)
        bot.send_message(message.chat.id, text='Выберите удобную вам дату для записи', reply_markup = markup)


    

    elif message.text == 'Назад':
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        btn1 = types.KeyboardButton('Записаться')
        markup.add(btn1)
        bot.send_message(message.chat.id, text='Выход в главное меню', parse_mode= 'html', reply_markup = markup) #отправка сообещния: в данный чат/текст/что отлеживаем (в моём случае html)

     
bot.polling(none_stop=True)

Вопрос задан более двух лет назад
299 просмотров

5 комментариев

Подписаться 1 Простой 5 комментариев

Пригласить эксперта

Ответы на вопрос 3

Комментировать

4 комментария

Алан Гибизов @phaggi Куратор тега Python

Уфф… медвежья услуга это называется.

Написано более двух лет назад
Vfreelance @vfreelancer

хоть бы в цикле
markup.add(button()) сделали

Написано более двух лет назад
Daniil_8_5 @Daniil_8_5

Vfreelance, вот так?

Написано более двух лет назад

Алан Гибизов @phaggi Куратор тега Python

Daniil_8_5, всё

вот это

if message.text == 'Записаться':
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        for add in buttons:
            markup.add(button(add), button('Назад'))
        bot.send_message(message.chat.id, text='Выберите удобную вам дату для записи', reply_markup = markup)

    if message.text == '26.01':
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        for add in buttons:
            markup.add(button(add), button('Назад'))
        bot.send_message(message.chat.id, text='Выберите удобную вам дату для записи', reply_markup = markup)

    if message.text == '27.01':
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        for add in buttons:
            markup.add(button(add), button('Назад'))
        bot.send_message(message.chat.id, text='Выберите удобную вам дату для записи', reply_markup = markup)

    if message.text == '28.01':
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        for add in buttons:
            markup.add(button(add), button('Назад'))
        bot.send_message(message.chat.id, text='Выберите удобную вам дату для записи', reply_markup = markup)

    if message.text == '29.01':
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        for add in buttons:
            markup.add(button(add), button('Назад'))
        bot.send_message(message.chat.id, text='Выберите удобную вам дату для записи', reply_markup = markup)

    if message.text == '30.01':
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        for add in buttons:
            markup.add(button(add), button('Назад'))
        bot.send_message(message.chat.id, text='Выберите удобную вам дату для записи', reply_markup = markup)

    if message.text == '31.01':
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        for add in buttons:
            markup.add(button(add), button('Назад'))
        bot.send_message(message.chat.id, text='Выберите удобную вам дату для записи', reply_markup = markup)

можно заменить на это:

true_messages = ['Записаться', '26.01', '27.01', '28.01', '29.01', '30.01', '31.01']
if message.text in true_messages:
    markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
    for add in buttons:
        markup.add(button(add), button('Назад'))
    bot.send_message(message.chat.id, text='Выберите удобную вам дату для записи', reply_markup = markup)

Написано более двух лет назад

Комментировать

Ваш ответ на вопрос

Войдите, чтобы написать ответ

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

JavaScript

+1 ещё

Простой
Какой код вбить, чтобы он склонял определëнное слово да ещë и повсеместно на странице/сайте?
- 1 подписчик
- 17 минут назад
- 7 просмотров
0

ответов
Python

Простой
Можно ли в питоне запустить консольную программу для каждого пользователя без ошибок?
- 1 подписчик
- 3 часа назад
- 71 просмотр
2

ответа
Python

+1 ещё

Простой
Чтоб изменения вступили в силу как делать на облаке?
- 1 подписчик
- 12 часов назад
- 62 просмотра
1

ответ
JavaScript

+1 ещё

Средний
Как завершить запрос?
- 2 подписчика
- 17 февр.
- 489 просмотров
1

ответ
Python

+1 ещё

Простой
Не отображаются данные мигрированные данные, почему???
- 1 подписчик
- 17 февр.
- 82 просмотра
0

ответов
Python

Простой
Почему при проверке Decimal на принадлежность возвращается false?
- 1 подписчик
- 16 февр.
- 117 просмотров
1

ответ
Python

+1 ещё

Простой
InvalidHeader и SSLError — как исправить?
- 1 подписчик
- 16 февр.
- 68 просмотров
2

ответа
Python

+1 ещё

Средний
Проблема с компиляцией .pyd в Nuitka: «SyntaxError: invalid or missing encoding declaration»?
- 1 подписчик
- 13 февр.
- 107 просмотров
1

ответ
Python

Простой
Как в Python удалить текст до определённых символов?
- 1 подписчик
- 12 февр.
- 194 просмотра
3

ответа
Python

Простой
Почему функция стала выдавать ошибку?
- 1 подписчик
- 12 февр.
- 210 просмотров
0

ответов
Показать ещё Загружается…

Python-разработчик

Uptrade

от 200 000 до 300 000 ₽

Python-разработчик

Dalyoko

от 2 000 до 4 000 $

Python developer

Greenway Global • Новосибирск

от 150 000 ₽

Натянуть верстку на Bitrix

19 февр. 2025, в 21:51

50000 руб./за проект

Разработать мобильное приложение iOS

19 февр. 2025, в 21:33

150000 руб./за проект

Парсинг и обход Cloudflare

19 февр. 2025, в 21:08

30000 руб./за проект

Мда... По виду это какой-то почасовой календарь.
DarkLight_723, А что в феврале будете делать? Заново расписывать все дни? Или работа бота планируется только на январь?

Просто интересно)

Answer 1 · 2023-01-26 21:38:56

Тут много путей уменьшить код.
Можно начать с того, что у вас многократно повторяется.
К примеру,

if message.text == '26.01':
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        bt1 = types.KeyboardButton('8:00')
        bt2 = types.KeyboardButton('9:00')
        bt3 = types.KeyboardButton('10:00')
        bt4 = types.KeyboardButton('11:00')
        bt5 = types.KeyboardButton('12:00')
        bt6 = types.KeyboardButton('13:00')
        bt7 = types.KeyboardButton('14:00')
        bt8 = types.KeyboardButton('15:00')
        bt9 = types.KeyboardButton('16:00')
        bt10 = types.KeyboardButton('17:00')
        bt11 = types.KeyboardButton('18:00')
        bt12 = types.KeyboardButton('19:00')
        bt13 = types.KeyboardButton('20:00')
        bt14 = types.KeyboardButton('21:00')
        back11 = types.KeyboardButton('Назад')
        markup.add(bt1, bt2, bt3, bt4, bt5, bt6, bt7, bt8, bt9, bt10, bt11, bt12, bt13, bt14, back11)
        bot.send_message(message.chat.id, text='Выберите удобную вам дату для записи', reply_markup = markup)

повторяется несколько раз. Значит, этот фрагмент можно записать один раз и затем многократно вызывать, подставляя только меняющиеся элементы.

Но и внутри видно многократное повторение нумерованной переменной, это явный признак плохого кода.
Эту проблему можно решить разными способами. В данном случае, когда значения идут равномерно (час за часом), можно их просто сгенерировать, рассчитать, и поместить в список.
Например:

first_hour = 8
last_hour = first_hour + 14
buttons = [f'{current}:00' for current in range(first_hour, last_hour)]
print(buttons)
print(buttons[0])

И этот список и передавать в нужное место вместо bt1, bt2, bt3, bt4, bt5, …

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

Следует отметить, что вам пока рано заниматься ботами. Сделайте что-то попроще, калькулятор например, игру «быки-коровы»… я не шучу.

Answer 2 · 2023-01-26 21:19:22

нечего было делать, вот помог хоть

from requests import get
import telebot
from telebot import types # для создания конопок!

bot = telebot.TeleBot('Token')
def button(name):
    return types.KeyboardButton(name)


@bot.message_handler(commands=['start', 'старт', 'Старт', 'Start']) # отслеживаем команды (/start, /старт)
def start_message(message):
    markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
    markup.add(button('Записаться'))

    mess = f' Здравствуйте {message.from_user.first_name}, нажмите на кнопку "Записаться" для продолжения'
    bot.send_message(message.chat.id, mess, parse_mode= 'html', reply_markup = markup) #


@bot.message_handler(content_types=['text'])
def bot_message(message):
    first_hour = 8
    last_hour = first_hour + 14
    buttons = [f'{current}:00' for current in range(first_hour, last_hour)]
    if message.text == 'Записаться':
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        for add in buttons:
            markup.add(button(add), button('Назад'))
        bot.send_message(message.chat.id, text='Выберите удобную вам дату для записи', reply_markup = markup)

    if message.text == '26.01':
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        for add in buttons:
            markup.add(button(add), button('Назад'))
        bot.send_message(message.chat.id, text='Выберите удобную вам дату для записи', reply_markup = markup)

    if message.text == '27.01':
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        for add in buttons:
            markup.add(button(add), button('Назад'))
        bot.send_message(message.chat.id, text='Выберите удобную вам дату для записи', reply_markup = markup)

    if message.text == '28.01':
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        for add in buttons:
            markup.add(button(add), button('Назад'))
        bot.send_message(message.chat.id, text='Выберите удобную вам дату для записи', reply_markup = markup)

    if message.text == '29.01':
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        for add in buttons:
            markup.add(button(add), button('Назад'))
        bot.send_message(message.chat.id, text='Выберите удобную вам дату для записи', reply_markup = markup)

    if message.text == '30.01':
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        for add in buttons:
            markup.add(button(add), button('Назад'))
        bot.send_message(message.chat.id, text='Выберите удобную вам дату для записи', reply_markup = markup)

    if message.text == '31.01':
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        for add in buttons:
            markup.add(button(add), button('Назад'))
        bot.send_message(message.chat.id, text='Выберите удобную вам дату для записи', reply_markup = markup)

    elif message.text == 'Назад':
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        markup.add(button('Записаться'))
        bot.send_message(message.chat.id, text='Выход в главное меню', parse_mode= 'html', reply_markup = markup)         #отправка
bot.polling(none_stop=True)

Answer 3 · 2023-01-27 00:45:31

Тут нужна одна функция. На вход получает месседж-текст. И объекты markup и bot.
Вот и все. Остально зарефакторится и схлопнется.

Как сократить данный код?

Войдите, чтобы написать ответ

Минуточку внимания

Войдите на сайт