Как выйти из обработчика сообщений в telegram bot?

Question

[Виталий Мудрый]

Здравствуйте. Решил начать изучать ка писать ботов для telegrama и столкнулся с проблемой.
Пишу пока простенькое приложение для показа погоды.
Вот index.js:

const bot = require('./bot');
const Controller = require('./controller');

bot.onText(/\/start/, msg => {
    let controller = new Controller(msg);

    controller.start();
});

bot.onText(/\/weather/, msg => {
    let controller = new Controller(msg);

    controller.weather();
});

Вот bot.js:

const TelegramBot = require('node-telegram-bot-api');

const token = '';

const bot = new TelegramBot(token, {
    webHook: {
        port: process.env.PORT || 5000
    }
});

bot.setWebHook();

module.exports = bot;

Вот controller.js

const request = require('request');
const cityToCoords = require('city-to-coords');
const bot = require('./bot');

class Controller {
    constructor(msg) {
        this.msg = msg;
    }

    get userId() {
        const {
            from: {
                id
            }
        } = this.msg;

        return id;
    }

    get userFirstName() {
        const {
            from: {
                first_name
            }
        } = this.msg;

        return first_name;
    }

    get userLastName() {
        const {
            from: {
                last_name
            }
        } = this.msg;

        return last_name;
    }

    get userUsername() {
        const {
            from: {
                username
            }
        } = this.msg;

        return username;
    }

    get chatId() {
        const {
            chat: {
                id
            }
        } = this.msg;

        return id;
    }

    start() {
        bot.sendMessage(this.userId, `Welcome, ${this.userFirstName}`);
    }

    weather() {
        bot.sendMessage(this.userId, 'Where do you want to know the weather forecast?')
            .then(() => {
                this.onText(/(.+)/, msg => {
                    return console.log(msg);
                });
            });
    }
}

module.exports = Controller;

Проблема вот тут:

bot.sendMessage(this.userId, 'Where do you want to know the weather forecast?')
   .then(() => {
       this.onText(/(.+)/, msg => {
             console.log(msg);
       });
  });

Я хочу спросить у пользователя для какого города он хочет просмотреть погоду. И потом что бы я не вводил оно все время возвращает console.log(msg). Как можно это модифицировать что бы этого не было. Спасибо

Comments

[dummyman]

Не понимаю вопроса. Ну, если не нравится console.log(msg), вы можете просто его удалить.

[Виталий Мудрый]

dummyman: console.log - это заглушка. Там должна вызваться функция для определения погоды. Я ввожу /weather -> меня спрашивает Where do you want to know the weather forecast? -> и потом я ввожу город. И в идеале оно должно остановится. То есть если я еще раз введу город оно не должно выводить по нему погоду, а у меня выводит.

[dummyman]

Виталий Мудрый: Дык элементарно же. Просто еще раз выполняешь bot.onText() с тем же RegExp, новый обработчик отменяет старый. Читайте тесты. Как правило, в тестах инфы больше чем в документации. Ну, и учитесь писать свои. По хорошему, сначала пишется структура модуля/класса, потом пишутся тесты в качестве требований к функционалу модуля. А уже потом пишется непосредственно сам код модуля. Можно настроить автозапуск тестов по сохранению файла и наблюдать прогресс своего творения не выходя из редактора кода.
https://github.com/yagop/node-telegram-bot-api/blo...

describe('#onText', function onTextSuite() {
    it('should call `onText` callback on match', function test(done) {
      const regexp = /\/onText (.+)/;
      botWebHook.onText(regexp, (msg, match) => {
        assert.equal(match[1], 'ECHO ALOHA');
        assert.ok(botWebHook.removeTextListener(regexp));
        return done();
      });
      utils.sendWebHookMessage(webHookPort2, TOKEN, {
        message: { text: '/onText ECHO ALOHA' },
      });
    });
    it('should reset the global regex state with each message', function test(done) {
      const regexp = /\/onText (.+)/g;
      botWebHook.onText(regexp, () => {
        assert.equal(regexp.lastIndex, 0);
        assert.ok(botWebHook.removeTextListener(regexp));
        return done();
      });
      utils.sendWebHookMessage(webHookPort2, TOKEN, {
        message: { text: '/onText ECHO ALOHA' },
      });
    });
  });

[dummyman]

Виталий Мудрый: я напишу в ответ, чтобы все интересующиеся видели.

Answer 1

[dummyman]

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

var rx=/(.+)/;
var emptyFn=function(){};
bot.sendMessage(this.userId, 'Where do you want to know the weather forecast?')
   .then(() => {
       this.onText(rx, msg => {
             console.log(msg);
             this.onText(rx, emptyFn);
       });
  });

Comments

[Виталий Мудрый]

Спасибо

[Виталий Мудрый]

Только есть проблема:
TypeError: this.onText is not a function
Но оно останавливаться и потом все нормально работает

[dummyman]

Виталий Мудрый:

В конструкции обратите внимание на то, что я не менял использование стрелочных функций. Если внимательно посмотреть на код, контекст (this) поменяться был не должен. Контест (this) мог поменяться только при усовершенстовании алгоритма. Так что надо искать в текущей версии скрипта.

var rx=/(.+)/;
var emptyFn=function(){};
bot.sendMessage(this.userId, 'Where do you want to know the weather forecast?')
   .then(() => {
       this.onText(rx, msg => {
             console.log(msg);
             this.onText(rx, emptyFn);
       });
  });