1

Ошибка только в фаерфоксе.

ReferenceError: vislideshow is not defined.

Сам код выглядит так:

vislideshow();

function vislideshow(){
    $('.slide5').delay( 5000 ).fadeOut( 5000 );
    $('.slide4').delay( 10000 ).fadeOut( 5000 );
    $('.slide3').delay( 15000 ).fadeOut( 5000 );
    $('.slide2').delay( 20000 ).fadeOut( 5000 );
    vislidhide();
}
function vislidhide(){
    $('.slide2').delay( 5000 ).fadeIn( 5000 );
    $('.slide3').delay( 15000 ).fadeIn( 5000 );
    $('.slide4').delay( 20000 ).fadeIn( 5000 );
    $('.slide5').delay( 25000 ).fadeIn( 5000 );
    setTimeout(vislideshow(), 10000);
}

Как исправить ошибку?

4
  • Поместить вызов функции после ее определения не пробовали? 9 окт 2015 в 12:09
  • ФФ очень придирчив к порядку определений и вызовов функции. Вызывайте после определения
    – Moonvvell
    9 окт 2015 в 12:10
  • @Виталий, вот такой код отлично работает в FF: jsfiddle.net/m0bdttkc Похоже проблема где-то еще 9 окт 2015 в 12:43
  • Если вам дан исчерпывающий ответ, отметьте его как верный (галка напротив выбранного ответа). 9 окт 2015 в 17:04

3 ответа 3

1

Либо перенести, либо вынести за $(document).ready сами функции

$(document).ready(function(){
    vislideshow();
});


function vislideshow(){
    $('.slide5').delay( 5000 ).fadeOut( 5000 );
    $('.slide4').delay( 10000 ).fadeOut( 5000 );
    $('.slide3').delay( 15000 ).fadeOut( 5000 );
    $('.slide2').delay( 20000 ).fadeOut( 5000 );
    vislidhide();
}
function vislidhide(){
    $('.slide2').delay( 5000 ).fadeIn( 5000 );
    $('.slide3').delay( 15000 ).fadeIn( 5000 );
    $('.slide4').delay( 20000 ).fadeIn( 5000 );
    $('.slide5').delay( 25000 ).fadeIn( 5000 );
    setTimeout(vislideshow(), 10000);
}
1

Объявление функции перенесите выше вызова, а то вы пытаетесь вызвать функцию, которую еще не объявили.

function vislideshow(){
    $('.slide5').delay( 5000 ).fadeOut( 5000 );
    $('.slide4').delay( 10000 ).fadeOut( 5000 );
    $('.slide3').delay( 15000 ).fadeOut( 5000 );
    $('.slide2').delay( 20000 ).fadeOut( 5000 );
    vislidhide();
}
function vislidhide(){
    $('.slide2').delay( 5000 ).fadeIn( 5000 );
    $('.slide3').delay( 15000 ).fadeIn( 5000 );
    $('.slide4').delay( 20000 ).fadeIn( 5000 );
    $('.slide5').delay( 25000 ).fadeIn( 5000 );
    setTimeout(vislideshow(), 10000);
}
vislideshow();

И всегда лучше выполнять JS после подгрузке всей страницы и ресурсов

$(function(){
    vislideshow();
});

Важно еще понимать принцип компиляции JS. JIT-компиляции

4
  • 1
    Объявления функций не надо выносить наверх - это сделает парсер. Проблема точно не в этом. 9 окт 2015 в 12:33
  • JSFiddle с примером "поднятия" определения функций, который работает в FF jsfiddle.net/m0bdttkc 9 окт 2015 в 12:44
  • @DmitriySimushev, да, действительно…хм
    – korytoff
    9 окт 2015 в 12:49
  • @korytoff, похоже есть еще какая-то проблема, о которой автор умолчал 9 окт 2015 в 12:50
0
setTimeout(vislideshow(), 10000);

Скобки убери:

setTimeout(vislideshow, 10000);

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.