Как исправить баг?

Question

[violpeople]

Здравствуйте!
Я бы хотел узнать, почему возникает такой баг(или просто почему так происходит и как это исправить). Использую aiogram

У меня есть сплит calldata в функции, который привязан к кнопке "Добавить в корзину"

@dp.callback_query_handler(lambda c: c.data.startswith('add_'))
async def add_to_cart(call: types.CallbackQuery):
    operation = call.data.split('_')[0]
    current_item = int(call.data.split('_')[-1])
    
    item = items[current_item]
    print(operation, current_item, item, prices)

(сделал принт, чтобы видеть в консоли, что происходит, и видно, что при нажатии на кнопку "next" бот переходит ко 2 товару, но по нажатию на кнопку "add" добавляет товар под номером 3)

Также есть список, из которого бот берет инфу

items = [
    'Атомик 60|70|80 см',#0
    'Болевар 60|70|80 см',#1
    'Вайт  О"Хара 40|60 см',

Так вот вопрос - почему когда я нажимаю на эту кнопку, то у меня добавляет товар следующий?
Например, бот находиться на "Болевар", а добавляет "Вайт О"хара"

Comments

[Алан Гибизов]

items - это не словарь, а список.
Кода по-моему не хватает, чтобы понять, почему так происходит.
Не понятно, что такое call.data... догадываюсь, что оно возвращает некую строку, которую вы делите на части по знаку подчеркивания методом split, получаете список, затем берете предпоследний элемент этого списка и присваиваете ему имя current_item.
Но что там внутре, и как оно соотносится со списком items, можно только гадать. А у меня гадальные карты потерялись, шар треснул, а кофе кончился...

[violpeople]

Алан Гибизов, извиняюсь, секунду.

[Алан Гибизов]

violpeople, можете привести пример call.data, хоть print-ом его распечатайте... и на какую команду оно реагировать должно по-вашему?
Я так понимаю, что там что-то вроде 'add_2_bla-bla', которое говорит, что надо добавить куда-то (в корзину, например) элемент из списка с порядковым номером 2?
Если это так, то причина, по которой происходит сдвиг, банальна - нумерация в списках начинается с нуля, а у обычных людей - с 1. Вот и сдвиг...

[violpeople]

Алан Гибизов, смотрите - есть бот, в котором по нажатию кнопки появляется окошко
в этом окошке 4 кнопки, но интересна только 2 - "Добавить в корзину" и "Следующий".
Код для кнопки "Следующий" -

@dp.callback_query_handler(lambda c: c.data.startswith('next_'))
async def send_next_item(call: types.CallbackQuery):
    current_item = int(call.data.split('_')[-1])
    operation = call.data.split('_')[0]
    print(current_item,operation)

    kb = InlineKeyboardMarkup(row_width=2)
    next = InlineKeyboardButton(text = 'Следующий',  callback_data = f'next_{current_item+1}')
    previous = InlineKeyboardButton(text= 'Предыдущий',callback_data= 'previous_1')
    add = InlineKeyboardButton(text= 'Добавить в корзину',callback_data= f'add_{current_item+1}')
    
    kb.add(previous,next,add)
    
    bac = InlineKeyboardButton(text= 'Назад',callback_data= 'bac')
    kb.add(bac)    
    await bot.send_photo(
        chat_id=call.message.chat.id,
        photo=photos[current_item],
        caption ='Название: '+ items[current_item] +'\n\nЦена: '+ price[current_item]  + '\n\nId: '+ id[current_item],
        reply_markup= kb)

В этой части кода все работает более менее, но до момента нажатия на кнопку "Добавить товар" - происходит добавление следующего товара по списку, хотя мне нужно добавление нынешнего.

сверху вывод в консоль, когда нажимаешь на кнопку "Добавить" на позиции 1(Болевар)

[Алексей]

violpeople, f'add_{current_item+1}', может тут проблема? Почему +1, если нужен текущий товар?

[Алан Гибизов]

violpeople, надо

add = InlineKeyboardButton(text= 'Добавить в корзину',callback_data= f'add_{current_item}')

[violpeople]

Алексей, Мне кажется, что здесь показывает таки правильно, потому что эта функция для кнопки "Следующий" - типа она отправляет товар с индексом + 1(было 1 - стало 2) И кстати, показывает она правильно: нажал кнопку - получил следующий товар

[Алексей]

violpeople, я о кнопке "Добавить в корзину"

[Алан Гибизов]

violpeople, просто попробуй.

[violpeople]

Алан Гибизов, точно... Спасибо огромное! XD Черканите что то в ответы - я отмечу решением

[Алан Гибизов]

violpeople, отметь решением, если не сложно.

[violpeople]

Алексей, напишите и вы в ответ, вы ведь тоже были правы)

[Даниил Рыжов]

violpeople, Посмотрите в сторону callback_data_factory, в будущем будет проще формировать подобные клавиатуры
Пример
документация

[Алексей]

violpeople, я мимо просто пробегал) да и какой смысл?) мне статистика не важна)

[violpeople]

Bl4ckm45k, спасибо))

Answer 1

[Алан Гибизов]

violpeople, надо

add = InlineKeyboardButton(text= 'Добавить в корзину',callback_data= f'add_{current_item}')