Как правильно объявить глобальные переменные

Question

Как правильно объявить глобальные переменные

1

Я знаю о подъеме.

Но мне интересно, какое рекомендуемое решение для глобального массива, например, работает как таблица поиска. Массив, который вы бы объявили постоянным и сделать глобальным на большинстве языков.

Подобно:

 var lookup = ["010101010", "0100101010", "01010101010", "etc"];

 function foo() {    
     // lookup is undefined
 }

Наверное, я мог бы передать его. Или сделать его частью window.document. Или какой-то трюк с этим, что я не совсем понимаю. Какой лучший способ сделать это?

Изменить: Хорошо, после прочтения комментариев, теперь я понимаю, что проблема, которую я испытывал, не была вызвана подъемом. В моем коде была и другая проблема. Он работает сейчас. Спасибо, что поняли это.

fwend 20 сен. 2017, в 18:57

Источник

3

Лучший способ - полностью избежать глобальных переменных.
Teemu 20 сен. 2017, в 16:16
4

lookup не определен в этой функции, если вы определяете lookup перед вызовом foo() .
Mark Meyer 20 сен. 2017, в 16:16

Теги:

javascript

hoisting

global

2 ответа

0

Если вы хотите сохранить некоторые статические данные, используйте статические методы:

class CONSTANT {

  /**
   * Public method that will return a new array you can manipulate
   *
   * It different of this.MY_CONSTANT = [X, Y, Z];
   *
   * Because accessing this.MY_CONSTANT will gives you an access to
   * the same element (if you modify it, it will get
   * modified for the next use of it)
   */
  static get MY_CONSTANT() {
    return [ 
        '010101010', 
        '0100101010',
        '01010101010',
    ];
  }

}

Для доступа:

CONSTANT.MY_CONSTANT

Ps: Это требует ES6

Grégory NEUT 20 сен. 2017, в 15:15

0

Я думаю, что вы хотите, чтобы class CONSTANT {} не class CONSTANT() {}
Mark Meyer 20 сен. 2017, в 16:23
0

@Mark_M о да, определенно :) опечатка, спасибо
Grégory NEUT 20 сен. 2017, в 16:24
0

Объявления классов не поднимаются, что означает, что вам нужно сделать это объявление, прежде чем вы сможете его использовать. Почему вы делаете это вместо простого объявления const в верхней части файла.
Mark Meyer 20 сен. 2017, в 16:26
0

Потому что я думаю, что лучше централизовать постоянные значения в вашем коде, чтобы его было проще читать. Вы также можете использовать const в верхней части файла ofc. Это зависит от константы, которую вы делаете, и от того, требуется ли она где-то еще в вашем коде.
Grégory NEUT 20 сен. 2017, в 16:28

Показать ещё 2 комментария

Ещё вопросы

Лучший способ - полностью избежать глобальных переменных.
lookup не определен в этой функции, если вы определяете lookup перед вызовом foo() .
Я думаю, что вы хотите, чтобы class CONSTANT {} не class CONSTANT() {}
@Mark_M о да, определенно :) опечатка, спасибо
Объявления классов не поднимаются, что означает, что вам нужно сделать это объявление, прежде чем вы сможете его использовать. Почему вы делаете это вместо простого объявления const в верхней части файла.
Потому что я думаю, что лучше централизовать постоянные значения в вашем коде, чтобы его было проще читать. Вы также можете использовать const в верхней части файла ofc. Это зависит от константы, которую вы делаете, и от того, требуется ли она где-то еще в вашем коде.

VHS · Accepted Answer · 2017-09-20T15-17-00.000Z

Просто убедитесь, что вы объявляете lookup до foo функции foo. См. Следующий фрагмент. Если вы объявите его после foo, вы получите ошибку undefined.

var lookup = ["010101010", "0100101010", "01010101010", "etc"];
foo();
 function foo() {    
     console.log(lookup);
 }

Вы не получите «неопределенную ошибку», вы получите значение как undefined .
@teemu, простите за неправильное словоблудие. Я подправил это.
Я имею в виду, что никаких ошибок не будет, вы просто получите undefined значение для lookup если оно объявлено с использованием var после вызова функции (из-за подъема). Значение не является ошибкой.