链式方法设计

Question

我写了一个简单的类来演示链式方法设计：

public class Cal {

    private Cal(){}

    private boolean isCheckArguments = false;

    public static Cal useAbs() {
        return new Cal(){   
          @Override int check(int i) {
                return Math.abs(i);
            }};
    }

    public static Cal useNormal() {
        return new Cal();
    }   

    public Cal checkArguments() {
        isCheckArguments =true;
        return this;
    }

     int check(int i){ return i;}

     public int plus(int i, int j) {    
        if(isCheckArguments && i<j){
            throw new IllegalArgumentException("i<j!");
        }
        return check(i+j);
    }
}

所以客户端代码可以是：

Cal cal = Cal.useAbs().checkArguments();
int sum = cal.plus(100,2);//IllegalArgumentException occurs

Cal cal2 = Cal.useAbs();
int sum2 = cal.plus(-100,2);//98

Cal cal3 = Cal.useNormal();
int sum3 = cal.plus(-100,2);//-98

我的问题是：这是一个合理的设计吗？比较：int plus(int a, int b, boolean useAbs, boolean checkArguments)。谢谢！

Answer 1

听起来你想要一个流畅的接口来构建一个服务类。番石榴做类似的事情。你会做这样的事情：

public interface Cal {
   int plus(int a, int b);
}

public class CalBuilder {
    class InternalCal implements Cal {
       boolean useAbs;
       boolean checkArgs;
       public int plus(int a, int b) {
          if(checkArgs) {
             // blah, blah blah
          }
          if(useAbs) {
             // doodle bug, doodle darn
          }
          return a+b; // whatevs
       }
    }
    boolean absSet=false;
    InternalCal holder=new InternalCal();
    public CalBuilder useNormal() {
        if(absSet) { throw new IllegalArgumentException(); } // already called
        holder.useAbs=false;    
        absSet=true;
        return this;
    }

    public CalBuilder useAbs() {
        if(absSet) { throw new IllegalArgumentException(); } // already called
        holder.useAbs=false;    
        absSet=true;
        return this;
    }

    public CalBuilder checkArguments() {
       if(holder.checkArgs) { throw new IllegalArgumentException(); }
       holder.checkArgs=true;
       return this;
    }

    public Cal build() {
       return holder;
    }
}

用法如下所示：

Cal cal=new CalBuilder().useAbs().checkArguments().build();
int sum=cal.plus(1,2);

Answer 2

这叫做fluent interface，对我来说看起来很合理。

我可能建议更改课程名称：Calc 比 Cal 更明显是计算器（可能是日历而不是计算器）。

Answer 3

最好的设计是简单的设计。你正在做的是混淆代码。它比“plus(a, b, useAbs, checkArguments)”更难阅读和修复错误。此外，在 useNormal 中，您返回的对象与“new Cal ()”相同，因为您覆盖了 check(int) 方法并使用 super.check(int) 返回父级的实现

Answer 4

恕我直言，这种“链接”方法在 Java 中并不常用，除非是构建器或本质上是构建器的东西。

您在这里描述的实际上是一个计算器构建器。

您可能应该有一个 CalculatorBuilder 类，您可以对其进行实例化、设置多项内容（例如 useAbs、checkArguments 等），并最终调用“build”。 Build 将返回一个计算器，除了初始化的状态外，它对它是如何构建的一无所知。

另外，我个人不喜欢混合了构建器风格的逻辑（例如，“useAbs”）和影响底层对象状态的东西（例如 checkArguments）的设计。我会说选一个。要么生成一个默认计算器并稍后设置所有内容，要么让一个构建器设置所有内容，然后创建其功能和行为无法更改的实例。