Tengo una function que procesa una gran lista de documentos JSON y los inserts en MongoDB .
Quiero log lo que está haciendo esta función en varias ubicaciones y luego enviar la carga útil a S3. Sin embargo, esto está condicionado por el llamado.
function doStuffWithJSON(options={}) { const apiRes = getJSONFromAPI(); if (options.logAndChuck) { console.log('we got results successfully') } //50 lines of code that are unconditionally being executed if (options.logAndChuck) { console.log('we did stuff with the result') } //Another 50 lines of code that are unconditionally being executed if (options.logAndChuck) { sendJsonToS3(apiRes) } } El patrón repetitivo aquí es que tengo la misma instrucción if en múltiples ubicaciones diferentes de la función. No estoy seguro de si hay alguna forma de evitar esto solo dada la naturaleza.
Me preguntaba si había algún patrón para limpiar cosas como esta.
¡Gracias!
Para poner en código las sugerencias en los comentarios de la pregunta:
function doStuffWithJSON(options={}) { const apiRes = getJSONFromAPI(); logAndChuck('we got results successfully') //50 lines of code that are unconditionally being executed logAndChuck('we did stuff with the result') //Another 50 lines of code that are unconditionally being executed if (options.logAndChuck) { sendJsonToS3(apiRes) } } function logAndChuck(options, message) { if (options.logAndChuck) { console.log(message) } }Realmente depende: en mi humilde opinión, no es un problema tener una declaración if simple varias veces. Sin embargo, si es más complejo, puede crear una función para esto, en el ámbito de la función local o superior tomando una función de devolución de llamada:
function doStuffWithJSON(options={}) { const exec = (callbackFn) => { if (options.logAndChuck) { callbackFn(); } }; const apiRes = getJSONFromAPI(); exec(() => console.log('we got results successfully')); //50 lines of code that are unconditionally being executed exec(() => console.log('we did stuff with the result')); //Another 50 lines of code that are unconditionally being executed exec(() => sendJsonToS3(apiRes)); }