JS-скрипт. Утечка памяти. Как исправить?

Question

[Александр Карабанов]

Привет.
Я вывожу логи с коммутаторов в окно браузера (этакий tail -F только в браузере). И всё даже работает:


Но с течением времени память заканчивается, SWAP растёт, компьютер начинает дико тормозить.
Вот так выглядит скрипт:

// Функция, которая проверяет нет ли в логе новых записей
  function updateLog()
  {
    $.getJSON('PHPTail.php?ajax=1&lastsize=' + lastSize + '&grep=' + grep,

    function(data) { lastSize = data.size;

    $.each(data.data, function(key, value)
    {
      // Выводим данные в соответствующий блок
      $("#results").append(value);
         delete data;
     }); 
  });
}

// Функция, которая удаляет старые записи
function removeOld()
{
  // Когда записей становиться моного стираем лишние
  if($(".line").length > 300) { $("#results .line:first").remove(); }
}

// Вызываем функцию, которая проверит нет ли в логе новых записей
var timerId = setTimeout(function tick() {
  updateLog();
  timerId = setTimeout(tick, 1600);
}, 1600);

// Вызываем функцию, которая удалит старые записи
var timerIdd = setTimeout(function tick() {
  removeOld();
  timerIdd = setTimeout(tick, 10000);
}, 10000);

Подозреваю, что функция removeOld() работает неправильно, либо вообще не работает. Как её заставить работать правильно?

Comments

[Maa-Kut]

Если removeOld не работает, то на страничке должны скапливаться записи в количестве более 300 штук. Этот эффект имеет место быть?

[Александр Карабанов]

Нет. Старые записи действительно стираются, вернее как... Их становится невидно, но я подозреваю, что в памяти они остаются.

Answer 1

[Lynn «Кофеман»]

Судя по коду, когда у вас больше 300 сообщений вы удаляете только одно, что довольно странно.

Ещё вместо setTimeout можно (и нужно) использовать setInterval.

function removeOld() {
  // Удаляем всё кроме последних трёхсот элементов.
  $(".line").slice(0, -300).remove();
}

setInterval(updateLog, 1600);
setInterval(removeOld, 10000);

Comments

[Александр Карабанов]

Да, мне надо удалять только одно самое старое. Так и задумано. Потом в лог добавиться ещё одна новая запись, и соответственно, ещё одна старая забись будет стёрта, таким образом в окне браузера постоянно присутствуют ~30 записей, что, в целом, достаточно.
Сейчас попробую setInterval...

[Lynn «Кофеман»]

Александр Карабанов: вы добавляете сообщения раз в полторы секунды, а удаляете раз в 10 секунд. Т.е. за 10 секунд вы можете добавить 5 сообщений, а удалите только одно. Либо вы чего-то не договариваете, либо у вас лог будет расти неограниченно

[Александр Карабанов]

А понял на счет " $(".line").slice(0, -300).remove();" спасибо. Так будет лучше.

[Lynn «Кофеман»]

Константин Громов: Что? Как раз и нужно удалить старые элементы

[Константин Громов]

Алексей Тен: Да, моя невнимательность.

Answer 2

[Андрей]

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

Comments

[Константин Громов]

setTimeout работает асинхронно и вместо увеличения стека просто ставит функцию в очередь на исполнение в общем потоке.