Как обрабатывать ошибку AJAX запроса в готовом проекте?

Question

[khipster]

У меня логика такая: не выводить же сразу сообщение об ошибке пользователю (вообще бесят все эти сообщения), нужно просто ещё пару раз попытаться послать запрос, а если не сработает, то уже ничего не поделаешь. timeout использовать не хочу, т.к. а вдруг у пользователя медленное соединение? И что в итоге? Я сам оборву запрос через N секунд, когда всё на самом деле было ладненько.

var request = new XMLHttpRequest();
console.log("пробуем отправить запрос 1 раз");
request.open("GET", "index.php", true);
var i = 2;
request.onreadystatechange = function() {
	if (request.readyState == 4 ) { // Если запрос завершен.
		if (request.status == 200) { // Если код ответа ОК.
			// Бинго!
		}
		else if (i != 4) {
			setTimeout(function() {
				console.log("пробуем отправить запрос " + i + " раз через 1 сек");
				request.open("GET", "index.php", true);
				request.send();
				i++;
			}, 1000);
		}
    else {
    	console.log("Ошибка!");
    }
	}
}
request.send();

https://jsfiddle.net/pau84w4p/5/

Answer 1

[Dark Hole]

Логика? Вы запускаете цикл. В итоге - 3 почти одновременных запросов.
+ прикрепляете онриди непонятно зачем.

Comments

[khipster]

Ну можно сделать таймаут на пару секунд.

[Dark Hole]

khipster: ну можно. Но в таком виде нельзя. Проще всего - если ошибка(т.е. код не равен 200) запускать еще один

[khipster]

Dark Hole: Во! Почему в таком виде нельзя то? :)
var i = 0;
var timerId = setInterval(function() {
var request = new XMLHttpRequest();
request.open("GET", "index.php", true);
request.onreadystatechange = function() {
if (request.readyState == 4 ) { // Если запрос завершен.
if (request.status == 200) { // Если код ответа ОК.
// Бинго!
clearInterval(timerId);
}
}
}
if (i == 2) {
// Печаль беда.
clearInterval(timerId);
}
i++;
}, 1000);

[khipster]

А всё понял, ты про то что я обработчик в цикл засунул. Ща перепишу.

[Dark Hole]

khipster: пишите прямо в кодепен или джсфиддл плиз

[khipster]

Dark Hole: https://jsfiddle.net/pau84w4p/5/ норм?

[Dark Hole]

khipster: мне так больше нравится:

var request = new XMLHttpRequest();
request.counter = 2;
function f(){
	request.counter--;
	request.open("GET", "index.php", true);
	request.send();
	request.onreadystatechange = function() {
		if (request.readyState == 4 ) { // Если запрос завершен.
			if (request.status == 200) { // Если код ответа ОК.
				// Бинго!
			} else {(request.counter)?(f();):(break;);}
		}
	}
}

[khipster]

Dark Hole: похоже на эльфийский)

[Dark Hole]

khipster: ага