使用 setInterval 移动轮播

发布于 2024-11-30 18:59:06 字数 350 浏览 2 评论 0原文

我试图使用以下代码每秒移动轮播元素：

function moveCarousel(){
        var x = $('.carousel_title.active');
        var next = x.next();
        x.removeClass('active');
        next.addClass('active');

    }

    setInterval(moveCarousel(),1000);

但有两件事似乎出了问题：

第一个循环立即发生
没有进一步的循环发生

我哪里出错了？

原文

I am trying to use the following code to move carousel elements through every second:

function moveCarousel(){
        var x = $('.carousel_title.active');
        var next = x.next();
        x.removeClass('active');
        next.addClass('active');

    }

    setInterval(moveCarousel(),1000);

but two things seem to go wrong:

The first cycle happens instantly
No further cycles occur

Where have I gone wrong?

分享到QQ

分享到微博

如果你对这篇内容有疑问，欢迎到本站社区发帖提问参与讨论，获取更多帮助，或者扫码二维码加入 Web 技术交流群。

发布评论

需要登录才能够评论，你可以免费注册一个本站的账号。

余厌 2024-12-07 18:59:06

你应该删除最后一行的大括号

function moveCarousel()
{
        var x = $('.carousel_title.active');
        var next = x.next();
        x.removeClass('active');
        next.addClass('active');

    }

    setInterval(moveCarousel,1000);

在这种情况下，你将一个函数（moveCarousel）传递给另一个函数（setInterval），因此该函数不会被执行（这就是大括号的用途）但要像对象一样传递。

您的原始代码将 undefined （因为 moveCarousel 不返回任何内容）传递给 setInterval 函数 - 并且 setInterval 将一个函数作为它的第一个参数 - 而不是 undefined

您也可以这样做：

setInterval(function () { moveCarousel(); }, 1000);

构造一个匿名函数来调用 moveCarousel。

you should remove the braces on your last line

function moveCarousel()
{
        var x = $('.carousel_title.active');
        var next = x.next();
        x.removeClass('active');
        next.addClass('active');

    }

    setInterval(moveCarousel,1000);

In this case you are passing a function (moveCarousel) to a nother function (setInterval) and thus the function is not to be executed (that's what the braces are for) but to be passed like an object.

Your original code was passing undefined (because moveCarousel does not return anything) to the setInterval function - and setInterval takes a function as it's first parameter - not undefined

You could also do this:

setInterval(function () { moveCarousel(); }, 1000);

where you construct an anonymous function to call moveCarousel.

回复收藏 0 原文

谈情不如逗狗 2024-12-07 18:59:06

您正在调用应该传递函数本身的函数：

function moveCarousel(){
    var x = $('.carousel_title.active');
    var next = x.next();
    x.removeClass('active');
    next.addClass('active');

}

setInterval(moveCarousel, 1000); // no () here

当前，您立即调用该函数，它的计算结果为：（

setInterval(undefined, 1000);

该函数不返回任何内容，因此它返回未定义），这不是您想要的。

You're calling the function where you should pass the function itself:

function moveCarousel(){
    var x = $('.carousel_title.active');
    var next = x.next();
    x.removeClass('active');
    next.addClass('active');

}

setInterval(moveCarousel, 1000); // no () here

Currently, you call the function immediately and it evaluates to: