Na falta buscar em um Promise.all erro não captura

Question

Na falta buscar em um Promise.all erro não captura

votos

1

Eu tenho vários JSON para carga e deve verificar se todos eles estão bem improvável. Então eu uso um Promise.all que esperar por toda a fetch.

O primeiro valid.jsonexiste, não o segundo, então o segundo fetchtermina com um 404. Mas, apesar da Promise.reject, Promise.allainda registra Success!em vez de jogar o último erro.

Existe algo que eu perdi sobre como Promise.allfunciona?

const json_pathes = [
    'valid.json',
    'not_valid.json'
];

function check_errors(response) {
    if (!response.ok) {
        Promise.reject('Error while fetching data');
        throw Error(response.statusText + ' (' + response.url + ')');
    }
    return response;
}

Promise.all(json_pathes.map(url =>
    fetch(url)
        .then(check_errors)
        .then(response => response.json())
        .catch(error => console.log(error))
))
.then(data => {
    console.log('Success!', data);
})
.catch(reason => {
    throw Error(reason);
});

// Console:
// Error: Not Found (not_valid.json)
// uncaught exception: Error while fetching data
// Array [ […], undefined ]

(Verificado todas as perguntas semelhantes é claro, mas nada ajudou 😕)

Código fixo depois abaixo respostas - edit:

const json_pathes = […]
Promise.all(json_pathes.map(url =>
    fetch(url)
        .then(response => {
            if (!response.ok)
                throw Error(response.statusText + ' (' + response.url + ')');
            return response;
        })
        .then(response => response.json())
        .catch(error => {
            throw error;
        })
))
.then(data => {
    // Success
})
.catch(error => {
    throw error;
});

javascript promise fetch

Publicado 02/12/2019 em 21:57
fonte usuário Joan

Em outras línguas...

2 respostas

votos

1

você tem que jogar de erro novamente quando você usa o método .catch (), ou o erro será silenciado

Promise.all(
  json_paths.map(url => 
    fetch(url)
      .then(response => response.json())
      .catch(err => {
        console.log(err);
        throw err
      })
  )
).then(data => {
  // all promise resolved
  console.log(data)
}).catch(err => {
  // some promise may not be resolved
  console.log(err)
})

Respondeu 02/12/2019 em 22:05
fonte usuário Bad Dobby

trincot · Accepted Answer · 2019-12-02 22:02

Esta chamada:

 .catch(error => console.log(error))

... irá retornar uma promessa cumprida, não um rejeitado. Sempre que você tratar uma rejeição e você quer que ele a borbulhar como uma rejeição, você deve explicitamente fazer isso:

 .catch(error => {
       console.log(error);
       throw error; // cascade...
 })

By the way, isso não tem efeito em tudo

 Promise.reject('Error while fetching data');

... que você não faz nada com este recém-criado promessa, rejeitado.