为什么clearInterval()只工作一次?

时间:2019-08-13 10:30:04

标签: javascript jquery slider

基本思想是确定页面上是否有视频,等到页面结束,然后以30秒的间隔滑动不包含视频的页面。它清除间隔并等待直到仅完成一次。第二次,它就像其他页面一样不断滑动。

function getPage(i) {
  return p[i];
}

var pages = '@ViewBag.pages';
var p = JSON.parse(pages.replace(/(&quot\;)/g, "\""));
var i = 0;
var mi = p.length - 1;
var interval;
var frame = document.getElementById("slider");
var frameDoc = frame.contentDocument || frame.contentWindow.document;
frame.onload = function() {
  if ($(document).find("iframe").contents().find("video").length == 1) {
    clearInterval(interval);
    console.log("interval was cleared successfully");
    console.log("the video is now playing!");
    var video = document.getElementById('slider').contentWindow.document.getElementById('video');
    video.onended = function() {
      console.log("video is finished!");
      i++;
      if (i > mi) {
        i = 0
      }
      frame.src = getPage(i);
    }
  } else {
    console.log("video is not found");
    interval = setInterval(function() {
      i++;
      if (i > mi) {
        i = 0
      }
      $.ajax({
        async: false,
        url: getPage(i),
        type: 'GET',
        success: function() {
          frame.src = getPage(i);
        }
      });

    }, 10000);
  }
}

1 个答案:

答案 0 :(得分:1)

这里的问题是,当页面没有视频时,clearInterval不会被调用。

让我们以以下情形为例,您有2页不带视频,而1页带视频按此顺序

  1. 加载第一页,启动setInterval并将引用存储在变量interval中。
  2. 在加载10000ms下一页后,由于该页面没有视频,因此不会清除先前的setInterval,并且interval被更新为新的setInterval。现在您有两个setIntervals正在运行,假设interval1interval2
  3. 10000ms之后,这次将执行2个setIntervals,interval1将页面移至第三页,interval2将页面移至第一页。
  4. 您将继续创建新的setIntervals,直到您转到带有视频的页面,并且当您到达带有视频的页面时,您仅清除了一个间隔,即最后一个间隔。仍然保留所有其他间隔,这些间隔会不断更新您的页面。

解决方案:

  1. 在iframe.onload中无条件运行clearInterval
var interval;
frame.onload = function() {
  interval && clearInterval(interval);
  if ($(document).find("iframe").contents().find("video").length == 1) {
  } else {
    interval = setInterval(function() {}, 10000);
  }
}
  1. 使用setTimeout,因为您只希望执行一次。
frame.onload = function() {
  if ($(document).find("iframe").contents().find("video").length == 1) {
  } else {
    setTimeout(function() {}, 10000);
  }
}
  

PS:我更喜欢第二个