Imagina 10 funciones diferentes:
const a = async (uid, data) => { ... }; const b = async (uid, data) => { ... }; const c = async (uid, data) => { ... }; const d = async (uid, data) => { ... }; const e = async (uid, data) => { ... }; const f = async (uid, data) => { ... }; const g = async (uid, data) => { ... }; const h = async (uid, data) => { ... }; const i = async (uid, data) => { ... }; const j = async (uid, data) => { ... };Y 4 llaves diferentes:
"key1", "key2", "key3", "key4"He creado el siguiente objeto:
const OBJECT = { // constant, will always look like this key1: [a, b, c, d, e, f, h, i, j], key2: [a, b, c, d, f, i, j], key3: [a, b, c, d, e, f, g, h, i, j], key4: [a, b, c, d, e, f, h, i, j], }Y estoy a punto de usarlo de la siguiente manera:
const uid = uuid.v4(); const dynamicData = { // can contains any key (key1, key2, key3, key4) key1, key2, }; let selectedMethods = []; Object.keys(dynamicData).forEach((key) => { if (key in OBJECT) { selectedMethods.push(...OBJECT[key]); } }); // Uniquefy the list using lodash selectedMethods = _.uniq(selectedMethods); const promises = selectedMethods.map( (method) => method(uid, dynamicData) ); await Promise.all(promises);Este código no está realmente limpio... como puede ver, estoy repitiendo muchas cosas al inicializar el objeto de las matrices (parece una implementación sin formato).
¿Cómo puedo cambiar la forma en que lo estoy inicializando? Además, como puede ver, hay un patrón en los parámetros de los diferentes métodos (todos aceptan lo mismo). Entonces, si cree que es posible hacer que este código sea menos espagueti, ¿cómo puedo hacerlo?
Puede extraer partes repetidas en una variable auxiliar y distribuirlas en el literal de matriz respectivo:
const abcd = [a, b, c, d]; const ef = [e, f]; const ij = [i, j]; const OBJECT = { key1: [...abcd, ...ef, h, ...ij], key2: [...abcd, f, ...ij], key3: [...abcd, ...ef, g, h, ...ij], key4: [...abcd, ...ef, h, ...ij], } Sin embargo, solo debe hacer esto cuando tenga sentido semánticamente. Si ef no tiene más significado en su dominio comercial, es probable que sea una duplicación coincidente y, de hecho, más trabajo si es probable que los sitios de uso cambien de forma independiente. Además, la abstracción innecesaria hace que el código sea más difícil de entender.
Otro enfoque que podría ser viable es poner el prefijo y el sufijo comunes en una función auxiliar:
const value = middle => [a, b, c, d, ...middle, i, j]; const OBJECT = { key1: value([e, f, h]), key2: value([f]), key3: value([e, f, g, h]), key4: value([e, f, h]), } También puede eliminarlos del OBJECT por completo y simplemente agregarlos siempre a la lista de selectedMethods .