5

Есть общий файл (main.js) для всех скриптов, в нём прописана функция для кнопки "наверх":

function button_top_right (n) {
    var top_button = $("#main").offset().top;
    $("#top_right").hide();
    $(window).scroll(function () {
        if ($(this).scrollTop() > top_button + n) {
            $('#top_right').fadeIn().addClass("fixed");
        } else {
            $('#top_right').fadeOut();
        }
    });
};
button_top_right (1500);

Для одной страницы мне надо вызвать эту функцию с другим параметром n.
Добавляю скрипт на эту страницу:

$(document).ready(function () {
    button_top_right (100);
});

Скрипт не срабатывает с ошибкой:

Uncaught ReferenceError: button_top_right is not defined

main.js подгружается раньше, чем вызов функции.
Что я делаю не так?

9
  • Кстати, если вы вызовете-таки эту функцию дважды на странице, то у вас будет висеть уже два обработчика scroll для window. Вряд ли это чем-то хорошим кончится.
    – Regent
    22 сен 2015 в 3:39
  • А объявление этой функции, случаем, не находится ли внутри другого document.ready?
    – Regent
    22 сен 2015 в 3:41
  • @Regent, вы правы. Если вынести функцию за пределы document.ready, то появится другая закономерная ошибка: Uncaught TypeError: Cannot read property 'top' of null. Да и два обработчика одновременно плохая идея, сразу не подумала об этом.
    – Елена
    22 сен 2015 в 4:00
  • По поводу top of null: достаточно button_top_right (1500); оставить внутри document.ready, а функцию вынести из него. В таком случае и функция будет везде доступна, и первый вызов будет своевременно.
    – Regent
    22 сен 2015 в 4:11
  • По поводу двух вызовов: можно при вызове функции удалять предыдущий обработчик (.off('scroll')), однако всегда ли в нужном порядке будут происходить вызовы - не знаю.
    – Regent
    22 сен 2015 в 4:12

1 ответ 1

6

Когда функция объявлена внутри document.ready, обратиться к ней извне не получится. Достаточно вынести саму функцию, а её вызовы оставить внутри document.ready:

function button_top_right(n) {
    ...
};
$(document).ready(function() {
    button_top_right(1500);
});

Так как внутри функции button_top_right при каждом вызове функции добавляется обработчик события scroll для window, имеет смысл убирать предыдущий обработчик либо перед вызовом функции, либо внутри функции перед добавлением нового:

$(window).off("scroll").on("scroll", function() {
    ...
});
1
  • Лучший, обнял, поднял!! 22 мар 2019 в 17:42

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.