Не выполняется условие if

[MacPherson]

Вчера начал знакомиться с JS, а сегодня уже пишу слайдер.

Не выполняется условие

if (document.getElementById('container').style.right == shirinaBlokov)
        {} // не выполняется
        else
        {... // здесь все подхватывает и работает не смотря ни на что

(если поставить вместо shirinaBlokov - '1000px' то все работает)

Думаю ответ здесь

var shirinaBlokov = massivDiv.length;
    shirinaBlokov *= summa;

Или может я пытаюсь сравнить различные типы? Style выдирается как строка, а shirinaBlokov числовой.. хотя оператор == сравнивает значение, но не типы

Edited June 10, 2012 by MacPherson

[s0rr0w]

if (document.getElementById('container').style.right == shirinaBlokov)

Если вы ничего не записывали в это поле, то там по умолчанию undefined.

[MacPherson]

Почему же не записывал? Записал пустые {} - работает. Весь код в else.

В моём случае надо было дописать if (document.getElementById('container').style.right == shirinaBlokov + 'px')

Edited June 10, 2012 by MacPherson

[nerv]

Вчера начал знакомиться с JS, а сегодня уже пишу слайдер.

забудьте. Если так уж приперло, используйте готовые "комбайны", например, jquery + плагины (или как там они называются)

Edited June 11, 2012 by nerv

[MacPherson]

Зачем забить? Я сделал слайдер Отлично работает

[nerv]

Если работает, это еще не значит, что "Отлично работает". В инете тонны говнокода, который работает. Я не утверждаю, что у Вас плохой код (или хороший), т.к. я его не видел (да и не очень хочется).

[Great Rash]

В моём случае надо было дописать if (document.getElementById('container').style.right == shirinaBlokov + 'px')

Лучше написать так:

  if ( parseInt( document.getElementById( 'container' ).style.right ) == shirinaBlokov)