6

fetch を適切に使用する方法について少し混乱しています。エラーステータスが戻ってきても解決するようです。以下のコードは正しいですか (フェッチを別のプロミスでラップしています)?

function a(url, config) {
  if (!config)
    config = {};

  config = Object.assign(config, {
    headers: {
      'content-type': 'application/json;charset=utf-8'
    }
  })
  return new Promise(
    function(resolve, reject) {
      fetch(url, config).then(
        function(res) {
          if (res.status == 200 && res.ok) {
            console.log("Promise resolved")
            resolve(res);
            return;
          }
          console.log("Promise rejected")
          reject(res);
        },
        function(rej) {
          console.log("promise rejected")
          reject(rej);
        }
      )
    }
  )
}

function b() {
  a('/test').then(
    function(res) {
      console.log('success');
      console.log(res)
    },
    function(rej) {
      console.log('Rejected');
      console.log(rej)
    }
  )
}

b();
(上記のコードは、コンソール経由でクロムで正常に実行されるはずです...コピー/貼り付けするだけです)

4

1 に答える 1

2

成功コールバックから拒否したい場合は、拒否されたプロミスを返すreturn Promise.reject('error occurred');か、スローするなどして、明示的に行う必要があります。

さらに、すでに promise オブジェクトを返しPromiseているため、コンストラクターを悪用しないでください。fetch

function a(url, config) {
    if (!config)
        config = {};

    config = Object.assign(config, {
        headers: {
            'content-type': 'application/json;charset=utf-8'
        }
    });

    return fetch(url, config).then(
        function(res) {
            if (res.status == 200 && res.ok) {
                console.log("Promise resolved")
                return res.json();
            }
            console.log("Promise rejected");
            throw 'promise rejected';
        },
        function(rej) {
            console.log("promise rejected");
            throw 'promise rejected';
        }
    );
}
于 2015-12-06T10:39:55.620 に答える