質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

Q&A

4回答

2538閲覧

whenを使用して非同期処理した場合のエラーハンドリングについて

beatle7

総合スコア8

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

0グッド

0クリップ

投稿2018/07/18 02:34

前提・実現したいこと

下記のようなコードで非同期通信を試みています。
URLの配列を作成し、コンテンツを非同期で取得しています。

javascript

1var targetArray = [ 2"a.html", 3"b.html", 4"c.html" 5] 6 7for(var i = 0; i < targetArray.length; i++){ 8 jqXHRList.push($.ajax({ 9 url: targetArray[i], 10 datatype: 'html' 11 }) 12} 13 14$.when.apply($, jqXHRList).done(function() { 15/**do something**/ 16});

発生している問題・エラーメッセージ

上記のケースにおいて、配列内のURLで一つでも500や404などのエラーが発生した場合、コンテンツ取得処理全体が停止してしまいます。
エラーが帰ってきた場合は配列内のURLをスキップして次のURLを取得させたいです。
どのような記述が適当でしょうか?

よろしくお願い致します。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答4

0

以下動作を確認してみてください

  • hoge.html

javascript

1<script src="http://ajax.googleapis.com/ajax/libs/jquery/1.12.4/jquery.min.js"></script> 2<script> 3$(function(){ 4 $("#text").text("start"); 5 var d = $.Deferred(); 6 $.when( 7 $.ajax({url:"fuga.php?sleep=2"}).done(function(data){$("#text").text($("#text").text()+" "+data);d.resolve();}), 8 $.ajax({url:"fuga.php?sleep=3"}).done(function(data){$("#text").text($("#text").text()+" "+data);d.resolve();}), 9 $.ajax({url:"fuga.php?sleep=1"}).done(function(data){$("#text").text($("#text").text()+" "+data);d.resolve();}), 10 $.ajax({url:"fuga.php?sleep=5"}).done(function(data){$("#text").text($("#text").text()+" "+data);d.resolve();}), 11 $.ajax({url:"fuga.php?sleep=4"}).done(function(data){$("#text").text($("#text").text()+" "+data);d.resolve();}) 12 ).done(function(){ 13 $("#text").text($("#text").text()+" end"); 14 }).fail(function(xhr,err){ 15 $("#text").text($("#text").text()+" "+err); 16 }); 17}); 18</script> 19<div id="text"></div>
  • fuga.php

PHP

1<?php 2$sleep=filter_input(INPUT_GET,"sleep",FILTER_VALIDATE_INT,["options"=>["default"=>0]]); 3sleep($sleep); 4if($sleep==3){ 5 header('HTTP/1.1 404 Not Found'); 6 print "not found"; 7 exit; 8} 9print $sleep; 10?>

非同期で5つのファイルをよみ、sleep=3が404エラーとなりますが
その後も読み続けます。
ただしエラーの処理なのですべてがおわってもdoneを実行しません

投稿2018/07/18 03:23

yambejp

総合スコア114572

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

failでエラー感知出来ませんか?

投稿2018/07/18 02:49

m.ts10806

総合スコア80765

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

so87 さんに便乗して脱jQuery脱co

js

1const apiRequest = url => fetch(url, { 2 method: 'POST' 3}); 4 5const apiUrlList = [ 6 '/echo/xml/', 7 '/echo/error/', 8 '/echo/json/', 9 '/echo/html/', 10] 11 12const execute = async urls => { 13 const resultPromises = urls.map(apiRequest); 14 const results = await Promise.all(resultPromises); 15 console.log('結果', results); 16} 17 18execute(apiUrlList);

投稿2018/07/18 05:59

og24715

総合スコア832

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

$.whenということで並行処理で示します。
coというライブラリを利用してます。大変便利です。
async/awaitやジェネレータを使っているのでブラウザで使うにはbabel等が必要ですが、
環境が整えば非同期処理が書きやすくなるので、環境構築頑張ってみてください。

js

1/** fetch使ってますがポイントはエラーをresolveで握りつぶすトコです */ 2const api = url => new Promise( resolve => { 3 fetch(url, {method : 'POST'}).then(resolve).catch(resolve); 4} ); 5 6/** api関すに投げるURLリスト定義 */ 7const apiUrlList = [ 8 '/echo/xml/', 9 '/echo/error/', 10 '/echo/json/', 11 '/echo/html/', 12] 13 14const execute = async () => { 15 // co.wrapでPromiseとして扱える → なのでawaitできちゃう 16 const result = await co.wrap( function* () { 17 // keyとvalueにPromiseをセットしたObjectを作成しておく 18 const jobList = apiUrlList.reduce( ( acc, url ) => { 19 acc[ url ] = api( url ); //Promise走らせちゃう 後でyieldで終わるまで待たせます 20 return acc; 21 }, {} ) 22 return yield jobList; //ここで全部のPromiseが終わるまで待て!の命令 yieldの登場です。 23 } )(); // Promiseなので実行させてawaitさせます 24 console.log( '結果',result ); //結果をコンソールに出力 URLをkeyにして、fetchの結果が格納されてます。 25 26} 27 28execute(); 29

デモ:https://jsfiddle.net/b0zf42mc/29/

直列処理の場合、await/asyncやジェネレータ/coならば
Promiseキューを作って、forでエラーを握りつぶしながら回せば完了です。
IEが無視できるのならば新しい書き方がおすすめです。

投稿2018/07/18 03:25

編集2018/07/18 03:39
so87

総合スコア764

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

so87

2018/07/18 03:49

ぜんぜん$.whenとか使ってなくてごめんなさい。jsは新しい非同期処理の仕組みが使えるようになってきているので、jqueryを投げ捨てる時が来ているということで。
so87

2018/07/18 07:20

そうでした。今回のような例題ではPromise.all()で十分ですね。 自分はPromise.allの戻り値が気に入らなくて、よくcoを使ってkey-valueなオブジェクトにPromiseつっこんで使ってるので、つい回りくどい回答となってしまいました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問