Можно ли написать функцию isset в javascript?

Question

[Dubrovin]

В некоторых участках кода есть потребность проверять наличие переменной, чтобы не крашилось приложение с фатальной ошибкой myVar is not defined:

( 'undefined' !== typeof myVar ) && ( null !== myVar )

В данном примере код заходит в первое условие, в котором переменная myVar стоит после оператора typeof, фатальной ошибки не происходит
Я хочу сделать функцию-обёртку над этим условием, вот таким способом:

isset: ( variable ) => {
	return ( 'undefined' !== typeof ( variable ) ) && ( null !== variable );
},

Далее, если myVar не определена, то вызов isset( myVar ) крашит приложение с ошибкой myVar is not defined.
Что я делаю не так и можно ли реализовать такую обёртку? Или есть более красивое решение?

Answer 1

[Dubrovin]

В общем нашёл решение сам, оно оказалось достаточно простым:
JS для глобальных переменных, которые не определены будет вызывать ошибку myVar is not defined, но если написать в коде window.myVar, то ошибки не будет
Конкретно в моём примере под WordPress нужно было, чтобы скрипт получал url для ajax запроса. В админке это window.ajaxurl, а во фронте k.ajaxurl, код выглядит следующим образом:
let url = window.ajaxurl || k.ajaxurl;
Если мы в админке, то url будет равен ajaxurl, если мы во фронте, то k.ajaxurl

Comments

[Денис]

Если вопрос касается WordPress, то может быть решение норм.

С точки зрения работы с JS - решение некорректно.
Использование глобальных переменных - крайне некорректно!
Да, язык позволяет вам так сделать, но это небезопасно.

[Dubrovin]

Бред, вот вам пример - jQuery использует глобальную переменную jQuery в качестве точки входа в приложение. Почитайте про паттерн фасад. По вашему, если jQuery использует глобальную переменную jQuery, то это не безопасно?
Конкретно в моём случае есть 2 глобальных переменных - первая это ajaxurl в window - её создаёт WordPress в админке, эту переменную используют все темы и плагины под WordPress для отправки ajax-запросов.
Вторая глобальная переменная - это k - точка входа в приложение (аналог переменной jQuery).
Есть ситуации, когда вы не можете обойтись без глобальных переменных, и язык предоставляет такие возможности не просто так.

[Игорь]

Не использовать стрелочную функцию там где не нужно.

Использовать контекст this внутри функции

Answer 2

[Сергей Краснов]

"myVar is not defined" - ошибка говорит о том, что такая переменная в принципе не была объявлена. А делать переменную свойством глобального объекта - это плохая практика. Всё можно написать без использования глобальных переменных.

Comments

[Dubrovin]

Отлично, расскажите это создателям самой популярной CMS в мире - WordPress, они объявляют глобальную переменную ajaxurl в админке, чтобы у всех плагинов/тем был к ней доступ.
Вы глубоко заблуждаетесь на счёт того, что использование глобальных переменных - это плохая практика, это как если говорить, что вообще использование переменных - это плохая практика. И да, не всё можно написать без их использования.

[Сергей Краснов]

Dubrovin, Сколько лет вы занимайтесь программированием, что бы критиковать людей? Вы задали вопрос, наверно хотите получить ответ. Так что же тогда не следуйте советам более опытных экспертов?

[Dubrovin]

Сергей Краснов, вы, во-первых, невнимательно прочитали вопрос, во-вторых, не обратили внимания, что ответ уже был дан, и, в-третьих, дали ответ, который вообще никак не относится к теме вопроса.

Answer 3

[Антон Швец]

Оберните код, где будет использоваться переменная в try
Например

try {
  unknownVar
} catch (e) {
 if (e instanceof ReferenceError) {
   console.log('Переменная unknownVar не определена')
 }
}

Comments

[Dubrovin]

Если этот код поместить в функцию-обёртку, то ошибка остаётся. А вне обёртки кода ещё больше

[Adamos]

Это правильный подход... но это не решение заявленного вопроса ;)

[Adamos]

Dubrovin, дело, видимо, в том, что код, в котором часто приходится проверять, объявлена ли переменная, с которой вы собираетесь работать, в принципе - попахивает.

[Антон Швец]

Adamos, ответ на вопрос - "нельзя написать такую функцию"
С одним исключением. Если не определяете переменную через var в глобальной области видимости. В этом случае она становится свойством глобального объекта, и можно сделать проверку на 'myVar' in window скажем

[Dubrovin]

Антон Швец, я уже нашёл ответ на свой вопрос, но, кстати, на ум ещё пришло вот такое решение (на всякий случай):

isset = ( typeofVar ) => {
  return ( 'undefined' !== typeofVar );
};

isset( typeof myVar); //false

[Антон Швец]

Dubrovin,

const x = undefined;
typeof x; //-> "undefined"

но переменная есть

[Dubrovin]

Антон Швец, согласен, но это "лабораторный" случай, в реальном приложении переменная или не определена, или в ней лежит необходимая строка

[Антон Швец]

Dubrovin, можно сделать через eval, но не уверен что везде сработает. И это максимально плохое решение :)

const isset = varName => new Function(`
try {
  console.log(${varName})
} catch(e) {
  console.log('undefined');
}`)();
isset('unknown') //-> "undefined"

[Антон Швец]

Dubrovin, в реальном приложении надо таки ловить исключение. Кто знает в какой момент переменная переменная перестанет быть глобальной.

[Dubrovin]

Антон Швец, мне нужно было присвоить переменной url значение переменной ajaxurl, а если его нет, то присвоить k.ajaxurl. В данном случае мне нужно выполнить конкретное действие - присвоение, без которого следующий код не будет выполнятся. Ловить исключения и выводить какие-то ошибки в консоль не уместно. Есть только 2 варианта - или переменная ajaxurl определена, или переменная k.ajaxurl определена. Нет варианта - переменные не определены. Результатом должно быть присвоение без каких-либо ошибок или предупреждений в консоли.
В моём примере проблема была в том, что вначале код был вот такой:

let url = ( ( 'undefined' !== typeof ajaxurl ) && ( null !== ajaxurl ) ) ? ajaxurl : k.ajaxurl;

и копипастить его в несколько мест приложения в конкретных компонентах не хотелось, в результате было найдено решение проще:
let url = window.ajaxurl || k.ajaxurl;
Цель достигнута, кода меньше и он проще читается и не вызывает ошибок :)

Answer 4

[Руслан Лопатин]

Такая проверка выполняется проще: myVar != null

Comments

[Dubrovin]

Будет фатальная ошибка myVar is not defined

[Adamos]

Ваш ответ проверяется еще проще: нажимаем F12 и вводим этот текст в консоли

[Игорь]

Dubrovin, return нужно делать

...
return myvar ? true : false

Answer 5

[Adamos]

typeof ( variable )
Скобки уберите.

Comments

[Dubrovin]

Да, это опечатка, это никак не влияет на результат

[Adamos]

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

[Антон Швец]

Adamos, именно. В функцию будет передаваться значение переменной, а ее нету.

[Adamos]

Dubrovin, боюсь, в JS вы сможете сделать только функцию типа

function isset(context, variable_name) {..}
isset(window, 'myVar');

Но сами вряд ли станете пользоваться таким монстром.

[Dubrovin]

Антон Швец, я это прекрасно понимаю, вопрос в том, как не дублировать каждый раз длинное условие

Answer 6

[Алексей Арх]

javascript isset

// Проверяем ключи
let test = {
  a: {
    b: [0, 1]
  }
};

console.log(test.isset('a.b'))   // true
console.log(test.isset('a.b.1')) // true
console.log(test.isset('a.b.5')) // false
console.log(test.isset('a.c'))   // false
console.log('abv'.isset('0')) // true