删除重复

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列表只是将所有参数推送到地图中并将该地图作为参数发送。