Ajax Too Slow - Recursion

A ajax code that i am using to request a page is consuming too much memory and is making the browser slow and everything lag. It seems like there is recursion going on and i dont k

Solution 1:

You are recursing and you shouldn't be using this form of nested setInterval. Doing this, will cause an explosion of interval instances. Instead of using setInterval, schedule additional requests using setTimeout.

setInterval will fire and continue firing every interval until you tell it to stop.

setTimeout will fire once.

---

Let's consider the following code which should address some of the issues you are having in this question as well as your other 2 questions.

First off, as we said before, don't use setInterval unless you actually want it to run forever. Additionally, don't nest the setInterval creations unless you actually mean to.

Instead, let's create a recursive function getTimeLeft() that will handle firing the request and scheduling the next check for time left after some duration.

This example also mocks the $.ajax() function so that you can see the function in action since we don't have an actual back end to use.

// Fake server side data to keep track of time leftsconst timeLefts = {
  foo: 0,
  bar: 0,
  fizz: 0,
  buzz: 0
};
const timeLeftsUpdateInterval = setInterval(() => {
  for (const [key, val] ofObject.entries(timeLefts)) {
    timeLefts[key] = Math.min(val + Math.random() * 10, 100);
  }
  if (Object.entries(timeLefts).every(([k, v]) => v >= 100)) {
    clearInterval(timeLeftsUpdateInterval);
  }
}, 1000);

// Mock $.ajax function to stub sending AJAX requestsfunction$ajax(kwargs) {
  return {
    done: cb => {
      setTimeout(() => {
        cb(timeLefts[kwargs.data.x]);
      }, 500);
    }
  };
}

// We will check for an update every second after the last request finishesconst timeLeftCheckInterval = 1000;

// Continuously check to see how much time is left for an elementfunctiongetTimeLeft(el) {
  // Make our request dataconst dataString = {
    s: "<?echo $_SESSION['currentview_'.$stamp]?>",
    r: "<?echo $search_usernumber?>",
    st: "<?echo $stamp?>",
    // My custom property to make this workx: el.dataset.item
  };

  // Make our request to get the time leftconst req = $ajax({ // Using our mock $.ajaxtype: "POST",
    url: "get_content_home.php",
    dataType: "html",
    data: dataString
  });

  // Once the request has finished
  req.done(data => {
    // Set the time left to the element
    el.innerHTML = data;

    // Have some condition so that you don't check for time left forever// Eventually there will be no time left right?  Why keep checking?if (data.timeleft <= 0) return;

    // Schedule another round of checking for time left after some durationsetTimeout(() => {
      getTimeLeft(el);
    }, timeLeftCheckInterval);
  });
}

// Kick off getting timeleft for all .itemsArray.from(document.querySelectorAll(".item"))
  .forEach(el =>getTimeLeft(el));

<ul><liclass="item"data-item="foo"></li><liclass="item"data-item="bar"></li><liclass="item"data-item="fizz"></li><liclass="item"data-item="buzz"></li></ul>

This code will address the issue that you are having in 2 Ajax non-blocking because each element will have it's own logic of going and fetching time left and updating itself.

This also addresses the issue that you are potentially facing in Timer in Ajax - Preemption because now the element won't check to see how much time is left again until after the previous check is finished.