Как правильно объявить условие?

Question

[Alibek Kulseitov]

В общем суть такая, нужно чтобы определенная функция срабатывала когда экран больше 767px а когда ниже другая.
Так вот проблема в том что эта функция сама имеет if/else statement. Как тут можно обернуть все это?

var realDocumentHeight = getRealDocumentHeight();
    
    function responsiveBird(x) {
        if (x.matches) { // If media query matches

            $('.parallax').scroll(function(){
                if ( $(this).scrollTop() > realDocumentHeight - $('.footer').outerHeight() - $('.parallax').outerHeight()) {
                    loop();
                } else {
                    console.log("Bye");
                }
            });

        } else {

            $(window).scroll(function(){
                if ( $(this).scrollTop() > realDocumentHeight - $('.footer').outerHeight() - $(window).outerHeight()) {
                    loop();
                } else {
                    console.log("Bye");
                }
            });
        }

    }

    var x = window.matchMedia("(max-width: 767px)");
    responsiveBird(x); // Call listener function at run time
    x.addListener(responsiveBird); // Attach listener function on state changes

Answer 1

[Александр Косицын]

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