Георгий Салтыков

Хотелость бы реальных советов и адекватной критики

Зачем столько longpoll.listen ?
Среди диалога с одним пользователем, напишите от имени другого в ЛС группы.

Глобальные переменные, тем более в многопользовательском приложении, плохо

Сначала reseived_message = event.text, потом elif reseived_message == event.text. Зачем?

Клавиатуры растянуты имхо

Нейминг переменных странный. name_user != "Назад" Почему имя проверяется на "Назад"?. Нет таких имен, либо переменная может содержать что-то другое, вместо имени.

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

text_user = event.text
if text_user == event.text

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

У вас проблемы с логикой. Всё в мгновение пойдёт по вульве как только напишет второй человек. А если будет больше, то каша будет просто у всех.

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

Как только кто-то пишет. Идете в базу, забираете состояние текущего пользователя (текущий шаг) и реагируйте на его сообщение относительно текущего шага. И так для каждого сообщения.

Критика:
У вас ужасный, не понятный код. Куча вложенных условий. Магические строки, магические числа. Переменная city сравнивается со строкой "Назад". Как это логически вообще обосновать?
Глобальные переменные, ошибочное использование принципа работы бота: longpoll.listen() - это метод для одного глобального цикла