Не работает функция автоматической смены изображения?

Question

[evgemiil]

https://jsfiddle.net/evgeniel/d6uc2c1g/2/
Кнопки работают, но функция которая автоматически сменяет изображения не работает, подскажите в чем ошибка?

P.s хотел бы услышать критику и советы)

Comments

[Kovalsky]

Критика:

1. Вы не смотрите в консоль,
2. Вы вешаете обработчики на элементы через HTML, это очень-очень-очень проблемный путь,

Советы:

1. Смотреть в консоль,
2. Вешать обработчики через JS,

[evgemiil]

Никита Полевой: спасибо за ответы, но почему setTimeout выполняется только один раз, меняет одно изображение?

[Kovalsky]

evgemiil: не знаю, у меня например ни разу не выполняется) так происходит потому что вы используете не чистый документа, а jsfiddle: в поле JAVASCRIPT нажмите на шестеренку справа сверху, увидите опцию Load Type. Она выставлена в onLoad, это значит что все скрипты будут выполнены по событию onLoad. Ну а у вас в коде еще один байндинг на onLoad, который естественно не выполняется.

[evgemiil]

Никита Полевой: не работает даже с браузера, он один раз меняет автоматически далее нет

[Kovalsky]

evgemiil: setTimeout - Sets a timer which executes a function or specified piece of code once after the timer expires. (Вызов функции или выполнение фрагмента кода после указанной задержки.)

Answer 1

[Kirill Sharonov]

Чтоб изображение менялось каждые 5 секунд, надо использовать setInterval, а не setTimeout

Answer 2

[Dark Hole]

Битые изображения, вот и все

Comments

[evgemiil]

Да я знаю, сейчас нет возможности выложить,спасибо за ответы, но почему setTimeout выполняется только один раз, меняет одно изображение?

[Dark Hole]

evgemiil: так и должно быть