Search code examples
jqueryajaxscrolllazy-loadingside-scroller

How to prevent multiple AJAX calls when using JQuery scroll event handler during lazy-loading?


I load data into my DIV with AJAX using 3 different functions. I am trying to prevent multiple AJAX calls when the browser scroller remains at the bottom of the DIV that I am lazy-loading data into.

This works but sometimes (just sometimes) the scroller remain at the bottom of my div and this will cause many AJAX calls to happen. How do I prevent that from happening?

$(document).ready(function() {

    //form submit by click
    $("#submit").click(function(e) {
        // Prevent Default Action In Case to Load data by form
        e.preventDefault();

        //prevent select to post empty data
        $('select').each(function() {
            if ($(this).val() == '') {
                $(this).attr('disabled', 'disabled');
            }
        });

        // Define what we need
        var loading = "<img src='/zojfa/images/loading.gif' alt='Loading...' />";
        var scrolltop = $('#scrollbox').attr('scrollTop');
        var scrollheight = $('#scrollbox').attr('scrollHeight');
        var windowheight = $('#scrollbox').attr('clientHeight');
        var post = $(this).attr("name") + "=" + $(this).val();
        var form_data = $('#search_form').serialize() + "&" + post;
        var scrolloffset = 20;

        //empty content if another value sent to code
        $('#content').empty();
        //load data
        loaddata(form_data, 0);
    });

    //listen to scroll function
    $('#scrollbox').scroll(function() {

        //define what we need
        var scrolltop = $('#scrollbox').attr('scrollTop');
        var scrollheight = $('#scrollbox').attr('scrollHeight');
        var windowheight = $('#scrollbox').attr('clientHeight');
        var scrolloffset = 20;

        //get number of div which will append to script in case of limit database to page 2 or 3 or...
        size = $('#content > div').children().size();



        //if we reach to bottom of div we are going to call to ajax function
        if (scrolltop >= (scrollheight - (windowheight + scrolloffset))) {
            var form_data = $('#formdata').val();

            //if remain of size(count of div) is 0 then we have more data to show because we limit data provided by script to 7 field(we handle situation that we had 14 or 21 respond from database in "next step" because if there is no data to show we dont have to let script to run)
            if (size % 7 == 0) {
                //call to load data function
                setTimeout(function() { loaddata(form_data, size) }, 1500);
            } else {
                //do nothing its just in case we need to append something like no more data to load
            }
        }
    });

    // page load finish
});


//function to load data
function loaddata(form_data, size) {
    number = "&size=" + size;
    //fetch new items
    $.post('dosearch.php', form_data + number, function(newitems) {
        //next step : if page echoing "" then do nothing
        if (newitems == '') {} else {
            //if we have data append these data to our div's #content
            $('#content').append(newitems);
        }
    });
}​

update

I did just as dear @Kent Pawar and dear @E.J. Brennan said but now I get more AJAX calls when I reach to bottom of the div and it still doesn't quite work.

$("#submit").click(function(e) {
    // Do some Default
    e.preventDefault();

    $('select').each(function() {
        if ($(this).val() == '') {
            $(this).attr('disabled', 'disabled');
        }
    });

    // var what we need
    var loading = "<img src='/zojfa/images/loading.gif' alt='Loading...' />";
    var scrolltop = $('#scrollbox').attr('scrollTop');
    var scrollheight = $('#scrollbox').attr('scrollHeight');
    var windowheight = $('#scrollbox').attr('clientHeight');
    var post = $(this).attr("name") + "=" + $(this).val();
    var form_data = $('#search_form').serialize() + "&" + post;
    var scrolloffset = 20;

    $('#content').empty();

    //load data
    loaddata(form_data, 0);
    $('select').each(function() {
        if ($(this).val() == '') {
            $(this).removeAttr('disabled');
        }
    });
});


$('#scrollbox').scroll(function() {
    //var what we need
    var scrolltop = $('#scrollbox').attr('scrollTop');
    var scrollheight = $('#scrollbox').attr('scrollHeight');
    var windowheight = $('#scrollbox').attr('clientHeight');
    var scrolloffset = 20;

    // when we reach
    size = $('#content > div').children().size();

    if ($('#scrollbox').data('ajaxready') === false)
        return;

    if (scrolltop >= (scrollheight - (windowheight + scrolloffset))) {
        $('#scrollbox').data('ajaxready', false);
        var form_data = $('#formdata').val();

        if (size % 7 == 0) {
            setTimeout(function() { loaddata(form_data, size) }, 1500);
        } else {

        }
    }
    $('#scrollbox').data('ajaxready', true);

    // page load finish
});

function loaddata(form_data, size) {
    number = "&size=" + size;
    //fetch new items
    $.post('dosearch.php', form_data + number, function(newitems) {
        if (newitems == '') {} else {
            $('#content').append(newitems);
        }
    });
}

Solution

  • Well the issue isn't with the scroller being at the bottom of the page but with the way the event handler works. There are times the scroller would be at the bottom of the page like say when there are no more post to load... observe Facebook's wall for example.

    Currently, the JQuery scroll event is triggered when scrolling occurs.

    JQuery docs:

    A scroll event is sent whenever the element's scroll position changes, regardless of the cause. A mouse click or drag on the scroll bar, dragging inside the element, pressing the arrow keys, or using the mouse's scroll wheel could cause this event.

    Now its the job of your script to make a single AJAX call to check if there is content to be loaded. You need to modify your script to stop multiple AJAX calls from taking place during this time and as I see @E.J. Brennan has already suggested the same :).

    You could add the flags as follows:

    //listen to scroll function
      $('#scrollbox').scroll(function(){
    
            //[Kent] Before we service the event, we check if the last scroll event was handled/completed.
            //If it is not yet compelted, don't start another one and jump out of the code.
            if ($(window).data('ajax_in_progress') === true)
                return;
    
            //define what we need
            var scrolltop=$('#scrollbox').attr('scrollTop');
            var scrollheight=$('#scrollbox').attr('scrollHeight');
            var windowheight=$('#scrollbox').attr('clientHeight');
            var scrolloffset=20;
    
            //get number of div which will append to script in case of limit database to page 2 or 3 or...
            size =  $('#content > div').children().size();
    
            //if we reach to bottom of div we are going to call to ajax function
            if(scrolltop>=(scrollheight-(windowheight+scrolloffset))){
                $(window).data('ajax_in_progress', true);  //[Kent] prevent more scroll events as AJAX request will soon begin. 
    
                var form_data = $('#formdata').val();
    
                // if remain of size(count of div) is 0 then we have more data to show because 
                // we limit data provided by script to 7 field(we handle situation that we had 
                // 14 or 21 respond from database in "next step" because if there is no data to 
                // show we dont have to let script to run)
                if(size % 7 == 0){
                    //call to load data function
                    setTimeout(function(){loaddata(form_data, size)}, 1500);
                }else{
                    //do nothing its just in case we need to append something like no more data to load
                }
            }
        });    
    
    
    
    //function to load data
    function loaddata(form_data, size){
        number = "&size=" + size;
        //fetch new items
        $.post('dosearch.php', form_data+number, function(newitems){
            // [Kent] This is the callback funciton that gets executed ONLY
            // when the AJAX request has completed. We will append the data
            // into the DOM and then reset the FLAG.
    
            //next step : if page echoing "" then do nothing
            if(newitems == ''){
            }else{
                //if we have data append these data to our div's #content
                $('#content').append(newitems).each(function() {
                    //Adding a callback to append.
                    // So we reset the flags here to indicate the scroll event 
                    // has been handled/completed and so we turn scrolling events back on
                    $(window).data('ajax_in_progress', false);
                });
            }
        });
    }