После нескольких повторений ajax -ошибка, в чем проблема?

Question

[paulvales]

Здравствуйте, выполняется AJAX каждые 15 секунд, после некоторого времени появляется ошибка ubderfiend. Как избавиться от этой ошибки?

function news() {
var msg = $('#formx').serialize();
for(var i = 1; i < 1000; i++) {window.clearInterval(i);}
var newstimer = window.setInterval('news();', 15000);
$('#conn').addClass('progress');
$.ajax({
type: 'POST',
url: '/a/new.php',
data: msg,
success:function(data) {
$('#conn').html(data);
},
complete: function() {
// деактивация индикатора
$('#conn').removeClass('progress');
},
error: function(xhr, str){
alert('Возникла ошибка: ' + xhr.responseCode);
}
});
}

Comments

[Евгений Петров]

Во-первых, посмотреть, в какой строке возникает ошибка и здесь написать.

Answer 1

[Евгений Петров]

Вместо генерации кучи отложенных вызовов

var newstimer = window.setInterval('news();', 15000);

нужно вызывать один раз
setTimeout(news, 15000);

Вы хоть понимаете, что после первого отложенного вызова news() через 15 секунд эта функция будет вызвана 2 раза, после следующих 15 секунд 4 раза, и так далее в геометрической прогрессии 2ⁿ?

Comments

[paulvales]

мне нужен именно интервал,а не один раз. Чтобы через каждые 15 сек происходило обновление

[Евгений Петров]

Вижу, вы просто не понимаете... Ещё раз на пальцах — вы постоянно вызываете функцию news. Или вы внутри неё создаёте отложенный вызов с помощью setTimeout, или вне этой функции запускаете генерацию отложенных функций с помощью setInterval.

Но, если вы в теле функции каждый раз вызываете setInterval, вы получаете через 15 секунд удвоенное количество вызовов. То есть, через 60 секунд вы получите 8 вызовов в одно и то же время. А через 2 минуты запустится 128 вызовов news. Вы этого хотите?)

[Евгений Петров]

Вот нормальный пример jsfiddle.net/petroveg/yhshyxhc
А вот с вашей логикой jsfiddle.net/petroveg/yhshyxhc/1
А вот пример с правильным применением setInterval jsfiddle.net/petroveg/yhshyxhc/2

[paulvales]

Евгений Петров: Благодарю, а от "for(var i = 1; i < 1000; i++) {clearTimeout(i);}" можно избавиться?

[Евгений Петров]

Я вообще не понимаю вашей логики. Если у вас есть потребность в постоянных запросах, то, во-первых, стоит посмотреть в сторону WebSockets. Во-вторых, если всё-таки AJAX, то после получения ответа вызывать setTimeout для этой же функции.

Не нужно вам никаких setInterval, ну а то, что вы с ними делаете, не укладывается в моё понимание...

[paulvales]

Евгений Петров: понял,использовать interval только вне тела функции

Answer 2

[xmoonlight]

try-catch

Answer 3

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

var newstimer = window.setInterval(function(){news();}, 15000);

Comments

[Евгений Петров]

Никакой разницы с вариантом автора, поскольку в коде нет ничего из внешних контекстов.

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

Евгений Петров: Только что тестировал в хроме, вариант автора мне выдает ошибку, что функция news не найдена.

[Евгений Петров]

Если функция news() находится в глобальном контексте, то она будет найдена при выполнении eval.

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

Я в document ready оборачиваю и у меня эту функцию не видно. Думаю, что у автора тоже функция в document ready находится. Хотя конечно не могу быть уверен в этом на 100%)

[Евгений Петров]

Я тоже не уверен — ведь с ошибкой ubderfiend я не сталкивался никогда) Однако, функция news() может быть в глобальном контексте — скрипт в конце body после формы и ждать построения DOM Не нужно, функция news() вызывается вручную (тем же кликом etc.) и также стоит просто в глобальном контексте.

Ну и я не вижу необходимости формировать анонимную функцию, если можно передать ссылку на news.

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

Евгений Петров: Скорее всего у автора все таки document ready. Кстати Вы автору сказали про setTimeout, а ему нужен setInterval )

[Евгений Петров]

Александр Крупнов: Если отложенный вызов news формировать в теле функции news, то именно setTimeout.

[paulvales]

Александр Крупнов: Просто у меня несколько AJAX функций на странице и чтобы они не перебивали друг друга из-за интервала я решил сперва очистить все интервалы и задавать каждому свой
...for(var i = 1; i < 1000; i++) {window.clearInterval(i);}
var newstimer = window.setInterval('news();', 15000); .....
...for(var i = 1; i < 1000; i++) {window.clearInterval(i);}
var newstimer = window.setInterval('my();', 15000); .....

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

Евгений Петров: А, Вы имели ввиду в теле функции, понял) Просто я подумал про внешний вызов функции в setInterval )

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

paulvales: У Вас AJAX вызовы обновляют одни и теже элементы на странице? Или разные все таки?

[Евгений Петров]

paulvales: Вы сейчас что-то страшное описываете.

[paulvales]

Александр Крупнов: не совсем понял. Ситуация такая:один запрос выводит в div#content информацию по всем заказам, другой выводит в этот же див только сегодняшние заказы и т.д.

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

paulvales: Так разделите все по элементам - в одним html элементах все заказы, в других только за сегодня и обновляйте все отдельными запросами. Но! Наверное вообще все лучше сделать на стороне сервера, чтобы одним запросом получать сразу всю информацию)

[paulvales]

Евгений Петров: ну я новичек в этом деле)что в сети более подходящее нахожу ,то и стараюсь использовать

[Евгений Петров]

Александр Крупнов: В коде ТС функция организует стэк отложенных вызовов себя в своём же теле. Вот результат такого поведения jsfiddle.net/petroveg/yhshyxhc/1

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

Евгений Петров: Да это я понял. Я бы вообще вынес setInterval вне тела функции и вызывал бы по timeout.

[Евгений Петров]

Александр Крупнов: С учётом того, что по истечению интервала времени будет запущен асихронный вызов, то использование setInterval вообще не нужно. Получили, запустили отложенный вызов. Получили, запустили...

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

Евгений Петров: Можно и так)