javascript - 为什么clearInterval()只起作用一次？

Question

基本思路是判断页面是否有视频，等到结束后继续滑动不包含视频的页面，间隔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);
  }
}

Answer 1

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

让我们考虑以下场景，您有 2 个没有视频的页面和 1 个有视频的页面(按此顺序)

1. 加载第一个页面，启动 setInterval 并将引用存储在变量 interval 中。
2. 加载下一个页面 10000 毫秒后，由于页面没有视频，之前的 setInterval 不会被清除，并且 interval 会更新为新的 setInterval。现在您有两个 setIntervals 正在运行，比如说 interval1 和 interval2
3. 10000ms 后，这次执行了 2 个 setIntervals，interval1 将页面移动到第 3 页，interval2 将页面移动到第 1 页。
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);
  }
}

使用 setTimeout 因为您希望它只执行一次。

frame.onload = function() {
  if ($(document).find("iframe").contents().find("video").length == 1) {
  } else {
    setTimeout(function() {}, 10000);
  }
}

PS: I prefer the second one