Почему такие странные вещи с областью видимости?

Question

[Михаил]

Здравствуйте. Код такой:

switch (data) {
        case 'buy': //как купить BTC
            const keys = message.buy.key[lng];
            keys.chat_id = msg.message.chat.id;
            keys.message_id = msg.message.message_id;
            bot.editMessageText(message.buy.text[lng], keys);
        break;

        case 'sell': //как продать BTC
            const keys = message.sell.key[lng];
            bot.editMessageText(message.sell.text[lng], keys);
        break;
    }

И в консоли вижу неожиданное:


Почему вообще компилятор ругается на переменную keys в блоке sell? Если я не определяю её и пытаюсь передать в bot.editMessageText, то консоль говорит: keys is not defined.

В чём ошибка?
З. Ы. В других местах программы переменная keys не встречатся. Переменные message и lng корректны и доступны в обоих блоках

Answer 1

[Дмитрий Беляев]

У Вас 2 раза объявление const keys внутри одного блока (весь switch является одним блоком)
можно обернуть каждый case в свой блок, тогда будет работать

Comments

[Михаил]

я просто не понимаю, почему тогда при попытке обратиться к переменной keys в блоке sell, не объявляя её там, получаю ошибку?

[Илья Шатохин]

mak_ufo: потому как это известный баг.

[Дмитрий Беляев]

mak_ufo: потому что константы имеют область видимости от своего объявления до конца блока в случае switch блок один, объявить константу 2 раза нельзя, но так как код после отклоненного case не выполняется константа не инициализируется
Илья Шатохин: это не баг, это особенность языка

[Илья Шатохин]

Дмитрий Беляев: согласен, невнимательно прочел вопрос. Кстати, SpiderMonkey лучше объясняет эту ошибку, чем V8 - "ReferenceError: can't access lexical declaration `keys' before initialization".

Answer 2

[Nick8]

Просто обернуть каждый case в свой блок

switch (data) {
        case 'buy': //как купить BTC
            {
                const keys = message.buy.key[lng];
                keys.chat_id = msg.message.chat.id;
                keys.message_id = msg.message.message_id;
                bot.editMessageText(message.buy.text[lng], keys);
                break;
            }

        case 'sell': //как продать BTC
            {
                const keys = message.sell.key[lng];
                bot.editMessageText(message.sell.text[lng], keys);
                break;
            }
    }

Ну и пиши default - никогда не знаешь, кто будет поддерживать твой код и как он может использовать твои конструкции :)