0

フェードイン/アウト効果で背景画像を変更するこの小さなスクリプトがあります。

/*controla la velocidad de la animación*/
    var slideshowSpeed = 1000;
    var transitionSpeed = 2000;
    var timeoutSpeed = 500;
    
    /*No tocar*/
    var interval;
    var activeContainer = 1;    
    var currentImg = 0;
    var animating = false;
    var navigate = function(direction) {
        // Si ya estamos navegando, entonces no hacemos nada!
        if(animating) {
            return;
        }
        currentImg++;
        if(currentImg == photos.length + 1) {
            currentImg = 1;
        }
        // Tenemos dos, uno en el que tenemos la imagen que se ve y otro d?nde tenemos la imagen siguiente
        var currentContainer = activeContainer;
        // Esto puedo optimizarlo con la funci?n modulo, y cambiar 1 y 2 por 0 y 1-> active = mod2(active + 1)
        if(activeContainer == 1) {
            activeContainer = 2;
        } else {
            activeContainer = 1;
        }
        // hay que decrementar el ?ndice porque empieza por cero
        cargarImagen(photos[currentImg - 1], currentContainer, activeContainer);
    };
    var currentZindex = -1;
    var cargarImagen = function(photoObject, currentContainer, activeContainer) {
        animating = true;
        // Nos aseguramos que el nuevo contenedor está siempre dentro del cajon
        currentZindex--;
        /*if(currentZindex < 0) currentZindex=1;*/
        // Actualizar la imagen
        $("#headerimg" + activeContainer).css({
            "background-image" : "url(" + photoObject + ")",
            "display" : "block",
            "z-index" : currentZindex
        });
        // FadeOut antigua
        // Cuando la transición se ha completado, mostramos el header 
        $("#headerimg" + currentContainer).fadeOut(transitionSpeed,function() {
            setTimeout(function() {
                
                animating = false;
            }, timeoutSpeed);
        });
    };
    
function iniciarPase(){
    var animating = false;
    //ver la siguente
     navigate("next");
    //iniciar temporizador que mostrará las siguientes
    interval = setInterval(function() {
        navigate("next");
    }, slideshowSpeed);
}
function pararPase(){
    var animating = true;
    console.log('paramos la animación');
    interval = clearInterval(interval);
}
/*Gestion de pase de imágenes de fondo*/
$(document).ready(function() {
    iniciarPase();
});

しかし、式pararPase()を含む関数はclearInterval、その上にあっても機能しないようです<body onload="pararPase()">

私が見逃しているものはありますか？

4

2 に答える 2

2

基本ロジックは正常に機能しています。ここにlive test caseがあります。

たとえば、ボタンがまだ存在しないときにpararPase外部のクリック ハンドラーとしてバインドしようとするなど、誤ってバインドする可能性が最も高いです。この点を証明するためにテスト ケースを更新しました。document.ready()

もう 1 つのオプションは、コード内の他のエラーです。Chrome JavaScript コンソールをチェックして、これに該当するかどうかを確認してください。

コメントで他の人が述べたように、clearInterval の戻り値を変数に代入することは無意味ですが、害はありません: 変数の値は「未定義」になります。

編集:iniciarPase複数回呼び出される可能性があります。これにより、最後のタイマーのみがクリアされる複数のタイマーが発生します。したがって、安全を期すために、これを関数に追加してください：（これは実際にダイオードが彼の答えで言おうとしたことです）

function iniciarPase(){
    var animating = false;
    //ver la siguente
    navigate("next");
    //iniciar temporizador que mostrará las siguientes
    if (interval)
        clearInterval(interval);
    interval = setInterval(function() {
        navigate("next");
    }, slideshowSpeed);
}
于 2011-12-19T11:31:39.483 に答える
0 
clearInterval(interval);
interval = setInterval(function() {
   navigate("next");
}, slideshowSpeed);
于 2011-12-19T11:43:30.117 に答える