刪除重複

Question

我有一個類包含10個方法，除了一個關鍵事件外，它們的功能幾乎相同。兩個例子如下：

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

正如你從上面的方法看，他們是從調用由requestBuilder提供不同的方法除了類似。其餘8個也類似。這裏有很多重複的代碼。我覺得有更好的方法來實現這一點，但不知道如何。任何意見和建議表示讚賞。

感謝， 薩拉

Answer 1

使用類似RequestBuilder，接受所有這些類型的參數：

public RequestBuilder { 
    // setters and getters for all properties 

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

然後

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

我想，這就是所謂的Strategy模式。它看起來很像Command模式，但是因爲你封裝算法似乎是策略:)

什麼Bozho建議是Builder模式。

我建議你一段時間瀏覽a list of patterns，或者購買Head First Patterns。真的很有趣的閱讀。

Answer 4

您可以在生成器對象傳遞給通用buildRequest方法。由於不僅算法而且參數也各不相同，所以我把它們放到了構建器中。我不認爲這就是一個很好的解決方案，但我想在這裏展示的命令模式：d（Extraneon展示瞭如何分離PARAMS和命令）

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

除了其他的答案，這也可能對您有用（如你只想插件你的方法，不使用你的參數爲「前」和「後」的方法）

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

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

} 

，那麼你可以使用它像這樣，

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

，而不是發送一個長PARAM eter列表只是將所有參數推送到地圖中並將該地圖作爲參數發送。