以下代码成功执行:

var $ = document.querySelector.bind(document);

function windowScroll() {
    var elems = ['section-1', 'section-2', 'section-3', 'section-4'];
    elems.forEach(function(elem) {
        if(window.pageYOffset >= $(`#${elem}`).offsetTop - $('.banner-header').offsetHeight) {
            var li = document.querySelectorAll('.banner-header > nav > a');
            for(var i = 0; i < li.length; i++) {
                li[i].classList.remove('active-nav-item');
            }
            $(`.banner-header > nav > a[href='#${elem}']`).classList.add('active-nav-item');
        } else {
            $(`.banner-header > nav > a[href='#${elem}']`).classList.remove('active-nav-item');
        }
    });
}

function elementEvent(element, onEvent, eventCallback) {
    if(element.addEventListener) {
        element.addEventListener(onEvent, eventCallback, false);
    } else {
        element.attachEvent(onEvent, eventCallback);
    }
}

elementEvent(window, 'scroll', windowScroll);


打开控制台时,我只是注意到一些东西,该控制台显示此错误消息,请参阅第6行:


  未捕获的TypeError:无法读取null的属性'offsetTop'


offsetTop属性使它变得复杂吗?我应该更改我的代码以使其正确吗?

最佳答案

offsetTop没问题。但是,错误Uncaught TypeError: Cannot read property 'offsetTop' of null表示父对象是null,并且null.offsetTop不存在。因此,看起来$(`#${elem}`)正在为某些元素返回null。确保数组中的每个元素都存在于DOM中。

07-24 16:37