web-dev-qa-db-ja.com

Runtime.getRuntime（）。exec（cmd）がハングしています

ファイルのリビジョン番号を返すコマンドを実行しています。 'ファイル名'。ただし、コマンドの実行に問題がある場合、アプリケーションはハングアップします。その状態を回避するにはどうすればよいですか？私のコードの下を見つけてください。

String cmd= "cmd /C si viewhistory --fields=revision --project="+fileName; 
Process p = Runtime.getRuntime().exec(cmd) ;  
BufferedReader in = new BufferedReader(new InputStreamReader(p.getInputStream()));  
String line = null; 
while ((line = in.readLine()) != null) {  
System.out.println(line);  
} 

} catch (Exception e) {  
e.printStackTrace();  
 }
javaprocessruntime.execprocessbuilder
13
user1688404

問題は、InputStreamのみを読み取っており、ErrorStreamを読み取っていないことだと思います。また、両方のストリームが並行して読み取られるように注意する必要があります。現在、出力ストリームからパイプされたデータがOSバッファーをいっぱいにした場合、リーダーにバッファーを空にする機会を与えるために、execコマンドが自動的に中断されることがあります。ただし、プログラムは出力の処理を待機しています。したがって、ハングが発生します。

次のように、入力ストリームとエラーストリームの両方を処理する別のクラスを作成できます。

public class ReadStream implements Runnable {
    String name;
    InputStream is;
    Thread thread;      
    public ReadStream(String name, InputStream is) {
        this.name = name;
        this.is = is;
    }       
    public void start () {
        thread = new Thread (this);
        thread.start ();
    }       
    public void run () {
        try {
            InputStreamReader isr = new InputStreamReader (is);
            BufferedReader br = new BufferedReader (isr);   
            while (true) {
                String s = br.readLine ();
                if (s == null) break;
                System.out.println ("[" + name + "] " + s);
            }
            is.close ();    
        } catch (Exception ex) {
            System.out.println ("Problem reading stream " + name + "... :" + ex);
            ex.printStackTrace ();
        }
    }
}

使い方は以下の通りです、

String cmd= "cmd /C si viewhistory --fields=revision --project="+fileName; 
Process p = Runtime.getRuntime().exec(cmd) ;  
s1 = new ReadStream("stdin", p.getInputStream ());
s2 = new ReadStream("stderr", p.getErrorStream ());
s1.start ();
s2.start ();
p.waitFor();        
} catch (Exception e) {  
e.printStackTrace();  
} finally {
    if(p != null)
        p.destroy();
}
31
Arham

このコードは、Arhamの答えと同じ考えに基づいていますが、Java 8並列ストリームを使用して実装されているため、もう少し簡潔になっています。

public static String getOutputFromProgram(String program) throws IOException {
    Process proc = Runtime.getRuntime().exec(program);
    return Stream.of(proc.getErrorStream(), proc.getInputStream()).parallel().map((InputStream isForOutput) -> {
        StringBuilder output = new StringBuilder();
        try (BufferedReader br = new BufferedReader(new InputStreamReader(isForOutput))) {
            String line;
            while ((line = br.readLine()) != null) {
                output.append(line);
                output.append("\n");
            }
        } catch (IOException e) {
            throw new RuntimeException(e);
        }
        return output;
    }).collect(Collectors.joining());
}

このようなメソッドを呼び出すことができます

getOutputFromProgram("cmd /C si viewhistory --fields=revision --project="+fileName);

呼び出しているプログラムがハングすると、このメソッドがハングすることに注意してください。これは、入力が必要な場合に発生します。

4
mikeyreilly