¿Hay alguna alternativa a cambiar en un método de fábrica?

Question

Veo esto bastante y me preguntaba si había alguna manera de refactorizar esto para evitar el cambio masivo. Este es un método en una fábrica: RoomControllerFactory, instanciando una ubicación de juego según su tipo. He aquí un ejemplo de un cambio en el método de fábrica:

  switch (location.getType()) 
      { 
       case Location.ROOMONE: 
        return new RoomOneController(location, data, view);   

       case Location.ROOMTWO: 
        return new RoomTwoController(location, data, view); 

       case Location.ROOMTHREE: 
        return new RoomThreeController(location, data, view); 

Answer 1

Respuesta corta, cambio de fábricas, eso es lo que hacen: el objetivo de este estilo de fábrica es centralizar la lógica de construcción en un solo lugar en lugar de tenerlo en la base del código.

Siempre que no esté utilizando una fábrica estática y codificando una interfaz IRoomControllerFactory, obtendrá todos los beneficios habituales de OOP al cambiarlo en tiempo de ejecución/para probarlo. Después de todo, dice 'Yo RoomControllerFactory, dame ¡una habitación para este identificador mágico!

Como una respuesta adicional a su pregunta, es posible que desee preguntarse por qué necesita tantas instancias concretas de RoomController? Tal vez al favorecer la composición sobre la herencia, ¿podría refactorizar las cosas para usar un Constructor?

Answer 2

En vista de que está utilizando un truco para proporcionar la funcionalidad de enumeración - ¿Por qué no añadir un método para su enumeración:

public static const ROOMONE : LocationType = new LocationType("locationone", 
    function(...) : RoomController { 
     return new RoomOneController 
    } 
); 

(excusa los errores tontos - actionscript no es mi primera lengua!)

en java que haría similar con:

public enum LocationType { 
    ROOMONE { 
     @Override 
     public RoomController getRoomController() { 
      return new RoomOneController(); 
     } 
    }; 
    public abstract RoomController getRoomController(); 
}