Не могу понять почему не работает цикл if {while}?

Question

[Игорь]

Закидываю скрипт в хромовскую консоль и страница полностью зависает
У меня проблеммы с написанием цыклов от слова совсем, прошу разьяснить или дать ссылочку куда почитать :)

if (jQuery("#header-outer").hasClass( "at-top" )) { 
	jQuery( ".cd-popup-container" ).addClass( "menu-margin" ); 
}while (jQuery("#header-outer").hasClass("invisible")) {
    	jQuery( ".cd-popup-container" ).removeClass( "menu-margin" );
	}

использую jQuery, а не $ что бы понимала консоль хрома

Answer 1

[Ilya Loginov]

У вас получается бесконечный while т.к. jQuery("#header-outer").hasClass("invisible") возвращает true. Если вы хотите перебрать элементы, то используйте each:

jQuery("#header-outer .invisible").each(function(){
...
});

Либо предусмотрите выход из цикла.
Также конструкции if while на сколько мне известно не существует, оберните цикл в блок else

Comments

[Игорь]

Простите за тупость я делаю какие то непонятные комбинации :D

jQuery("#header-outer .at-top").each(function(){
	jQuery( ".cd-popup-container" ).addClass( "menu-margin" );
}
jQuery("#header-outer .invisible").each(function(){
	jQuery( ".cd-popup-container" ).removeClass( "menu-margin" );
});

Как это правильно сделать?

[Ilya Loginov]

Игорь, а что сделать требуется?

[Игорь]

Ilya Loginov, У меня на сайте вылазит меню в зависимости от скрола(если опустились ниже 200 +- пикселей ), плагин в шаблоне
У меня есть поап размеров на всю страницу, я хочу сделать что бы при появлении меню у которого меняеться класс, добавлялся класс к моему попапу и он опускался на 200 пикселей вниз и наоборот

[Ilya Loginov]

Игорь, используйте setInteraval для отслеживания изменения класса и добавляйте класс к попапу

Answer 2

[Станислав Шендаков]

С самим кодом все нормально. Проблема с логикой. Если убрать лишние фигурные скобки то получится:

if (jQuery("#header-outer").hasClass( "at-top" )) 
  jQuery( ".cd-popup-container" ).addClass( "menu-margin" ); 

while (jQuery("#header-outer").hasClass("invisible")) 
   	jQuery( ".cd-popup-container" ).removeClass( "menu-margin" );

Т.е. ничего особенного. Другие возникают вопросы. Ищется элемент с id = header-outer. Id по определению может быть только один, неважно имеет он класс invisible, или не имеет, какой тут может быть while, если в цикле классы этого элемента не изменяются, функций, что могут изменить его состояние не вызывается. IMHO стОит не в хромовской панели код вводить, а записать в файл и отлаживать в хромовском отладчике присваивая промежуточные значения переменным и наблюдая их значение.