1

У меня есть функция, отправляющая запросы на сервер и принимающая его ответ:

function send(send_id,url,data,cb_func){
        var xhr_data= new XMLHttpRequest();
        xhr_data.open('post',url+'&global_code='+global_code, true);
        xhr_data.timeout = 1500;
        xhr_data.ontimeout = function(){
               console.log("Лимит ожидания был превышен!");
               setTimeout(send(send_id,url,data,cb_func), 200);
        }
        var params = "&param"+123;
        xhr_data.setRequestHeader("Content-type", "application/x-www-form-urlencoded");
        xhr_data.setRequestHeader("Content-length", params.length);
        xhr_data.setRequestHeader("Connection", "close");
        xhr_data.onreadystatechange = function(){   
             if(!(this.readyState==4 && this.status==200 )) return; 
             cb_func && cb_func(JSON.parse(this.responseText));
            }
         xhr_data.send(params);
         };

Из-за того, что связь между сервером и клиентом непостоянна, возникает ситуация, когда программа ждет ответ на свой запрос, а он не приходит, так как связь потеряна. Это приводит к зависанию приложения. Я установила timeout, однако он не всегда срабатывает.

Как можно обработать такого рода ошибки?

2
  • Установка в конце функции таймаута var timer = setTimeout( function(){ xhr_data.abort();},60000); проблемы не решает совсем(((
    – nita
    23 дек 2015 в 5:46
  • в таком виде setTimeout(send(send_id,url,data,cb_func), 200); setTimeout бесполезен, его можно либо убрать либо переписать например так setTimeout(function(){send(send_id,url,data,cb_func)},200);
    – Grundy
    23 дек 2015 в 9:59

1 ответ 1

1

У Вас проблема в том, что вы ожидаете только статус 200. Поэтому, если статус не 200, и запрос выполнен (допустим со статусом 500), Вы не увидите результата. Попробуйте изменить .onreadystatuschange следующим образом:

xhr_data.onreadystatechange = function(){   
    if (this.readyState!==4) return;
    cb_func && cb_func(JSON.parse(this.responseText), this.status);   
}

Тогда, вторым аргументом в cb_func получите статус завершения запроса.

Есть еще вариант другой, когда первым аргументом передаете ошибку, если она есть, а вторым - результат (подход часто используется в node.js). Тогда, в случае this.status===200, можно вызвать cb_func(null, JSON.parse(this.responseText)), а в случае неудачи, к примеру cb_func(new Error('Request error. Status: '+this.status)).

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


Upd: Попробуйте, все-таки, в cb_func передавать просто this.responseText, а внутри cb_func парсить. Т.к. в случае срабатывания таймаута - this.responseText будет равен пустой строке, а при попытке вызвать JSON.parse('') Вы получите эксепшн.

11
  • подскажите, у меня вопрос по инструкции return, правильно ли я понимаю что если ответ от сервера не равен статусу 4, то return (т.е. вернуть ответ), и дальше ф-ция не будете отработана? (или return не останавливает выполнение цикла или ф-ции как например continue) 23 дек 2015 в 6:17
  • this.readyState это не сответ от сервера, это состояние запроса (1 - запрос открыт, 2 - приняты заголовки, 3 - получение ответа, 4 - запрос завершен, ответ получен), цикла там не будет, это callback функция, которая вызывается каждый раз, когда изменяется состояние запроса (либо браузер получил кусок тела ответа). return останавливает выполнение функции, именно это от него и требуется в данном случае (т.к. запрос не завершен, не стоит пытаться распарсить ответ). Подробнее про onreadystatechange можно почитать на xmlhttprequest.ru, или developer.mozilla.org 23 дек 2015 в 6:23
  • спасибо) у меня на самом деле периодически обрывается связь с сервером и никакой ответ, соответственно не приходит. Я попробовала отслеживать статус в cb_func, но я туда просто не захожу и соответственно не могу ничего в ней обработать.
    – nita
    23 дек 2015 в 8:23
  • Если Вы пробовали отслеживать статус так, как в вопросе, что вы ничего не увидите, если сервер ответил каким-либо статусом отличным от 200. Сэмулировать обрыв связи вменяемо не получается, но даже заморозка виртуалки с http сервером не мешает вызову onreadystatechande с this.readyState===4 по таймату. Но статус не будет 200, соответственно в Вашем коде, функция cb_func не вызовется. 23 дек 2015 в 9:29
  • Я пробовала выносить статус в функцию, но она не запустилась, просто все оборвалось на if, причем в условие тоже ничего не пошло. Может я просто неправильно все тестировала (сервер и клиент соединены друг с другом кабелем), запуская приложение и останавливая соединение? В итоге наблюдалась такая картина, что после разрыва соединения запросы преспокойно пытались достучаться до сервера, а после того, как связь была восстановлена, приложение зависло на отправке запроса. Почему так происходит, Вы не могли бы предположить?
    – nita
    23 дек 2015 в 13:02

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.