设为首页收藏本站
切换到窄版

鱼C论坛

 找回密码
 立即注册
鱼C论坛»论坛 › 萌新训练营 › 萌新交流区 › js定时器实现轮播图时，第一次点击切换按钮和列表无效
返回列表 发新帖
查看: 1621|回复: 1

[已解决]js定时器实现轮播图时，第一次点击切换按钮和列表无效

[复制链接]
ZhihanXing
发表于 2023-6-25 19:49:33 | 显示全部楼层 |阅读模式

马上注册，结交更多好友，享用更多功能^_^

您需要 登录 才可以下载或查看，没有账号？立即注册

x
下面代码是我写的一个轮播图的效果。当页面加载后，页面的图片会每隔一秒切换成另外的图片，点击prev按钮会切换到前一张图片，点击next按钮会切换到后一张图片，点击列表中的小圆点会切换到对应的图片。但是当页面加载完毕后，立刻点击prev按钮和列表没有切换到相应的图片，第二次点击才会生效。而页面加载完成后，第一次点击next按钮，会切换到往后第二张图片而不是往后第一张图片，请问这是为什么？
  1. <!DOCTYPE html>
  2. <html lang="en">

  4. <head>
  5.     <meta charset="UTF-8" />
  6.     <meta http-equiv="X-UA-Compatible" content="IE=edge" />
  7.     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
  8.     <title>轮播图点击切换</title>
  9.     <style>
  10.         * {
  11.             box-sizing: border-box;
  12.         }
  13.         
  14.         .slider {
  15.             width: 560px;
  16.             height: 400px;
  17.             margin: 100px auto;
  18.             overflow: hidden;
  19.         }
  20.         
  21.         .slider-wrapper {
  22.             width: 100%;
  23.             height: 320px;
  24.             transition: all .3s;
  25.         }
  26.         
  27.         .slider-wrapper img {
  28.             width: 100%;
  29.             height: 100%;
  30.             display: block;
  31.         }
  32.         
  33.         .slider-footer {
  34.             height: 80px;
  35.             background-color: rgb(100, 67, 68);
  36.             padding: 12px 12px 0 12px;
  37.             position: relative;
  38.         }
  39.         
  40.         .slider-footer .toggle {
  41.             position: absolute;
  42.             right: 0;
  43.             top: 12px;
  44.             /* display: flex; */
  45.         }
  46.         
  47.         .slider-footer .toggle button {
  48.             margin-right: 12px;
  49.             width: 28px;
  50.             height: 28px;
  51.             /* appearance: none; */
  52.             border: none;
  53.             background: rgba(255, 255, 255, 0.1);
  54.             color: #fff;
  55.             border-radius: 4px;
  56.             cursor: pointer;
  57.         }
  58.         
  59.         .slider-footer .toggle button:hover {
  60.             background: rgba(255, 255, 255, 0.2);
  61.         }
  62.         
  63.         .slider-footer p {
  64.             margin: 0;
  65.             color: #fff;
  66.             font-size: 18px;
  67.             margin-bottom: 10px;
  68.         }
  69.         
  70.         .slider-indicator {
  71.             margin: 0;
  72.             padding: 0;
  73.             list-style: none;
  74.             display: flex;
  75.             align-items: center;
  76.         }
  77.         
  78.         .slider-indicator li {
  79.             width: 8px;
  80.             height: 8px;
  81.             margin: 4px;
  82.             border-radius: 50%;
  83.             background: #fff;
  84.             opacity: 0.4;
  85.             cursor: pointer;
  86.         }
  87.         
  88.         .slider-indicator li.active {
  89.             width: 12px;
  90.             height: 12px;
  91.             opacity: 1;
  92.         }
  93.     </style>
  94. </head>

  96. <body>
  97.     <div class="slider">
  98.         <div class="slider-wrapper">
  99.             <img src="images/01.jpg" alt="" />
  100.         </div>
  101.         <div class="slider-footer">
  102.             <p>aaaaaa</p>
  103.             <ul class="slider-indicator">
  104.                 <li class="active"></li>
  105.                 <li></li>
  106.                 <li></li>
  107.                 <li></li>
  108.                 <li></li>
  109.                 <li></li>
  110.                 <li></li>
  111.                 <li></li>
  112.             </ul>
  113.             <div class="toggle">
  114.                 <button class="prev">&lt;</button>
  115.                 <button class="next">&gt;</button>
  116.             </div>
  117.         </div>
  118.     </div>
  119.     <script>
  120.         //首先创建数组对象，存入轮播图的信息
  121.         const data = [{
  122.             url: "images/01.jpg",
  123.             title: "aaaaaa"
  124.         }, {
  125.             url: "images/02.jpg",
  126.             title: "bbbbbb"
  127.         }, {
  128.             url: "images/03.jpg",
  129.             title: "cccccc"
  130.         }, {
  131.             url: "images/04.jpg",
  132.             title: "dddddd"
  133.         }, {
  134.             url: "images/05.jpg",
  135.             title: "eeeeee"
  136.         }, {
  137.             url: "images/06.jpg",
  138.             title: "ffffff"
  139.         }, {
  140.             url: "images/07.jpg",
  141.             title: "gggggg"
  142.         }, {
  143.             url: "images/08.jpg",
  144.             title: "hhhhhh"
  145.         }, ];
  146.         //获取需要的对象
  147.         const img = document.querySelector('.slider-wrapper img');
  148.         const title = document.querySelector('.slider-footer p');
  149.         const lis = document.querySelectorAll('.slider-indicator li');
  150.         const prev_button = document.querySelector('.toggle .prev ');
  151.         const next_button = document.querySelector('.toggle .next ');
  152.         let i = 0;
  153.         //这是更改轮播图内容的函数
  154.         function change() {
  155.             if (i < data.length) {
  156.                 //更改图片和文字
  157.                 img.src = data[i].url;
  158.                 title.innerHTML = data[i].title;
  159.                 //更改li的样式
  160.                 for (let j = 0; j < lis.length; j++) {
  161.                     lis[j].classList.remove('active');
  162.                 }
  163.                 lis[i].classList.add('active');
  164.                 i++;
  165.             } else {
  166.                 i = 0;
  167.             }
  168.         }
  169.         //这是两个按钮切换轮播图的函数
  170.         function prev() {
  171.             i -= 1;
  172.             //保持变量i大于等于0
  173.             if (i < 0) {
  174.                 i = data.length - 1;
  175.             }
  176.             // console.log("当前i的值是：" + i);
  177.             // console.log("prev函数执行了！");
  178.             //更改图片和文字
  179.             img.src = data[i].url;
  180.             title.innerHTML = data[i].title;
  181.             //更改li的样式
  182.             for (let j = 0; j < lis.length; j++) {
  183.                 lis[j].classList.remove('active');
  184.             }
  185.             lis[i].classList.add('active');
  186.         }

  188.         function next() {
  189.             i += 1;
  190.             //保持变量i大于等于0
  191.             if (i > data.length - 1) {
  192.                 i = 0;
  193.             }
  194.             // console.log("当前i的值是：" + i);
  195.             // console.log("next函数执行了！");
  196.             //更改图片和文字
  197.             img.src = data[i].url;
  198.             title.innerHTML = data[i].title;
  199.             //更改li的样式
  200.             for (let j = 0; j < lis.length; j++) {
  201.                 lis[j].classList.remove('active');
  202.             }
  203.             lis[i].classList.add('active');
  204.         }

  206.         //这是点击li切换到对应轮播图的函数
  207.         function lis_change() {
  208.             for (let j = 0; j < lis.length; j++) {
  209.                 lis[j].addEventListener("click", function() {
  210.                     img.src = data[j].url;
  211.                     title.innerHTML = data[j].title;
  212.                     for (let k = 0; k < lis.length; k++) {
  213.                         lis[k].classList.remove('active');
  214.                     }
  215.                     lis[j].classList.add('active');
  216.                     i = j;
  217.                 });
  218.             }
  219.         }

  221.         //开启定时器
  222.         let timer = setInterval(change, 1000);
  223.         //关闭定时器的函数
  224.         function close_timer() {
  225.             clearInterval(timer);
  226.         }
  227.         //再次开启定时器的函数
  228.         function open_timer() {
  229.             timer = setInterval(change, 1000);
  230.         }
  231.         //这是两个按钮点击事件
  232.         prev_button.addEventListener("click", prev);
  233.         next_button.addEventListener("click", next);
  234.         //当鼠标指针悬浮在按钮之上，关闭定时器
  235.         prev_button.addEventListener("mouseover", close_timer);
  236.         next_button.addEventListener("mouseover", close_timer);
  237.         //鼠标移出按钮，重新开启定时器
  238.         next_button.addEventListener("mouseout", open_timer);
  239.         prev_button.addEventListener("mouseout", open_timer);
  240.         //鼠标悬浮在li上，关闭定时器
  241.         for (let j = 0; j < lis.length; j++) {
  242.             lis[j].addEventListener("mouseover", close_timer);
  243.         }
  244.         //鼠标移出li，重新开启定时器
  245.         for (let j = 0; j < lis.length; j++) {
  246.             lis[j].addEventListener("mouseout", open_timer);
  247.         }
  248.         //鼠标点击lis事件，跳转到对应轮播图
  249.         for (let j = 0; j < lis.length; j++) {
  250.             lis[j].addEventListener("click", lis_change);
  251.         }
  252.     </script>
  253. </body>

  255. </html>
复制代码
最佳答案
月排行榜 / 总排行榜
isdkz
2023-6-25 19:52:24
这个问题的根本原因在于你的定时器和按钮点击事件的冲突。定时器每隔一秒就会自动调用 change 函数，使得 i 加1；而在你点击按钮或列表项时，你也会改变 i 的值。这两个行为可能会在极短的时间内发生，从而产生你所描述的问题。

解决方案是：在点击按钮或列表项时，首先关闭定时器，然后再修改 i 的值，最后重新开启定时器。另外，你在给每个列表项绑定点击事件时出现了问题，你应该直接调用 lis_change 函数，而不是在事件处理器中调用。以下是修改后的部分代码：

  1. //这是两个按钮点击事件
  2. prev_button.addEventListener("click", function() {
  3.     close_timer();
  4.     prev();
  5.     open_timer();
  6. });

  8. next_button.addEventListener("click", function() {
  9.     close_timer();
  10.     next();
  11.     open_timer();
  12. });

  14. //鼠标点击lis事件，跳转到对应轮播图
  15. for (let j = 0; j < lis.length; j++) {
  16.     lis[j].addEventListener("click", function() {
  17.         close_timer();
  18.         i = j;
  19.         img.src = data[i].url;
  20.         title.innerHTML = data[i].title;
  21.         for (let k = 0; k < lis.length; k++) {
  22.             lis[k].classList.remove('active');
  23.         }
  24.         lis[j].classList.add('active');
  25.         open_timer();
  26.     });
  27. }
复制代码


另外，我发现你的 change 函数在 i 等于 data.length 时没有正确地回到0，应该调整如下：

  1. function change() {
  2.     //更改图片和文字
  3.     img.src = data[i].url;
  4.     title.innerHTML = data[i].title;
  5.     //更改li的样式
  6.     for (let j = 0; j < lis.length; j++) {
  7.         lis[j].classList.remove('active');
  8.     }
  9.     lis[i].classList.add('active');
  10.     i++;
  11.     if (i >= data.length) {
  12.         i = 0;
  13.     }
  14. }
复制代码


最后，注意你在 prev 和 next 函数中直接修改了 i 的值，然后才更改了轮播图，这意味着你实际上是跳过了一张图片。你应该先更改当前的图片，然后再让 i 增加或减少。在你的 prev 和 next 函数中，去掉 i -= 1; 和 i += 1; 这两行代码，然后把它们移到函数的末尾，应该就能解决你的问题了。
小甲鱼最新课程 -> https://ilovefishc.com
回复

使用道具 举报

isdkz
发表于 2023-6-25 19:52:24 | 显示全部楼层    本楼为最佳答案   
这个问题的根本原因在于你的定时器和按钮点击事件的冲突。定时器每隔一秒就会自动调用 change 函数，使得 i 加1；而在你点击按钮或列表项时，你也会改变 i 的值。这两个行为可能会在极短的时间内发生，从而产生你所描述的问题。

解决方案是：在点击按钮或列表项时，首先关闭定时器，然后再修改 i 的值，最后重新开启定时器。另外，你在给每个列表项绑定点击事件时出现了问题，你应该直接调用 lis_change 函数，而不是在事件处理器中调用。以下是修改后的部分代码：

  1. //这是两个按钮点击事件
  2. prev_button.addEventListener("click", function() {
  3.     close_timer();
  4.     prev();
  5.     open_timer();
  6. });

  8. next_button.addEventListener("click", function() {
  9.     close_timer();
  10.     next();
  11.     open_timer();
  12. });

  14. //鼠标点击lis事件，跳转到对应轮播图
  15. for (let j = 0; j < lis.length; j++) {
  16.     lis[j].addEventListener("click", function() {
  17.         close_timer();
  18.         i = j;
  19.         img.src = data[i].url;
  20.         title.innerHTML = data[i].title;
  21.         for (let k = 0; k < lis.length; k++) {
  22.             lis[k].classList.remove('active');
  23.         }
  24.         lis[j].classList.add('active');
  25.         open_timer();
  26.     });
  27. }
复制代码


另外，我发现你的 change 函数在 i 等于 data.length 时没有正确地回到0，应该调整如下：

  1. function change() {
  2.     //更改图片和文字
  3.     img.src = data[i].url;
  4.     title.innerHTML = data[i].title;
  5.     //更改li的样式
  6.     for (let j = 0; j < lis.length; j++) {
  7.         lis[j].classList.remove('active');
  8.     }
  9.     lis[i].classList.add('active');
  10.     i++;
  11.     if (i >= data.length) {
  12.         i = 0;
  13.     }
  14. }
复制代码


最后，注意你在 prev 和 next 函数中直接修改了 i 的值，然后才更改了轮播图，这意味着你实际上是跳过了一张图片。你应该先更改当前的图片，然后再让 i 增加或减少。在你的 prev 和 next 函数中，去掉 i -= 1; 和 i += 1; 这两行代码，然后把它们移到函数的末尾，应该就能解决你的问题了。
小甲鱼最新课程 -> https://ilovefishc.com
回复 支持 反对

使用道具 举报

返回列表 发新帖
高级模式
B Color Image Link Quote Code Smilies
您需要登录后才可以回帖 登录 | 立即注册

本版积分规则

小黑屋|手机版|Archiver|鱼C工作室 ( 粤ICP备18085999号-1 | 粤公网安备 44051102000585号)

GMT+8, 2025-6-9 18:15

Powered by Discuz! X3.4

© 2001-2023 Discuz! Team.

快速回复 返回顶部 返回列表