Проверка существования переменной , и элемента

[Jin X]

Делаю так:

function myFunc(x);
{
  if (x) var y = document.getElementById(x);
  . . .
  if (y) y.innerHTML = 'бла-бла-бла';
}

Грамотно ли так проверять существование переменной y (несмотря на то, что этот код работает)?

Добавлено 05.05.15, 10:49
И заодно сюда же вопрос: как узнать, существует ли элемент с именем x ?

Добавлено 05.05.15, 10:58
Вот ещё один вариант:

function myFunc(x);
{
  var y = document.getElementById(x);
  . . .
  if (y) y.innerHTML = 'бла-бла-бла';
}

Добавлено 05.05.15, 11:00
Т.е. в первом случае проверяется значение x (если оно НЕпустое, то переменная y создаётся и далее идёт присвоение innerHTML).
Во втором случае y проверяется элемент x; если он существует, то идёт присвоение innerHTML.

[Serafim]

Цитата Jin X @ 05.05.15, 10:25

Грамотно ли так проверять существование переменной y

И да и нет. Код будет работать, но в случае y = 0, y = null, y = [] - работать не будет.

if (y !== undefined) { ... }

Добавлено 05.05.15, 12:09
Ой, не заметил, что y - это DOM объект. Тогда всё ок =)

[Jin X]

Погоди.

  var y = document.getElementById('blabla');
  alert(y);

выдаёт null, при этом всё работает...

Добавлено 05.05.15, 13:08
И вот так тоже срабатывает:

  var y = document.getElementById('blabla');
  if (y == null) alert('null');

=== тоже

[Serafim]

Цитата Jin X @ 05.05.15, 13:07

выдаёт null, при этом всё работает...

потому что переменная существует, но содержит null. Еслиб не существовала (вопрос же твой, цитата: "Грамотно ли так проверять существование переменной") - был бы андефайн.

[Jin X]

Так, ты же говоришь:

Цитата Serafim @ 05.05.15, 12:08

в случае y = 0, y = null, y = [] - работать не будет.

[Serafim]

Ну так y у нас DOM-объект, а null отрицательное значение, т.е. которое и следует игнорировать.

Добавлено 06.05.15, 15:42
Не будет работать в случаях:

1) var a = false || null; (т.е. ожидаешь либо false, либо null)
2) var a = false || [];
3) var a = false || 0;
4) var a = false || '';

В этих случаях проверка через if(a) ожидая, что она будет успешной только при a = false - ошибочна.

[Jin X]

Ок, а в первом случае, когда:

function myFunc(x);
{
  if (x) var y = document.getElementById(x);
  . . .
  if (y) y.innerHTML = 'бла-бла-бла';
}

Проверка if (y) корректна? Ведь, если if (x) не выполнится, переменная y не создастся.

Добавлено 06.05.15, 18:11
У меня, например, конструкция:

if (y !== undefined) alert('ok!'); else alert('undefined!');

вообще не работает, если y не определено!
Uncaught ReferenceError: y is not defined

Вот так, может?

if (typeof y != 'undefined') alert('ok!'); else alert('undefined!');

[Serafim]

Цитата Jin X @ 06.05.15, 18:05

Проверка if (y) корректна?

Цитата Serafim @ 05.05.15, 12:08

Ой, не заметил, что y - это DOM объект. Тогда всё ок =)

Добавлено 06.05.15, 21:05

Цитата Jin X @ 06.05.15, 18:05

Вот так, может?

точно сглупил я.

CoffeScript предлагает такой же вариант

Сообщение отредактировано: Serafim - 06.05.15, 21:06

[fatalist]

Помнится Дядя Боб меня научил:

if (typeof(element) != 'undefined')
   //exists

Добавлено 07.05.15, 03:48

Цитата Jin X @ 06.05.15, 18:05

Вот так, может?

а, ну вот оно

Сообщение отредактировано: fatalist - 07.05.15, 03:48

[Jin X]

Цитата Serafim @ 06.05.15, 21:04

Цитата Jin X @ 06.05.15, 18:05

Проверка if (y) корректна?

Цитата Serafim @ 05.05.15, 12:08

Ой, не заметил, что y - это DOM объект. Тогда всё ок =)

y - это DOM-объект только при выполнении условия if (x). А вот при НЕвыполнении y будет undefined!
Uncaught ReferenceError: y is not defined

[Serafim]

ты же сам сказал, что null, а не undefined

Цитата Jin X @ 05.05.15, 13:07

выдаёт null, при этом всё работает...

Добавлено 07.05.15, 10:27
Короче, проверять наличие всегда стоит так:

typeof some !== "undefined" && some !== null

Обычно в любых вопросах по "микроалгоритмам" JS я обращаюсь к кофейной реализации, там много всяких приёмчиков, которые упрощают жизнь, в том числе и данный случай с проверкой: http://coffeescript.org/#try:some%20%3D%20...7Bsome%3F%7D%22

[Jin X]

Цитата Serafim @ 07.05.15, 10:23

ты же сам сказал, что null, а не undefined

ты видишь, что там 2 разных примера?
в первом будет undefined, во втором ( var y = document.getElementById(x); ) = null, если элемента с именем x не сущестует.

Сообщение отредактировано: Jin X - 07.05.15, 20:52

[Serafim]

Цитата Jin X @ 07.05.15, 20:51

ты видишь, что там 2 разных примера?

Не обратил внимание, что там другая переменная

Добавлено 07.05.15, 21:07
Если аргумент не задан, то очевидно undefined, что, учитывая замечание fatalist - проверяется с помощью typeof x !== 'undefined' && x !== null. y возвращает либо DOM, либо null, что проверяется либо точно так же, либо как ты написал

Сообщение отредактировано: Serafim - 07.05.15, 21:09