setInterval работает неправильно!

@BART · Регистрация: 17.11.2009

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

Сдравствуйте!Решил поправтиковаться и написать программу которая должна создавать падающие снежинки на экране, вот код:

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
window.onload  = function()
{   
    var img;
    img  = document.createElement('img');
    img.src = '1.jpg';
    var arr  = new Array(); 
    function Place(arr)
    {
        for(var i = 0;i<15;i++)
        {   
            arr.push(img.cloneNode());
            arr[i].style.marginLeft = Math.random()*1000+'px';
            document.body.appendChild(arr[i]);
        }
    }
    Place(arr);
    function DownFall(mas)
    {
        for(var j = 0;j<mas.length;j++)
        {
            mas[j].style.marginTop += 20+'px';
            document.body.appendChild(mas[j]);
        }
    }
    setInterval(DownFall,200,arr);
}

Но вот в чем проблема, функция которая засунута в setInterval выполняеться только 1 раз, в чем причина не могу понять, возможно вы мне подскажите, заранее спасибо!

@GuardCat · 18.10.2011, 15:09

setInterval нормально выполняется, просто ваша функция не фунциклирует. Вставьте туда алерт что ли для проверки. Вы прибавляете значение к строке и сверху ещё намазываете "px" (строка 21 вашего листинга). Попробуйте вычленить числовую часть. Как-то так, например:

JavaScript
1
2
3
var element;
element=mas[j].style.marginTop
mas[j].style.marginTop=Number(element.substr(0,element.length-2))+20+'px';

@BART · 18.10.2011, 15:13 **[ТС]**

Спасибо, я понял в чем была ошибка!

@Vicont · 19.10.2011, 01:59

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

Спасибо, я понял в чем была ошибка!

И в чем же? Лично я Ваш код проверил и нашел в нем три ошибки.

Во-первых:
Opera, Netscape Navigator и FireFox требуют наличия параметра в методе cloneNode():
- cloneNode(булево значение)
булево значение - это значение управляет тем, включает ли копия узла все дочерние узлы исходного (true) или нет (false).

В нашем случае нужно вызывать метод с параметром false:
arr.push(img.cloneNode(false));

Во-вторых:
Как уже было сказано выше строка:

JavaScript
1
mas[j].style.marginTop += 20+'px';

Вызовет ошибку, т.к. будет содержать в себе строковое значение, к примеру, изначально свойство:

JavaScript
1
mas[j].style.marginTop

будет иметь значение пустой строки - '', а операция:

JavaScript
1
mas[j].style.marginTop += 20+'px';

эквивалентна пустая строка + 20 + 'px' и равна строке '20px', при втором присваивании будет уже ошибка:

JavaScript
1
mas[j].style.marginTop += 20+'px';

эквивалентна '20px' + 20 + 'px' и равна строке '20px20px'

Поэтому строку:

JavaScript
1
mas[j].style.marginTop += 20+'px';

заменяем на:

JavaScript
1
mas[j].style.marginTop = parseInt(mas[j].style.marginTop)+ 20+'px';

А в функцию Place добавляем строку:

JavaScript
1
arr[i].style.marginTop = 0;

Т.к. функция parseInt трансформирующая строку в число, при получении пустой строки вернет не 0 а значение NaN(Not a Number), что приведет к ошибке, а так изначально свойству marginTop мы присвоили значение 0 (в функции Place).

В-третьих:

Это строка будет работать во всех браузерах, кроме IE:

JavaScript
1
setInterval(DownFall,200,arr);

Т.к. третий параметр не поддерживается IE.

Заменяем строку:

JavaScript
1
setInterval(DownFall,200,arr);

На:

JavaScript
1
setInterval(function(){DownFall(arr);},200);

В-четвертых: Передавать arr функции Place - лишнее, функция Place и без этого имеет доступ к переменной arr.

И получаем кроссбраузерный и работающий код:

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
 window.onload  = function()
{   
   var img;
   img  = document.createElement('img');
   img.src = '1.jpg';
   var arr  = new Array(); 
 
   function Place()
   {
      for(var i=0; i<15; i++)
      {       
          arr.push(img.cloneNode(false));
          arr[i].style.marginLeft = Math.random()*1000+'px';
          arr[i].style.marginTop = 0;
          document.body.appendChild(arr[i]);
       }
   }
   Place();          
            
   function DownFall(mas)
   {
       for(var j=0; j<mas.length; j++)
       {
            mas[j].style.marginTop = parseInt(mas[j].style.marginTop)+ 20+'px';
            document.body.appendChild(mas[j]);
        }
                        
     }
     setInterval(function(){DownFall(arr);},200);
}

Новые блоги и статьи Все статьи Все блоги /
Загрузка PNG-файла с альфа-каналом с помощью библиотеки SDL3_image на Android 8Observer8 27.01.2026 Содержание блога SDL3_image - это библиотека для загрузки и работы с изображениями. Эта пошаговая инструкция покажет, как загрузить и вывести на экран смартфона картинку с альфа-каналом, то есть с. . .	влияние грибов на сукцессию anaschu 26.01.2026 Бифуркационные изменения массы гриба происходят тогда, когда мы уменьшаем массу компоста в 10 раз, а скорость прироста биомассы уменьшаем в три раза. Скорость прироста биомассы может уменьшаться за. . .	Воспроизведение звукового файла с помощью SDL3_mixer при касании экрана Android 8Observer8 26.01.2026 Содержание блога SDL3_mixer - это библиотека я для воспроизведения аудио. В отличие от инструкции по добавлению текста код по проигрыванию звука уже содержится в шаблоне примера. Нужно только. . .	Установка Android SDK, NDK, JDK, CMake и т.д. 8Observer8 25.01.2026 Содержание блога Перейдите по ссылке: https:/ / developer. android. com/ studio и в самом низу страницы кликните по архиву "commandlinetools-win-xxxxxx_latest. zip" Извлеките архив и вы увидите. . .
Вывод текста со шрифтом TTF на Android с помощью библиотеки SDL3_ttf 8Observer8 25.01.2026 Содержание блога Если у вас не установлены Android SDK, NDK, JDK, и т. д. то сделайте это по следующей инструкции: Установка Android SDK, NDK, JDK, CMake и т. д. Сборка примера Скачайте. . .	Использование SDL3-callbacks вместо функции main() на Android, Desktop и WebAssembly 8Observer8 24.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .	моя боль iceja 24.01.2026 Выложила интерполяцию кубическими сплайнами www. iceja. net REST сервисы временно не работают, только через Web. Написала за 56 рабочих часов этот сайт с нуля. При помощи perplexity. ai PRO , при. . .	Модель сукцессии микоризы anaschu 24.01.2026 Решили писать научную статью с неким РОманом

@BART 7 / 2 / 0 Регистрация: 17.11.2009 Сообщений: 116
	18.10.2011, 15:13 [ТС]
	Спасибо, я понял в чем была ошибка! 0