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

时间:2013-10-09 19:11:53

标签: java multithreading cmd destroy

我启动了一个通过此SyncPipe Runnable输出到System.out的cmd应用程序:

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(); 
    }
}

3 个答案:

答案 0 :(得分:1)

  

我现在的问题:我如何制作(新的Thread(新的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();
 }

另一个答案可能是在available()上使用InputStream方法,因此您可以循环并检查stop标记。请参阅此答案:https://stackoverflow.com/a/1089079/179850

答案 1 :(得分:1)

InputStream#read()

  

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

所以当你进入for循环

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

在到达流的末尾之前它将无法退出,即。进程停止或您自己关闭流。

如果SyncPipe的整点是将内容传递给标准输出/错误流,为什么要阻止Thread运行它?

答案 2 :(得分:1)

线程将读取EOS并在基础进程退出时退出。你自己不需要做任何特别的事情。

编辑在我看来,通过阅读您的评论到其他答案,您的真正问题正在结束这一过程。一旦发生这些线程,这些线程就会立即脱落。你正在攻击错误的问题。