0
340
Viewsmensaje de error No capturado (en promesa), pero no puedo encontrar el problema después de agregar la función de captura
¿Por qué la captura agregada todavía no funciona en absoluto?
function maxRequest(url = ``, times = 3) { // closure function autoRetry (url, times) { console.log('times = ', times); times--; return new Promise((resolve, reject) => { fetch(url).then(value => { if(value.status === 200) { console.log(`✅ `, value); resolve(value); } else { throw new Error(`❌ http code error: ${value.status }`); } }).catch((err) => { console.log(`❌ Error`, err); if (times < 1) { reject('💩 over max request times!'); } else { autoRetry(url, times); } }); }); } return autoRetry(url, times); } maxRequest(`https://cdn.xgqfrms.xyz/json/badges.js`) .then(res => res.json()) .then(json => { console.log('json =', json); return json; }, err => { console.log('error =', err); throw new Error(err); }) .catch(err => console.log(`err =`, err)) .finally(() => { console.log('whatever close loading...'); }); 
·
Juan Pablo Isaza
1 answers
Answer question0
Su uso del antipatrón de construcción Promise explícito enmascara el hecho de que no regresa de su bloque catch . En una situación de reintento, crea una Promesa de segundo intento llamando a autoRetry , pero no hace nada con ella, por lo que el bloque catch en su llamada a maxRequest no lo protege y termina con una Promesa rechazada no detectada.
function maxRequest(url = ``, times = 3) { // closure function autoRetry (url, times) { console.log('times = ', times); times--; // No more `new Promise`, just a chain. return fetch(url).then(value => { if(value.status === 200) { console.log(`✅ `, value); return value; } else { throw new Error(`❌ http code error: ${value.status }`); } }).catch((err) => { console.log(`❌ Error`, err); if (times < 1) { // Converted to rejection. You could still throw a // string, but throwing an Error is more idiomatic. throw new Error('💩 over max request times!'); } else { // Make sure to return your Promise here. Don't worry // about returning a Promise; it's automatically wrapped // like Promise.resolve() does, so this chained-fetch() // Promise eventually gets the same result as the returned // recursive Promise. return autoRetry(url, times); } }); } return autoRetry(url, times); } maxRequest(`https://cdn.xgqfrms.xyz/json/badges.js`) .then(res => res.json()) .then(json => { console.log('json =', json); return json; }, err => { console.log('error =', err); throw new Error(err); }) .catch(err => console.log(`err =`, err)) .finally(() => { console.log('whatever close loading...'); }); En aras de la legibilidad, es posible que también desee cambiar los nombres de las variables, por lo que los times no se usan tanto para el ámbito externo como para el interno. También puede reformular esto como una función async mucho más corta que se repite.
·
Juan Pablo Isaza
Report
Answer question