对象与静态方法设计

Question

如下所示，有两种简单的方法可以制作流式复印机（介绍Apache Commons或类似工具的栏）。我应该去哪一个？为什么？

public class StreamCopier {
private int bufferSize;

public StreamCopier() {
    this(4096);
}

public StreamCopier(int bufferSize) {
    this.bufferSize = bufferSize;
}

public long copy(InputStream in , OutputStream out ) throws IOException{
    byte[] buffer = new byte[bufferSize];
    int bytesRead;
    long totalBytes = 0;
    while((bytesRead= in.read(buffer)) != -1) {
        out.write(buffer,0,bytesRead);
        totalBytes += bytesRead;
    }

    return totalBytes;
}
}

与

 public class StreamCopier {

 public static long copy(InputStream in , OutputStream out)
     throws IOException {
     return this.copy(in,out,4096);
 }

 public static long copy(InputStream in , OutputStream out,int bufferSize)
     throws IOException {
     byte[] buffer = new byte[bufferSize];
     int bytesRead;
     long totalBytes = 0;
     while ((bytesRead= in.read(buffer)) != -1) {
         out.write(buffer,0,bytesRead);
         totalBytes += bytesRead;
     }

     return totalBytes;
}
}

Answer 1

我将使用非静态（实例）版本，并将其作为显式依赖项（使用setter）提供给消费者：

• 然后将其模拟用于单元测试是微不足道的，因此，消费者的测试不会与实现耦合。
• 换出功能很简单，例如：使用子类；
• 与依赖项注入系统配合良好。

编辑

为了回应（“有用！”）的评论“这如何帮助嘲笑？”，这是可能的工作方式：

class ThingThatUsesStreamCopier {

    // our copier instance. set in constructor, but might equally use
    // a setter for this:
    private StreamCopier copier;

    public ThingThatUsesStreamCopier(StreamCopier copier) {
        this.copier = copier;
    }

    public void makeCopy(Stream in, Stream out) {
        // probably something a little less trivial...
        copier.copy(in, out);
    }
}

当我进行测试时ThingThatUsesStreamCopier，我可以创建a的模拟对象版本StreamCopier并ThingThatUsesStreamCopier使用此模拟实例化该对象。

这样，我就可以完全控制模拟的行为，因此我的测试与的任何实际实现都没有关联StreamCopier。我仅测试消费者，而不测试消费者加消耗者。