Refactoring di una classe astratta esistente e dei suoi parametri

Il punto di una fabbrica è quello di nascondere il fatto che è possibile ottenere oggetti da classi leggermente diverse. Pertanto, se alcuni di questi oggetti necessitano di determinati dati e altri no ...

• O non sono abbastanza simili per essere prodotti dalla stessa fabbrica. Quindi è necessario avere più di una fabbrica e prendere la decisione prima di quanto si sta facendo attualmente.

• O sono sono abbastanza simili a quelli che il cliente non dovrebbe preoccuparsi di quello che ricevono. Quindi non puoi aspettarti che il cliente si preoccupi se inviare o meno i dati extra. Ne consegue che devono passare sempre , ma a volte la fabbrica li ignorerà senza preoccuparsi del client con i dettagli di implementazione. Qualunque altra cosa avrebbe onorato il cliente con la stessa distinzione che la fabbrica avrebbe dovuto nascondere.

Lo gestisco utilizzando qualcosa come Builder che contiene varie fabbriche. Il Builder ha degli hash che contengono le fabbriche per creare vari tipi di oggetti, e guarderà nell'hash e recupererà il factory corretto in base a qualsiasi criterio.

Ad esempio, ho più tipi di domande, e tutte queste possono essere fatte da alcune implementazioni di un IQuestionFactory. La maggior parte delle implementazioni di IQuestionFactory sono sottoclassi di BaseQuestionFactory e ricevono XML che rappresenta una domanda e la analizzano in base alle regole della domanda, restituendo una sottoclasse di BaseQuestion.

Alcune domande richiedono più informazioni, ad esempio se si tratta di una domanda di compilazione di moduli, potrebbe esserci una sezione dell'XML che contiene le domande che definiscono i campi utilizzati nel modulo. Le fabbriche che fanno queste domande implementano IEnhancedQuestionFactory e il builder controllerà la Factory che recupera da QuestionFactoryRegistry e verifica se implementa questa interfaccia. Se lo fa, passa l'XML completo del file che conteneva le domande oltre al singolo XML che va al metodo createQuestion.

Per mostrarti quanto può essere flessibile, alcune delle implementazioni di IQuestionFactory sono semplicemente shell che contengono più fabbriche di domande, e possono guardare il codice XML in entrata e quindi chiamare una factory interna e restituire qualsiasi cosa faccia. Lo usiamo quando un file XML contiene più tipi di domande.

Tuttavia, sembra che tu abbia bisogno di qualcosa di più del modo in cui costruiamo Esercizi (che sono essenzialmente Controllori avvolti attorno a una serie di Domande), dove l'hash (ExerciseFactoryMap) è impostato per restituire una fabbrica predefinita per la maggior parte del tempo, ma nei casi in cui ha una specifica fabbrica registrata, la restituirà. E abbiamo una Factory specifica che creerà una sottoclasse Exercise con una proprietà aggiuntiva, e ha la capacità aggiuntiva di guardare l'XML e trovare le informazioni di cui ha bisogno per popolare quella proprietà.

Quindi, sono d'accordo con Killian Foth che il punto di a factory è di nascondere i dettagli dell'implementazione di come viene costruito l'oggetto, ma non c'è nulla che dice che non è possibile combinare più Factory in modo creativo (ad esempio, una Factory che contiene altre Fabbriche che fanno il vero lavoro).

• Il solito approccio in casi come questo (Command patternish) sta passando parametri aggiuntivi nel costruttore. Nel tuo caso, ciò significa aggiungere otherArgument a B durante la costruzione.

• Girare la logica potrebbe aprire alcune opzioni. Ad esempio spostando doStuff in AParams:

  for (String arg : listOfArgs) {
      AParams p = AParams.create(arg, otherArgument));
      for (A a : listOfA) {
          p.doStuff(a); // Let AParams to the lifting depending on type of A
      }
  }
  

• In alternativa puoi ingoiare il tuo orgoglio e controllare con instanceof nel ciclo e impostare otherArgument manualmente prima di chiamare doStuff .

Se quanto sopra non è possibile, non hai altra scelta che fare quello che stai facendo (allargando l'argomento dei parametri). Non analizzare troppo questa roba.