rimuovere la duplicazione

Question

Ho una classe contiene 10 metodi che stanno facendo quasi le stesse cose a parte un evento chiave. Due esempi sono i seguenti:

Public String ATypeOperation(String pin, String amount){ 
    doSomething(); 
    doMoreStuff(); 
    requestBuilder.buildATypeRequest(pin, amount); 
    doAfterStuff(); 
} 
 


Public String BTypeOperation(String name, String sex, String age){ 
    doSomething(); 
    doMoreStuff(); 
    requestBuilder.buildBTypeRequest(name, sex, age); 
    doAfterStuff(); 
} 
 

Come si può vedere dai metodi di cui sopra, sono simili a parte chiamare diversi metodi forniti da requestBuilder. Anche il resto 8 è simile. C'è molto codice duplicato qui. Sento che c'è un modo migliore per implementarlo, ma non so come. Tutte le idee e i suggerimenti sono apprezzati.

Grazie, Sarah

Answer 1

usare qualcosa di simile RequestBuilder, che accetta tutti questi tipi di parametri:

public RequestBuilder { 
    // setters and getters for all properties 

    public Request build() { 
     doStuff(); 
     Request request = new Request(this); 
     doAfterStuff(); 
     return request; 
    } 
} 

e poi

new RequestBuilder().setAge(age).setName(name).build(); 

Answer 2

What’s the nearest substitute for a function pointer in Java?

Function Pointers in Java

Answer 3

interface RequestBuilder { 
    void doStuff(params); 
} 

public RequestBuilder getARequestBuilder() { 
    return new RequestBuilder() { 
    void doStuff(params) { 
     // impl.details 
    } 
    } 
}  

public RequestBuilder getBRequestBuilder() { 
    return new RequestBuilder() { 
    void doStuff(params) { 
     // impl.details 
    } 
    } 
}  

public String buildRequest(yourParams, RequestBuilder builder){ 
    doBefore(); 
    builder.doStuff(yourParams); 
    doAfter(); 
} 

penso che questo è chiamato il modello Strategy. Assomiglia molto allo schema Command ma poiché si incapsula un algoritmo sembra essere Strategia :)

Ciò che Bozho suggerisce è il modello Builder.

Ti consiglio di scorrere tra a list of patterns un po 'di tempo o di acquistare i modelli Primo capo. Lettura davvero divertente

Answer 4

Si potrebbe passare l'oggetto costruttore di un metodo buildRequest generico. Dal momento che non solo l'algoritmo ma anche gli argomenti variano, li metto nel costruttore. Non penso che questo è una bella soluzione, ma ho voluto mostrare un modello di comando qui: D (Extraneon mostrò come scindere params e di comando)

// call somewhere in the code: 
    Builder b = new BTypeBuilder(); 
    b.age = "20"; b.sex = "female"; b.name = "eve"; 
    String res = buildRequest(b); 

    Public String buildRequest(Builder builder) 
    { 
     doSomething(); 
     doMoreStuff(); 
     builder.build(); 
     doAfterStuff(); 
    } 

    // Command pattern 
    class BTypeBuilder implements Builder 
    { 
     String name, age, sex; 

     // Constructor here 

     void build() 
     { 
      // Do your stuff here 
     } 
    } 

    class ATypeBuilder implements Builder 
    { 
     String pin, amount; 

     // Constructor here 

     void build() 
     { 
      // Do your stuff here 
     } 
    } 

    public interface Builder 
    { 
     void build(); 
    } 

Answer 5

In aggiunta alle altre risposte, questo potrebbe essere utile anche per voi (se si vuole solo il plugin il vostro metodo, non usare i parametri per la 'prima' e 'dopo' metodi)

interface Function0<R> { 
    R apply(); 
} 

public void performOperation(Function0<Void> operation) { 
    doSomething(); 
    doBeforeStuff(); 
    operation.apply(); 
    doAfterStuff(); 

} 

allora si potrebbe usare in questo modo,

final RequestBuilder builder = new RequestBuilder(); 
    performOperation(new Function0<Void>() { 
     public Void apply() { 
      builder.buildATypeRequest("1234", "2445"); 
      return null; 
     } 
    }); 

    performOperation(new Function0<Void>() { 
     public Void apply() { 
      builder.buildBTypeRequest("1234", "2445", "1234"); 
      return null; 
     } 
    }); 

Answer 6

Invece di inviare una lunga param lista eter basta premere tutti i parametri in una mappa e inviare quella mappa come argomento.