【问题标题】:Close Scanner without closing System.in关闭扫描仪而不关闭 System.in
【发布时间】:2023-04-05 12:01:01
【问题描述】:

我正在尝试将我的应用程序的一个大型且经常使用的部分重新分解为单独的方法,以使其更易于维护。

其中一些方法要求用户输入并进行输入验证,所以我使用了 Scanner 和 System.in 但是当我关闭我的扫描仪时,我也会关闭 System.in

所以我的问题是,我只能通过使用 CloseShieldInputStream 屏蔽 System.in 来防止它被关闭,还是应该开始将 Scanner 传递给方法?

【问题讨论】:

  • 请贴一些代码...
  • 您真的需要关闭扫描仪吗?我建议让垃圾收集器处理它 - 如果不关闭底层对象,就无法关闭它。
  • 我的方法声明了一个 Scanner,读取并返回 nextLine(),关闭 Scanner,下次运行时让我头疼。如果我不关闭它,Eclipse 会纠缠我潜在的资源泄漏,忽略它是否安全?
  • Eclipse 在这种情况下是完全错误的——或者更确切地说,它过于谨慎了。关闭 Scanner 除了关闭底层 InputStream 之外没有任何用处,所以如果您不想关闭 System.in,请不要在 Scanner 上调用 close()

标签: java java.util.scanner system.in


【解决方案1】:

只需使用自定义 FilterInputStream 代替 System.in:

new FilterInputStream(System.in) {
    @Override
    public void close() throws IOException {
        //don't close System.in! 
    }
}

【讨论】:

  • 这几乎就是 commons 的 CloseShieldInputStream 所做的,如果您最终担任这个职位,您可能想再看看您的软件设计。除非您使用第三方解决方案,否则不需要这样的解决方案。 (适用于此处的所有答案)
【解决方案2】:

您可以通过实现自定义装饰器来忽略关闭。

public class UnClosableDecorator extends InputStream {

    private final InputStream inputStream;

    public UnClosableDecorator(InputStream inputStream) {
        this.inputStream = inputStream;
    }

    @Override
    public int read() throws IOException {
        return inputStream.read();
    }

    @Override
    public int read(byte[] b) throws IOException {
        return inputStream.read(b);
    }

    @Override
    public int read(byte[] b, int off, int len) throws IOException {
        return inputStream.read(b, off, len);
    }

    @Override
    public long skip(long n) throws IOException {
        return inputStream.skip(n);
    }

    @Override
    public int available() throws IOException {
        return inputStream.available();
    }

    @Override
    public synchronized void mark(int readlimit) {
        inputStream.mark(readlimit);
    }

    @Override
    public synchronized void reset() throws IOException {
        inputStream.reset();
    }

    @Override
    public boolean markSupported() {
        return inputStream.markSupported();
    }

    @Override
    public void close() throws IOException {
        //do nothing
    }
}

并在 main 中使用它

public static void main(String[] args) throws Exception {
        System.setIn(new UnClosableDecorator(System.in));
}

【讨论】:

  • 这和写我自己的 CloseShieldInputStream 版本不一样吗?
  • 我不太清楚你的意思,但如果你需要在关闭时对输入流执行一些操作,请在 UnClosableDecorator.close() 方法中执行此操作
  • CloseShieldInputStream 是一个代理流,可以防止底层输入流被关闭。 commons.apache.org/io/apidocs/org/apache/commons/io/input/…
  • 其实我的实现和CloseShieldInputStream不太一样。关闭时的 CloseShieldInputStream 将用读取时始终返回 -1 的虚拟对象替换输入流(这意味着 InputStream 已关闭),而我在关闭时的实现什么也不做。我不确定哪种解决方案更适合您。
  • 我怀疑我应该开始绕过扫描仪,这似乎是最终最不痛苦的方式。
【解决方案3】:

你可以让它不关闭它,只需将持有变量设置为 null

【讨论】:

  • 由于 Eclipse 发出了关于扫描仪永远不会关闭的警告,这是否可以忽略?
  • 我认为这是一种可改进的编程风格。程序员最终可能会留下确实必须关闭的开放资源。
猜你喜欢
  • 1970-01-01
  • 2014-06-29
  • 2015-03-19
  • 1970-01-01
  • 1970-01-01
  • 2017-07-22
  • 1970-01-01
  • 2018-08-27
  • 1970-01-01
相关资源
最近更新 更多