Как сделать пошагового телеграм бота на PHP?

Question

[Артур]

Приветствую.

Пишу телеграм бота используя библиотеку https://github.com/TelegramBot/Api и Yii2

Задача: сделать пошаговое заполнение профиля.

У бота написал такой код:

$bot->command('start', function ($message) use ($bot) {
	$user = new User();
	$user->setAttributes(['t_id' => $message->getFrom()->getId()]);
	$user->save();
	$bot->sendMessage($update->getMessage()->getChat()->getId(), "Введите имя: ");
});

// Шаг 1, вводим имя
$bot->on(function (Update $update) use ($bot) {
	$user = User::find()->where(['t_id' => $update->getMessage()->getFrom()->getId())->one();
	$user->setAttribute('name', $update->getMessage()->getText());
	$user->save();
	$bot->sendMessage($update->getMessage()->getChat()->getId(), "Введите фамилию: ");
}, function (Update $update) {
	if (!$update->getMessage()) {
		return false;
	}

	$user = User::find()->where(['t_id' => $update->getMessage()->getFrom()->getId())->one();
	if (!$user) {
		return false;
	}
	
	return empty($user->name);
});

// Шаг 2, вводим фамилию
$bot->on(function (Update $update) use ($bot) {
	$user = User::find()->where(['t_id' => $update->getMessage()->getFrom()->getId())->one();
	$user->setAttribute('lastname', $update->getMessage()->getText());
	$user->save();
	$bot->sendMessage($update->getMessage()->getChat()->getId(), "Имя и фамилия введены! Спасибо");
}, function (Update $update) {
	if (!$update->getMessage()) {
		return false;
	}

	$user = User::find()->where(['t_id' => $update->getMessage()->getFrom()->getId())->one();
	if (!$user) {
		return false;
	}
	
	return empty($user->lastname);
});

В on() первая функция выполняется, если результат второй функции true.

В коде не нравится разбиение логики и зависимости - если изменяем в одной функции критерии выполнения, в другой надо сменить тоже - например если надо поменять порядок ввода имя и фамилия, или допустим если нужно будет добавить после имен отчество, придется переписывать почти всё.

И очень сложно поддерживать такой код, когда шагов уже не 2 а 5-6 уже становится сложно в этом разобраться.

Подскажите, как можно такой алгоритм упростить, сделать более легко поддерживаемым? Чтобы легче было менять местами шаги и дополнять их.

Буду благодарен за ссылки на литературу, ключевики которые надо гуглить, гитхаб ссылкам с хорошими примерами, и любым другим советам/критике :)

Answer 1

[Максим]

Если правильно понял вас, то Используйте Listeners и весь функционал обработчиках производите там. То есть бот посылает команду и event вы вешаете на это событие на свой listener. В итоге получается такая схема событие -> обработчик. Когда событие возникает тогда и работает обработчик.

В обработчике уже помещаете логику в зависимости от переданных данных.

Comments

[Артур]

Т.е. написать один перехватчик $bot->on(...) который отправляет все события на мой обработчик, уже в котором я уже пишу логику и согласно логике вызываю нужные нужные функции в нужной последовательности? Верно я вас понял?

[Максим]

Артур, $bot->on(...) это как раз таки и есть событие от бота. Вы на него вешаете анонимную фунцию. Вместо этого можно вешать свой класс (обработчк). Вы можете поступить так:
1. Создать свой один обработчик, где пишете всю эту логику.
2. Пользоваться анонимной функцией, как и сейчас, но создавать свои события в зависимости от данных и свои обработчики в зависимости от события. Но тут вам нужен будет:
- EventDispatcher
- Event
- Listener
3. Создайте свой сервис:
- из команд по паттерну CQRS
- из UseCase по паттерну DDD
4. Создать сервис UseCase и события в нём.

Я бы в таком случае рекомендовал третий вариант. Но события придают гибкость. Вы можете навесить любой обработчик когда угодно и не лезть в код.

Бот запускается и пользователь выбирает команду. Вы получаете эту команду от бота. Если команда === Фамилия, то вызываете сервис $service->addLastName($name);

[Артур]

Максим, понял, наверно с событиями самый подходящий вариант. Спасибо!

[Максим]

Артур, не за что) Попробуйте так