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

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

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

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

Javascript

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

 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);
}

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