entfernen Duplikation

Question

Ich habe eine Klasse enthält 10 Methoden, die fast die gleichen Dinge tun, abgesehen von einem Schlüsselereignis. Zwei Beispiele sind unten angegeben:

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(); 
} 
 

Wie Sie aus den oben genannten Methoden sehen können, sind sie auseinander ähnlich der Aufruf von verschiedenen Methoden zur Verfügung gestellt von requestBuilder. Die restlichen 8 sind auch ähnlich. Es gibt viel doppelten Code hier. Ich denke, es gibt einen besseren Weg, dies zu implementieren, aber ich weiß nicht wie. Irgendwelche Ideen und Vorschläge werden geschätzt.

Danke, Sarah

Answer 1

Verwendung so etwas wie RequestBuilder, die alle diese Arten von Parametern akzeptiert:

public RequestBuilder { 
    // setters and getters for all properties 

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

und dann

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(); 
} 

Ich denke, das das Strategy Muster genannt wird. Es sieht sehr ähnlich dem Command Muster aus, aber weil Sie einen Algorithmus kapseln, scheint es Strategie zu sein :)

Was Bozho vorschlagen, ist das Builder Muster.

Ich empfehle Ihnen, durch a list of patterns einige Zeit zu durchsuchen, oder kaufen Head First Patterns. Wirklich Spaß beim Lesen.

Answer 4

Sie konnten die Erbauer Objekt zu einem allgemeinen buildRequest-Methode übergeben. Da nicht nur der Algorithmus, sondern auch die Argumente variieren, lege ich sie in den Builder. Ich denke nicht, das ist eine schöne Lösung, aber ich wollte hier ein Befehlsmuster zeigen: D (Extraneon zeigte, wie params und Befehl zu entkoppeln)

// 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

Neben anderen Antworten, könnte dies auch für Sie nützlich sein (If Sie wollen einfach nur Ihre Methode Plugin, nicht Ihre Parameter für die ‚vor‘ und ‚nach‘ Methoden)

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

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

} 

dann könnte man es so mit verwenden,

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

Anstatt eine lange param senden In der Liste drücken Sie einfach alle Parameter in einer Karte und senden diese Karte als Argument.