Ошибка при скролле страницы: Uncaught TypeError: Cannot read property 'top' of undefined

@Kayker · Регистрация: 22.05.2014

Студворк — интернет-сервис помощи студентам

Доброго времени суток.
Написал скрипт посекционной прокрутки лендинга, скрипт работает, но в консоли появляется ошибка. которая множится при скроле
scroll.js:5 Uncaught TypeError: Cannot read property 'top' of undefined
at scroll.js:5
html выглядит так

HTML5
1
2
3
4
5
<section class='visible'></section>
<section></section>
<section></section>
<section></section>
<section></section>

Вот скрипт, ошибка в пятой строке

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
$(document).ready(function () {
    var lastScrollTop = 0;
    $(window, 'body,html').scroll(function (event) {
        var st = $(this).scrollTop();
        var action_top = $('.visible').offset().top;
        if (st > action_top) {
          if(st-action_top>250){
              var top= $('.visible').next('section').offset().top;
              var next=$('.visible').next('section');
              $('.visible').removeClass('visible');
              $('body,html').animate({scrollTop: top},1000);
              setTimeout(function () {
                  next.addClass('visible');
              },1300)
              lastScrollTop = $(window).scrollTop();
              return false
          }
 
 
 
 
        }
        else {
            var top= $('.visible').prev('section').offset().top;
            var next=$('.visible').prev('section');
            $('.visible').removeClass('visible');
            $('body,html').animate({scrollTop: top},1000);
            setTimeout(function () {
                next.addClass('visible');
            },1300)
            lastScrollTop = $(window).scrollTop();
            return false
        }
 
 
    });
})

@Уф · 08.02.2018, 10:54

а че не понятного, когда ты мышкой крутишь то событие скролл срабатывает и видит твой класс визибл, а когда из функции перемотка запускается в animate, то ты класс visible удаляешь и в течении всей секунды опять к этому событию обращаешься, но оно уже не может найти элементы с таким классом. решение простое:

JavaScript
1
2
3
4
    var lastScrollTop = 0;
    $(window, 'body,html').scroll(function (event) {
        if($('.visible').length==0) return;
        .........

@Balanaar · 08.02.2018, 10:57

В определённый момент на 10й строке вы удаляете класс visible с элемента.
Последующие после этого выборки $('.visible') дают пустую коллекцию, а применение метода offset() к пустой коллекции вернёт undefined. Вы не можете считать свойство top у undefined.

@Kayker · 08.02.2018, 11:42 **[ТС]**

Спасибо, ушла ошибка, не учёл, что событие scroll срабатывает не только на прокрутку, но и воообще на проскроленную страницу

Новые блоги и статьи Все статьи Все блоги /
Подстановка значения реквизита справочника в табличную часть документа Maks 10.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеПерсонала", разработанного в конфигурации КА2. Задача: при выборе сотрудника (справочник Сотрудники) в ТЧ документа. . .	Очистка реквизитов документа при копировании Maks 09.04.2026 Алгоритм из решения ниже применим как для типовых, так и для нетиповых документов на самых различных конфигурациях. Задача: при копировании документа очищать определенные реквизиты и табличную. . .	модель ЗдравоСохранения 8. Подготовка к разному выполнению заданий anaschu 08.04.2026 https:/ / github. com/ shumilovas/ med2. git main ветка * содержимое блока дэлэй из старой модели теперь внутри зайца новой модели 8ATzM_2aurI	Блокировка документа от изменений, если он открыт у другого пользователя Maks 08.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа, разработанного в конфигурации КА2. Задача: запретить редактирование документа, если он открыт у другого пользователя. / / . . .
Система безопасности+живучести для сервера-слоя интернета (сети). Двойная привязка. Hrethgir 08.04.2026 Далее были размышления о системе безопасности. Сообщения с наклонным текстом - мои. А как нам будет можно проверить, что ссылка наша, а не подделана хулиганами, которая выбросит на другую ветку и. . .	Модель ЗдрввоСохранения 7: больше работников, больше ресурсов. anaschu 08.04.2026 работников и заданий может быть сколько угодно, но настроено всё так, что используется пока что только 20% kYBz3eJf3jQ	Дальние перспективы сервера - слоя сети с космологическим дизайном интефейса карты и логики. Hrethgir 07.04.2026 Дальнейшее ближайшее планирование вывело к размышлениям над дальними перспективами. И вот тут может быть даже будут нужны оценки специалистов, так как в дальних перспективах всё может очень сильно. . .	Горе от ума kumehtar 07.04.2026 Эта мне ментальная установка, что вот прямо сейчас, мол, мне для полного счастья не хватает (нужное вписать), и когда я этого достигну - тогда и полный кайф. Одна из самых сильных ловушек на пути. . . .

@Balanaar 2463 / 1769 / 625 Регистрация: 11.07.2016 Сообщений: 4,067
	08.02.2018, 10:57
	В определённый момент на 10й строке вы удаляете класс `visible` с элемента. Последующие после этого выборки `$('.visible')` дают пустую коллекцию, а применение метода `offset()` к пустой коллекции вернёт `undefined`. Вы не можете считать свойство `top` у `undefined`. 1

@Kayker 0 / 0 / 0 Регистрация: 22.05.2014 Сообщений: 77
	08.02.2018, 11:42 [ТС]
	Спасибо, ушла ошибка, не учёл, что событие scroll срабатывает не только на прокрутку, но и воообще на проскроленную страницу 0