当使用显式构造函数调用时，如何避免“潜在的资源泄漏”警告？

Question

在我以前的代码中，为了方便起见，我经常使用以下构造函数习语：

MyClass( InputStream input ) {
  // do actual stuff
}
MyClass( String filename ) {
  this( new FileInputStream(filename) )
}

通常，我会对可自动关闭的FileInputStream使用try-with块，但这在这里是不可能的，因为调用必须始终是构造函数中的第一条语句。也就是说，以下内容不编译：

MyClass( InputStream input ) {
  // do actual stuff
}
MyClass( String filename ) {
  try( FileInputStream fis = new FileInputStream(filename) ) {
    this( fis );
  }
}

那么，什么才是正确的（也是优雅的？）如何使用这个习语而不冒潜在资源泄漏的风险？任何帮助都很感激。

Answer 1

即使您可以将构造函数调用包装在try-with-resources中，这真的有意义吗？构造函数很可能将inputstream存储在某种内部状态，因此在构造函数中关闭它将使它在随后用于该类的其他方法时无效。

更好的解决方案是使MyClass实现AutoCloseable并在close()方法中关闭输入流，从而使类的用户能够控制如何关闭传递给MyClass实例的输入流。