Runtime.getRuntime（）。exec（cmd）挂起

我正在执行一个命令，它返回一个文件的版本号; '文件名'。但是如果执行该命令时出现问题，则应用程序挂起。我能做些什么来避免这种情况？请在下面找到我的代码。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(); 
}

来源

2012-10-22 user1688404

看看[ProcessBuilder]（http://docs.oracle.com/javase/7/docs/api/java/lang/ProcessBuilder.html）。如果没有输出，'readLine'将永远阻塞。这是一个更简单的API来完成这种事情。 – RNJ

如果没有输出，'readLine'将永远阻塞。 – assylias

@assylias：我如何检查是否没有输出？ – 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(); 
}

来源

2012-10-22 09:52:55 Arham

这对我来说至少是完美的！谢谢 –

是的，代码块也救了我从无知嘿嘿:) – Akyo

非常有帮助谢谢！ – welterw8

这个代码是基于同样的想法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);

注意的是，如果需要输入这个方法，如果程序您呼叫挂起挂起，这会发生。

来源

2014-08-08 12:15:19 mikeyreilly

很好的运行'mvn verify'，我没有用'cmd/C'封装它 - 你知道封装第二个shell有什么好处吗？ –

我完全复制了OP的命令字符串，并以“cmd/C”开头。我认为在这种情况下包装它没有任何好处。 – mikeyreilly

Runtime.getRuntime（）。exec（cmd）挂起

回答

相关问题