Неправильное использование setTimeout?

Question

Я написал сценарий, который запускает 2 URL-адреса, основанных на некоторой логике случайных чисел, и я пытаюсь установить задержку, прежде чем один из них будет запущен (полсекунды), но я не думаю, что он работает правильно. Правильно ли я делаю это? Код ниже:

var clicks = "http://www.urlone.com"; 
var impressions = "http://www.urltwo.com"; 
var randomNumber = (Math.random()*100); 

function callOut() { 
for (var i = 0; i < lengthVal; i++){ 
    if (randomNumber < 75) { 
     var randomCounter = (Math.random()*100); 
      if (randomCounter < 50) { 
       setTimeout("image1.src = clicks;",500); 

      } 
      else if (randomCounter > 50) { 
       setTimeout("image1.src = impressions;",500);      
      } 
    } 
    } 
} 

Answer 1

Обязательно описал использование setTimeout со строкой. Вот как я это сделал. Это лучший способ сделать это?

var clicks = "http://www.urlone.com"; 
var impressions = "http://www.urltwo.com"; 
var conversions = "http://www.urlthree"; 
var lengthVal = (Math.random() * 20 + 20); 
var image1 = new Image(); 
var image2 = new Image(); 
var globalCounter = -1; 



function callOut() { 
    var ord = (Math.random() * 9999999999999) + ""; 

    var randomNumber = (Math.random() * 100); // Random value for each call 
    if (randomNumber < 75) { 
     var randomCounter = Math.random() * 100; 
     alert(randomCounter); 
     if (randomCounter < 50) { 
      image1.src = clicks + ord + "?"; 
     } 
     if (randomCounter > 50) { 
      image2.src = impressions + ord + "?"; 
     } 
    } 
    if (globalCounter++ < lengthVal) { 
     setTimeout(callOut, 1000); // Call itself after another second 
    } 
} 

Answer 2

setTimeout первый параметр должен быть функцией. Не строка кода.

код в альтернативном синтаксисе - это строка кода, которую вы хотите выполнить после задержки миллисекунд. (Используя этот синтаксис не рекомендуется по тем же причинам, как с помощью Eval())

MDN

setTimeout(function(){...}, 500); 

Answer 3

Взято отсюда: http://www.codescream.com/?p=18 прочитать это должно помочь :)

Если вы хотите сделать задержку с SetTimeout вы должны сделать именно это:

setTimeout(function() { 
     doThings() 
}, 1000); 

и никогда этого:

setTimeout("doThings()", 1000); 

Answer 4

setTimeout("image1.src = clicks;",500); 

Для этого image1 должны быть объявлены в глобальном контексте, например:

var image1 = document.getElementById('image1'); 

Но лучше использовать функцию здесь.

function setImageSrcClicks(){ 
    document.getElementById('image1').src = 'http://clicks_url'; 
} 
setTimeout(setImageSrcClicks,500);