Почему при выполнении ajax запроса, получив 404 ошибку, останавливается весь скрипт?

Question

[Артём Щурин]

Всем добрый день

Отправляю ajax запрос, с заведомо неправильным именем файла, ибо ситуция такая может случится решил проверить, получаю 404, ну норм его и ждали, а потом остановка выполнения последующего кода.

добавил метод fail() в котором вызвал abort() на неудачном запросе

var cleanSvg;
var ajaxRequest = $.ajax({
    url: url,
    dataType: 'text',
    success: function(xml) {

        cleanSvg = cleanLayout($(xml));

    },
    error: function() {

       console.error('Failed to load: ' + url);

    }
}).done(function() {

     callback(cleanSvg);

}).fail(
     ajaxRequest.abort();
);

Проблему то решил, но для меня оказалась совсем неочевидной остановка дальнейшего выполнение скрипта, как минимум метод error отработал и в консоли сначала появилась 404 ошибка, которая пришла из jQuery, а затем уже мною написанное console.error()

Разве остановка скрипта нормальное поведение и каждый раз такие вещи вручную нужно обрабатывать или я совсем запросы отправлять не умею?

Comments

[Илья Шатохин]

Оберните код через соответствующий тег, а то это невозможно читать.

[Артём Щурин]

.fail(function(){
         console.error('Failed to load: ' + url);
         ajaxRequest.abort();
    }
)

если всю обработку ошибок поместить в fail() опять же получаю остановку скрипта

Answer 1

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

А зачем вы используете error и fail одновременно? success и done?
С учётом ввода обещаний технически грамотным считается употребление done и fail.
Вызывать abort() после того, как запрос зафейлится?.. А зачем? А вот указать timeout стоило бы.

Comments

[Артём Щурин]

Error можно убрать не спорю.
Одну и ту же svg могут запросить несколько экземпляров, в success я обрабатываю полученную svg, в done вызов колбэка экзепляра запросившего ее, если еще один экземпляр её запросит, запрос не пойдет, просто добавится еще 1 done с колбэком запросившего экзепляра.
Если не вызываю abort(), получаю полную остановку последующего кода

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

Артём Щурин: Всё равно не понимаю.
1. Вызвали функцию, передали каллбэк в метод then(). Функция возвращает обещание.
2. Внутри функции проверили наличие объекта. Если уже загружен и сохранён, резолвим с объектом.
3. Если объекта нет, запрашиваем и на всякий пожарный указываем timeout (если запрос зависнет). В done() резолвим полученный объект. Из fail() резолвим пустышку.

Коллбэк-функция проверяет полученный объект и что-то делает, если он есть. Никакой отановки не происходит, да и вроде не с чего — запросы-то асинхронные.

[Артём Щурин]

Евгений Петров: остановка происходит когда я запрос на несуществующий адрес отправляю

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

Неважно что — callback функция, переданная как параметр success, или в метод done(), выполнится после того, как readyState будет 4.
Зачем и кому нужен abort()?

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

Я вам про timeout потому и сказал, что abort() требуется только для случая, когда от сервера нет ответа.
Если сработал любая callback-функция, abort() не имеет смысла.

[Артём Щурин]

Евгений Петров: так почему при 404 ошибке от ервера, когда адрес запросы был неправильный, дальнейшее выполнение скрипта прекращается?

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

Артём Щурин: Я не понимаю, что вы имеете ввиду под прекращением.