Tengo varias clases diferentes con el mismo código duplicado. Aparte de la identidad diferente de cada this , estos códigos duplicados son aparentemente idénticos.
He aquí un ejemplo simplificado:
class classA { ... doSomething() { if (!this.id || !this.data || !this.isAllowed()) { return null; } return someImportedFunc(this.data, this.metadata, this.doSomethingElse()); } } class classB { ... doSomething() { if (!this.id || !this.data || !this.isAllowed()) { return null; } return someImportedFunc(this.data, this.metadata, this.doSomethingElse()); } } // Many other classes like this Pensé en mover el método doSomething a otra clase y luego importarlo en todas estas clases. Pero eso daría como resultado un método muy desordenado que necesita muchos, muchos argumentos para dar cuenta de todos los diferentes this 's. Se vería como:
class exportedClass { function doSomething(id, data, metadata, isAllowed, doSomethingElse) { if (!id || !data || !isAllowed()) { return null; } return someImportedFunc(data, metadata, doSomethingElse()); } } class classA { ... methodThatUsesDoSomething() { const result = exportedClass.doSomething( this.id, this.data, this.metadata, this.isAllowed, this.doSomethingElse); ... } }Excepto con 15 argumentos en lugar de 5, ya que este es un ejemplo simplificado.
¿Cómo puedo evitar el código duplicado en este tipo de situación?
Suponiendo que desee una solución sin herencia, mi sugerencia sería pasar "esto" como argumento, ya que los prototipos se pueden manipular como parámetros. Eso le permitiría mantener los parámetros al mínimo sin perder datos.