尝试与资源时调用超级构造函数

Question

当在构造函数中打开InputStream，然后将其传递给超级构造函数时，是否有任何使用try-with-resources的好方法？

基本上就是我想要做的是这样的：

public class A { 
    public A(InputStream stream) { 
     // Do something with the stream but don't close it since we didn't open it 
    } 
} 

public class B { 
    public B(File file) { 
     // We open the stream so we need to ensure it's properly closed 
     try (FileInputStream stream = new FileInputStream(file)) { 
      super(new FileInputStream(file)); 
     } 
    } 
} 

但是，当然，因为super必须在构造函数中的第一条语句，这是不允许的。有没有什么好的方法来实现这一目标？

Answer 1

考虑使用静态工厂方法，而不是直接使用构造函数。使至少B的构造私人，并创建一个方法，如

private B(InputStream is) { 
    super(is); 
    // Whatever else is needed 
} 

public static B newInstance(File file) { 
    B result; 
    try (FileInputStream stream = new FileInputStream(file)) { 
     result = new B(stream); 
    } 
    // Further processing 
    return result; 
} 

Answer 2

另一种方式去：

public class A { 
    protected A(){ 
     // so he can't be called from the outside, subclass ensure that init is done properly. 
    } 

    public A(InputStream stream) { 
     init(stream); 
    } 
    // not be able to call it from outside 
    protected final init(InputStream is){ 
     //here goes the code 
    } 
} 

public class B { 
    public B(File file) { 
     // We open the stream so we need to ensure it's properly closed 
     try (FileInputStream stream = new FileInputStream(file)) { 
      init(stream); 
     } 
    } 
} 

我在这里张贴这是一个可能的答案，但是我在这里consdering：

1. 可以更新的代码
2. 你正在移动的构造函数的代码到init方法，由于受保护的空参数构造函数，只有子类必须正确处理对init的调用。有些人可能会认为设计不太好。我的观点是尽快对子类进行子类化，只有当你使用它时，你必须更多地了解它。

Answer 3

很遗憾，我手边没有编译器来测试，但是你可以不这样做，如下所示。

public class B { 
    private static InputStream file2stream(File f){ 
     // We open the stream so we need to ensure it's properly closed 
     try (FileInputStream stream = new FileInputStream(file)) { 
      return stream; 
     }catch(/*what you need to catch*/){ 
      //cleanup 
      // possibly throw runtime exception 
     } 
    } 
    public B(File file) { 
     super(file2stream(file)) 
    } 
}