Question

我试图将函数带到循环之外，然后从内部调用它，但是我不确定如何做到这一点。

const links = document.querySelectorAll( 'a' );

for ( let i = 0; i < links.length; i++ ) {

    links[i].addEventListener( 'click', ( event ) => {
        const targetID = '#' === event.currentTarget.getAttribute( 'href' ) ? 'start' : event.currentTarget.getAttribute( 'href' );
        ...rest of the function...
    } );
}

这是我到目前为止尝试过的：

const links = document.querySelectorAll( 'a' );

function smoothScrolling( event ) {
    const targetID = '#' === event.currentTarget.getAttribute( 'href' ) ? 'start' : event.currentTarget.getAttribute( 'href' );
    ...rest of the function...
}

for ( let i = 0; i < links.length; i++ ) {

    links[i].addEventListener( 'click', smoothScrolling( event ) );
}

我不确定为什么，但是出现以下错误：Uncaught TypeError: Cannot read property 'currentTarget' of undefined。

Answer 1

您几乎了解了...问题是您正在调用函数并传递结果。相反，您只想传递函数本身，就像它是一个对象一样。试试这个：

const links = document.querySelectorAll( 'a' );

function smoothScrolling( event )
{
     const targetID = '#' === event.currentTarget.getAttribute( 'href' ) ? 'start' : 
     event.currentTarget.getAttribute( 'href' );
     ...rest of the function...
}

for ( let i = 0; i < links.length; i++ )
{
    links[i].addEventListener( 'click', smoothScrolling );
}

通过指定不带任何参数的函数，它将被传递而不是被调用。完成此操作的方法是调用smoothScrolling，然后使用其结果，这不是您想要的。

将函数移出循环

1 个答案: