¿Devolver una Promesa creada manualmente sin resolver desde una función asíncrona se considera un antipatrón?
Como las funciones asíncronas devuelven promesas de forma predeterminada, es un poco extraño crear una new Promise y devolverla desde la misma función async .
Tengo dos métodos diferentes, en el primero tengo dudas de si estoy haciendo o no el new Promise . En el otro, definitivamente estoy seguro de que es un antipatrón.
Primera función:
export default async (uri) => { const blob = await new Promise((resolve, reject) => { const xhr = new XMLHttpRequest(); xhr.onload = function () { resolve(xhr.response); }; xhr.onerror = function (e) { reject(new TypeError("Network request failed")); }; xhr.responseType = "blob"; xhr.open("GET", uri, true); xhr.send(null); }); return blob; };Segunda función:
export async function uploadImageToStorage( imageUri, storageFolder = "images", stateObserver = undefined ) { const blob = await uriToBlob(imageUri); const imageId = blob._data.blobId; const storageRef = storage.ref(storageFolder).child(imageId); return new Promise((resolve, reject) => { storageRef.put(blob).on( "state_changed", stateObserver, function error(err) { blob.close(); reject(err); }, function complete() { blob.close(); resolve(imageId); } ); }); }¿Se considera un antipatrón la implementación del primer método?
En la segunda función, como storageRef.put(blob).on() no devuelve una Promesa, mi API está basada en devolución de llamada, necesito envolverla dentro de una Promesa, resolviendo en una devolución de llamada complete() . Si lo refactorizo a:
export async function uploadImageToStorage( imageUri, storageFolder = "images", stateObserver = undefined ) { const blob = await uriToBlob(imageUri); const imageId = blob._data.blobId; const storageRef = storage.ref(storageFolder).child(imageId); await new Promise((resolve, reject) => { storageRef.put(blob).on( "state_changed", stateObserver, function error(err) { blob.close(); reject(err); }, function complete() { blob.close(); resolve(imageId); } ); }); return imageId; }La nueva implementación no sería considerada como un antipatrón ¿verdad?
No hay nada malo con esto en cualquier caso. Debe devolver una Promise , pero la API que está utilizando se basa en la devolución de llamada, por lo que debe envolverla en una Promise creada manualmente en algún momento.
Sin embargo, no tiene sentido await esa Promise dentro de su función async si todo lo que va a hacer es devolver el valor al que se resolvió. En ese caso, puede devolver directamente la Promise no resuelta.
Después de hacer eso, si ya no usa await en la función, también puede eliminar la palabra clave async . Pero lo dejaría como una señal para el lector casual de que el valor de retorno será una Promise .