JQuery div background image cycle

Question

Любая идея, почему этот код поврежден или не работает? Это не мой код, но он кажется правильным, конечно, я мог бы пропустить очевидное. Мне просто нужно фоновое изображение div для циклического перехода к новому в массиве/a каждые 5 секунд.

var imageIndex = 0; 
var imagesArray = new Array(); 

//Set Images 
imagesArray[0] = "images/self-whitewater.png"; 
imagesArray[1] = "images/fishing.png"; 
imagesArray[2] = "images/solo-kayaking.png"; 

function changeBackground(){ 
$("main-blocki").css("background","url('"+ imagesArray[imageIndex] +"')");       
    imageIndex++; 
    if (imageIndex > imageArray.length) 
     imageIndex = 0; 
} 

$(document).ready(function() { 
    setInterval("changeBackground()",5000); 
}); 

Answer 1

Ваш проблема заключается в вашем заявлении if, как указано @thatidiotguy.

Но вы также можете сделать это в oneliner без инструкции if.

var imageIndex = 0; 
var imagesArray = [ 
    "images/self-whitewater.png", 
    "images/fishing.png", 
    "images/solo-kayaking.png" 
]; 

function changeBackground(){ 
    var index = imageIndex++ % imagesArray.length; 
    $("main-blocki").css("background","url('"+ imagesArray[index] +"')");  
} 

$(document).ready(function() { 
    setInterval(changeBackground, 5000); 
}); 

Отметьте imageIndex++ % imagesArray.length. Это увеличивает глобальный imageIndex, удостоверяясь, что значение не больше imagesArray.length.

Answer 2

У вас есть синтаксические ошибки здесь:

if (imageIndex > imageArray.length) 
     imageIndex = 0; 

Там нет переменной называется imageArray

Вы должны использовать веб-отладчик, чтобы показать вам, когда у вас есть синтаксические ошибки. Одна синтаксическая ошибка и Javascript имеют изящество, чтобы убить каждый другой скрипт, который вы используете или пытаетесь запустить.

Answer 3

if (imageIndex > imageArray.length) 

выше линии не так: вы должны хотеть, чтобы проверить imagesArray.length вместо

... и вы должны проверить с> = оператора также, поскольку индексы 0 на основе