ПОМОГИТЕ, ПОЖАЛУЙСТА! В данном коде не работает setTimeout()

@grytsiuko · Регистрация: 07.02.2015

Студворк — интернет-сервис помощи студентам

Здравствуйте!
Помогите, пожалуйста, решить проблему данного кода. Не вызывается повторно функция через setTimeout(). Один раз выполняется и всё.
li - это массив элементов списка меню.

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
function cycl (interval, time) {
    for (i = 0; i<li.length; i++) {
            if (li[i].style.height!='0em') {
                function hide() {
                    height = parseFloat(li[i].style.height);
                    height -= interval;
                    li[i].style.height=height+'em';
                    if (li[i].style.height!='0em') {
                        setTimeout(function(){hide()}, 10);
                    };
                    mediaz=1;
                };
                hide();
            } else {
                function show() {
                    height = parseFloat(li[i].style.height);
                    height += interval;
                    mediaz=0;
                    li[i].style.height=height+'em';
                    if (li[i].style.height != inter+'em') {
                        setTimeout(function(){show()}, 10);
                    };
                };
                show();
            };
    };
};

@vovandr · 07.02.2015, 18:50

Попробуйте так

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
function hide(i, interval) {
        height = parseFloat(li[i].style.height);
        height -= interval;
        li[i].style.height=height+'em';
        if (li[i].style.height!='0em') {
            setTimeout(function(){hide(i, interval)}, 10);
        };
        mediaz=1;
    }
    
    function cycl (interval, time) {
        for (i = 0; i<li.length; i++) {
            if (li[i].style.height!='0em') {
                (function (i, interval) {
                    hide(i, interval);
                })(i, interval);
            } else {
                function show() {
                    height = parseFloat(li[i].style.height);
                    height += interval;
                    mediaz=0;
                    li[i].style.height=height+'em';
                    if (li[i].style.height != inter+'em') {
                        setTimeout(function(){show()}, 10);
                    };
                };
                show();
            };
        };
    };

@Curry · 07.02.2015, 18:52

Может массив li пустой? Или вызывается но не виден эффект? Попробуйте alert() -ом посмотреть или console.log().
Однозначно, интервал слишком мал - 10 мс, начните с 500.

@grytsiuko · 07.02.2015, 19:12 **[ТС]**

Массив li не пустой, это было проверено еще на начале написания скрипта. Сменил интервал - всё равно, функция выполняется только один раз.

Добавлено через 3 минуты
Смена мест функций сделала еще хуже, скрипт теперь вообще не работает.
Вообще не понятно, чего оно не вызывается..(

@Curry · 07.02.2015, 19:22

Я повторил Ваш способ, и даже упростил, и всё работает в Опере, хроме и ИЕ11

HTML5
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
<HTML>
<HEAD>
<SCRIPT LANGUAGE=javascript>
<!--
 
var cnt=0;
function onLoad(){
    function show() {
        txOut.innerText= ++cnt;
//        setTimeout(function(){show()}, 10);
        setTimeout(show, 10);
    };
    show();
}
 
//-->
</SCRIPT>
</HEAD>
<BODY onLoad="onLoad()">
<p id=txOut><p>
</BODY>
</HTML>

Пробуйте отладочную печать, искать куда не проходит.

@grytsiuko · 07.02.2015, 19:30 **[ТС]**

Только сейчас заметил, что после одного выполнения функции выскакивает ошибка Uncaught TypeError: Cannot read property 'style' of undefined.
Попробовал даже передать параметром этот массив, но всё равно

JavaScript
1
2
3
4
5
6
7
8
9
10
function hide(li) {
                    height = parseFloat(li[i].style.height);
                    height -= interval;
                    li[i].style.height=height+'em';
                    if (li[i].style.height!='0em') {
                        setTimeout(function(){hide(li)}, 10);
                    };
                    mediaz=1;
                };
                hide(li);

я выложил только эту функцию, вторая идентичная (скрыть и показать)

@vovandr · 07.02.2015, 19:44

Только сейчас заметил, что после одного выполнения функции выскакивает ошибка Uncaught TypeError: Cannot read property 'style' of undefined.

Ясно вам дал понять, что переменную надо сохранять в замыкании, но вы по-прежжнему продолжаете искать ошибку в вашем говнокоде

@Curry · 07.02.2015, 19:44

Сообщение от grytsiuko

Cannot read property 'style' of undefined

Значит у Вас li[i] undefined.

@vovandr · 07.02.2015, 19:45

ф-ю show перенесите по аналогии, как я вам написал

@Curry · 07.02.2015, 19:56

vovandr, Вы правы.
Можно, кажется, и не переносить, а объявить дополнительный var

JavaScript
1
2
3
4
5
6
7
8
9
10
                var i2=i;
                function hide() {
                    height = parseFloat(li[i2].style.height);
                    height -= interval;
                    li[i2].style.height=height+'em';
                    if (li[i2].style.height!='0em') {
                        setTimeout(function(){hide()}, 10);
                    };
                    mediaz=1;
                };

@grytsiuko · 07.02.2015, 19:58 **[ТС]**

Офигеть, работает!!
Спасибо, vovandr!
Если вам не сложно, объясните, пожалуйста, что значит это:

JavaScript
1
2
3
(function (i, interval) {
                    hide(i, interval);
                })(i, interval);

т.е. зачем функция взята в скобки, еще и вторые скобки рядом?
Просто js я начал изучать недавно, не всё познал)
Буду благодарен, если объясните.

@vovandr · 07.02.2015, 20:05

grytsiuko создается функция вызываемая не месте. В качестве аргумента получает i. Таким образом функция, которая находится внутри нее (hide и show) с помощью замыкания имеет доступ к текущему значению i. В вашем случае hide и show вызывалась на последней итерации...когда фактически весь цикл for заканчивался. Почитайте про замыкания в js лучше, так как объяснять у меня плохо получается

@grytsiuko · 07.02.2015, 20:12 **[ТС]**

Две недели мучался над этим скриптом. И, когда обратился на киберфорум, сразу же помогли!
Большое спасибо!

Новые блоги и статьи Все статьи Все блоги /
Подстановка значения реквизита справочника в табличную часть документа Maks 10.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеПерсонала", разработанного в конфигурации КА2. Задача: при выборе сотрудника (справочник Сотрудники) в ТЧ документа. . .	Очистка реквизитов документа при копировании Maks 09.04.2026 Алгоритм из решения ниже применим как для типовых, так и для нетиповых документов на самых различных конфигурациях. Задача: при копировании документа очищать определенные реквизиты и табличную. . .	модель ЗдравоСохранения 8. Подготовка к разному выполнению заданий anaschu 08.04.2026 https:/ / github. com/ shumilovas/ med2. git main ветка * содержимое блока дэлэй из старой модели теперь внутри зайца новой модели 8ATzM_2aurI	Блокировка документа от изменений, если он открыт у другого пользователя Maks 08.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа, разработанного в конфигурации КА2. Задача: запретить редактирование документа, если он открыт у другого пользователя. / / . . .
Система безопасности+живучести для сервера-слоя интернета (сети). Двойная привязка. Hrethgir 08.04.2026 Далее были размышления о системе безопасности. Сообщения с наклонным текстом - мои. А как нам будет можно проверить, что ссылка наша, а не подделана хулиганами, которая выбросит на другую ветку и. . .	Модель ЗдрввоСохранения 7: больше работников, больше ресурсов. anaschu 08.04.2026 работников и заданий может быть сколько угодно, но настроено всё так, что используется пока что только 20% kYBz3eJf3jQ	Дальние перспективы сервера - слоя сети с космологическим дизайном интефейса карты и логики. Hrethgir 07.04.2026 Дальнейшее ближайшее планирование вывело к размышлениям над дальними перспективами. И вот тут может быть даже будут нужны оценки специалистов, так как в дальних перспективах всё может очень сильно. . .	Горе от ума kumehtar 07.04.2026 Эта мне ментальная установка, что вот прямо сейчас, мол, мне для полного счастья не хватает (нужное вписать), и когда я этого достигну - тогда и полный кайф. Одна из самых сильных ловушек на пути. . . .

ПОМОГИТЕ, ПОЖАЛУЙСТА! В данном коде не работает setTimeout()

Решение

Решение

@Curry Модератор 5158 / 3490 / 536 Регистрация: 01.06.2013 Сообщений: 7,581 Записей в блоге: 9
	07.02.2015, 18:52
	Может массив li пустой? Или вызывается но не виден эффект? Попробуйте alert() -ом посмотреть или console.log(). Однозначно, интервал слишком мал - 10 мс, начните с 500. 1

@grytsiuko 3 / 3 / 0 Регистрация: 07.02.2015 Сообщений: 24
	07.02.2015, 19:12 [ТС]
	Массив li не пустой, это было проверено еще на начале написания скрипта. Сменил интервал - всё равно, функция выполняется только один раз. Добавлено через 3 минуты Смена мест функций сделала еще хуже, скрипт теперь вообще не работает. Вообще не понятно, чего оно не вызывается..( 0

@vovandr 636 / 523 / 195 Регистрация: 19.08.2013 Сообщений: 1,400
	07.02.2015, 19:45
	Сообщение было отмечено grytsiuko как решение Решение ф-ю show перенесите по аналогии, как я вам написал 2

@vovandr 636 / 523 / 195 Регистрация: 19.08.2013 Сообщений: 1,400
	07.02.2015, 20:05
	Сообщение было отмечено grytsiuko как решение Решение grytsiuko создается функция вызываемая не месте. В качестве аргумента получает i. Таким образом функция, которая находится внутри нее (hide и show) с помощью замыкания имеет доступ к текущему значению i. В вашем случае hide и show вызывалась на последней итерации...когда фактически весь цикл for заканчивался. Почитайте про замыкания в js лучше, так как объяснять у меня плохо получается 1

@grytsiuko 3 / 3 / 0 Регистрация: 07.02.2015 Сообщений: 24
	07.02.2015, 20:12 [ТС]
	Две недели мучался над этим скриптом. И, когда обратился на киберфорум, сразу же помогли! Большое спасибо! 0