结束在 InputStream.read() 循环中的线程

Question

我启动一个 cmd 应用程序，通过这个 SyncPipe Runnable 输出到 System.out：

public class SyncPipe implements Runnable {

    private final InputStream is;
    private final OutputStream os;

    public SyncPipe(InputStream is, OutputStream os) {
        this.is = is;
        this.os = os;
    }

    public void run() {
        try {
            final byte[] buffer = new byte[1024];
            for ( int length = 0; ( length = is.read(buffer) ) != -1; )
                os.write(buffer, 0, length);
            System.out.print("stopped");
        } catch ( Exception ex ) { 
            ex.printStackTrace(); 
        }
    }

}

我用cmd = "C:/bin/read.exe -f D:/test.jpg"启动RunIt

private class RunIt implements Runnable {

    public int p;
    public String cmd;

    public RunIt (int p, String cmd) {
        this.p = p;
        this.cmd = cmd;
    }

    public void run() {
        ProcessBuilder pb = new ProcessBuilder("cmd");
        try {
            process = pb.start();
            (new Thread(new SyncPipe(process.getErrorStream(), System.err))).start();
            (new Thread(new SyncPipe(process.getInputStream(), System.out))).start();
            OutputStream out = process.getOutputStream();
            out.write((cmd + "\r\n").getBytes());
            out.flush();
            out.close();

            try {
                process.waitFor();
            } catch ( InterruptedException e ) {
                e.printStackTrace();
            }

            println("Stopped using %d.", p);
        } catch ( IOException ex ) {
            ex.printStackTrace();
        }
    }

}

我现在的问题是：我怎样才能让(new Thread(new SyncPipe(process.getErrorStream(), System.err))) 死掉？给 SyncPipe 一个布尔变量 stop，在运行时将其设置为 true，然后通过 for ( int length = 0; ( length = is.read(buffer) ) != -1 && !stop; ) 检查它并没有成功。

非常感谢。

---

我最终完成了@Gray 建议的解决方法。现在可以使用了：

public void run() {
    try {
        final byte[] buffer = new byte[1024];
        do
            if ( is.available() > 0 ) {
                int length = is.read(buffer);
                if ( length != -1 )
                    os.write(buffer, 0, length);
                else
                    stop = true;
            }
        while ( !stop );
    } catch ( Exception ex ) { 
        ex.printStackTrace(); 
    }
}

Answer 1

当底层进程退出时，线程会读取 EOS 并退出。你不必自己做任何特别的事情。

编辑在我看来，从阅读您的 cmets 到其他答案，您的真正问题正在结束该过程。一旦发生这种情况，这些线程就会解开。你攻击的问题是错误的。

Answer 2

InputStream#read() 状态

此方法阻塞，直到输入数据可用，结束 检测到流，或者抛出异常。

所以当你进入 for 循环时

for ( int length = 0; ( length = is.read(buffer) ) != -1; )
    os.write(buffer, 0, length);

直到到达流的末尾，它才能退出，即。该过程停止或您自己关闭流。

如果SyncPipe 的全部目的是将内容传递给您的标准输出/错误流，您为什么要停止Thread 运行它？

Answer 3

我现在的问题：如何让 (new Thread(new SyncPipe(process.getErrorStream(), System.err))) 死掉？

我相信你将不得不从它下面关闭输入流。我怀疑它在读取时被阻塞了，并且没有设置 stop 变量（即使是正确的 volatile）将使读取线程解除阻塞。

您将需要执行以下操作：

 InputStream is = process.getInputStream();
 InputStream es = process.getErrorStream();
 ...
 is.close();
 es.close();

代码大致如下所示。我不确定您的waitFor() 电话是否正在返回。

 InputStream is = process.getInputStream();
 InputStream es = process.getErrorStream();
 (new Thread(new SyncPipe(es, System.err))).start();
 (new Thread(new SyncPipe(is, System.out))).start();
 try {
     OutputStream out = process.getOutputStream();
     out.write((cmd + "\r\n").getBytes());
     out.flush();
     out.close();
     try {
         process.waitFor();
     } catch ( InterruptedException e ) {
         e.printStackTrace();
     }
 } finally {
     is.close();
     es.close();
 }

另一个答案可能是在InputStream 上使用available() 方法，这样您就可以循环并检查您的stop 标志。看到这个答案：https://stackoverflow.com/a/1089079/179850