setTimeout

[moron]

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

Код очень простой, но он почему то не работает: http://jsfiddle.net/Sc57E/ . Я пытался разобраться сам: читал синтаксис этого метода таймоут, выводил отдельно алертом кусочки скрипта...ничо у меня не вышло. В чем ошибка?

Edited June 10, 2012 by moron

[nerv]

http://jsfiddle.net/Sc57E/3/

[moron]

  On 6/10/2012 at 10:46 AM, nerv said:

http://jsfiddle.net/Sc57E/3/

Но задача была совсем другая : вы повесили на клик смену на рандомный цвет с задержкой. Моей целью было упорядоченная смена цвета квадрата, на цвета из массива. Т.е любое событие и квадрат меняет свой цвет: сначало на желтый, зеленый, красный с интервалом в определенное время, главное чтобы это было заметно человеческому глазу, а у меня получается так, что смена очень быстрая и просто не успеваем заметить других цветов, кроме последнего - черного . Задержки не получается, а тут я вовсе где-то прочитал, что в жс и вовсе таймеров никаких нет, а таймоут и интервал ничего не держат, а просто повторяют сами себя и какой-то поток идет дальше ( О_о ) .Но каждый пример с моими знаниями полезен - cпасибо.

Чучуть изменил http://jsfiddle.net/Sc57E/8/

Edited June 10, 2012 by moron

[nerv]

откуда мне было знать, что Вам нужно. Из Вашего кода ничего не понятно. http://jsfiddle.net/Sc57E/9/ Если это опять "не то", то Вам нужна анимация цвета

[moron]

Одно событие меняет цвет на квадрате несколько раз постепенно из массива с неким интервалом. За меня не надо писать скрипт, мне надо объяснить почему не работает мой.

  On 6/10/2012 at 11:31 AM, nerv said:

Из Вашего кода ничего не понятно.

И не понятно, а что именно? как писать лучше?

Edited June 10, 2012 by moron

[nerv]

  Quote

За меня не надо писать скрипт, мне надо объяснить почему не работает мой.

а почему он должен работать? ) Предлагаю Вам написать к нему комментарии и выложить его сюда.

[moron]

http://jsfiddle.net/Sc57E/12/ раскомментировал как мог.

[NeoXidizer]

Твоя ошибка - думать, что когда цикл дойдет до таймера, он остановится и подождет его выполнения, а затем возобновится. На самом деле, функции setTimeout и setInterval откладывают выполнение только того действия или функции, которая указана в скобках. Получается, каждая итерация цикла запускала таймер заново, а таймер, при запуске, запускал действие и свою попытку "подождать", что у него и не получалось.

Правильно сделать вот так

http://jsfiddle.net/CJ2Yf/

[nerv]

  On 6/10/2012 at 9:36 PM, NeoXidizer said:

Правильно сделать вот так

var myArray = new Array ("yellow", "green", "red", "blue", "black");
function changeColor(id){
    var i=0;
    var interval = setInterval(function() {
        document.getElementById(id).style.backgroundColor=myArray[i++];
        if(i >= myArray.length) clearInterval(interval);
    }, 1000);
}
changeColor('ololo');

ну-ну. Сколько раз вызывается таймер, столько ищется элемент по id.

moron,

1. избыточность в именах переменных (myArray/myColor). Зачем mу? Ну твои, они, твои, чьи же еще )

2. document.getElementById(rainbow).style.backgroundColor=myColor; - это я уже комментировал, смотри выше

3. все значения цветастого массива - массива цветов

4. //объявляем перменную муКолор, в которую будут постепенно записаны все переменные из массива - неверное трактование. В переменную будут поочередно записываться строки из массива.

5. про создание массива с помощью конструктора молчу...

6. еще не есть гуд писать функцию прямо в таймер, но для примера сойдет

Edited June 11, 2012 by nerv

[NeoXidizer]

а что "ну-ну" то?

[hedgehog]

  On 6/11/2012 at 6:37 AM, nerv said:

6. еще не есть гуд писать функцию прямо в таймер, но для примера сойдет

Извиняюсь за офтопик... А если function используется для передачи параметров функции, вызываемой по таймеру? Я использую такую запись:

setInterval(function(){doSomething(arg1,arg2)},5000)

[nerv]

NeoXidizer, так я там все написал ) Если уж говорите, что "так правильно", то делайте правильно все )

hedgehog, правильно заметили. Я выразился не совсем точно. Использовать обертку (как в Вашем случае) вполне допустимо (скорее необходимо), а вот писать "многокода" прямо в таймер, все-таки не стоит (на мой взгляд).

[NeoXidizer]

  On 6/11/2012 at 3:46 PM, nerv said:

NeoXidizer, так я там все написал ) Если уж говорите, что "так правильно", то делайте правильно все )

это глупый перфекционизм, который мешает лишь разработчику. У автора не работал его код, я его переписал, показав, как правильно в его случае применить таймер

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

[nerv]

>чем написать эту самую строчку внутри таймера, как это сделал я

на этот счет я ответил выше

>это глупый перфекционизм, который мешает лишь разработчику

если человек хочет называться (js-)разработчиком, он должен стремиться к полному контролю над кодом (и как следствие производительностью). Считаете, что это глупо? Пожалуйста, это Ваше мнение.

p.s.: себя я js-разработчиком не считаю

Edited June 12, 2012 by nerv