Hay una función:
toggleSelect(key: string, object: RegistryLayerItemGeneric, selected: boolean) { if (selected) { let objects = this.state.selectedRegistryObjects.get(key); if (objects && object.ObjectId in objects) { delete objects[object.ObjectId]; } this.state.selectedRegistryObjects.set(key, { ...objects, }); return; } const objects = { ...(this.state.selectedRegistryObjects.get(key) || {}), ...{ [object.ObjectId]: object }, }; this.state.selectedRegistryObjects.set(key, objects); }Esta función elimina el elemento en el mapa y lo agrega si no existe. ¿Cómo puedo mejorarlo? Creo que esta función se complegó para la comprensión.
Hay algunas cosas que haría aquí que deberían ayudar.
Dividir la función en múltiples funciones para que cada función solo haga una tarea . He comenzado a hacer esto a continuación.
Cambie el nombre de los parámetros para que sean más descriptivos de lo que contienen, es decir, en lugar de "seleccionado: bool", diga que lo seleccionado es "mapElementExists: bool".
Cambie el nombre de la función para que sea más descriptiva, de modo que el lector sepa qué se activa aquí
He comenzado a hacer esto a continuación.
toggleMapElements(key: string, object: RegistryLayerItemGeneric, mapElementExists: boolean) { if (mapElementExists) { removeMapElement(key: string, object: RegistryLayerItemGeneric); } else { addMapElement(key: string, object: RegistryLayerItemGeneric) } } removeMapElement(key: string, object: RegistryLayerItemGeneric) { let objects = this.state.selectedRegistryObjects.get(key); if (objects && object.ObjectId in objects) { delete objects[object.ObjectId]; } this.state.selectedRegistryObjects.set(key, { ...objects, }); return; } addMapElement(key: string, object: RegistryLayerItemGeneric) { const objects = { ...(this.state.selectedRegistryObjects.get(key) || {}), ...{ [object.ObjectId]: object }, }; this.state.selectedRegistryObjects.set(key, objects); }Esa es definitivamente una función complicada.
Aquí hay una versión más simple.
toggleSelect(key: string, object: RegistryLayerItemGeneric, selected: boolean) { const objects = (this.state.selectedRegistryObjects.get(key) || {}); if (selected) { delete objects[object.ObjectId]; } else { objects[object.ObjectId] = object; } this.state.selectedRegistryObjects.set(key, {...objects}); }No tengo un entorno de mecanografiado frente a mí, por lo que no puedo confirmar si pasa todo el ruido de mecanografiado, pero esto funcionará desde una perspectiva de javascript.