使用 processbuilder 从 Java 调用 cmd 命令

Question

我正在尝试使用 processbuilder 从 Java 调用 cmd 命令。但是我面临的问题很少。

1. 当我使用使用 Arrays.asList 构建的 List 时，应用程序在执行 br.readline() 后无限挂起（不是因为循环，而是在 readLine 方法中）。使用字符串数组给出输出。我检查了 grepcode，它看起来应该没有问题，因为当从 processbuilder 调用 start 方法时，它们都被转换回数组。 （链接：http://grepcode.com/file/repository.grepcode.com/java/root/jdk/openjdk/6-b27/java/lang/ProcessBuilder.java#ProcessBuilder）。我不确定是什么导致了这个异常。

2. 此程序有时无法提供正确的输出。如果我使用 process.destroy() 方法并获取 exitValue，它会显示 1。如果我注释掉 process.destroy() 方法，则会出现进程未退出的异常。我想这可能是线程竞争条件。但是，没有 process.waitFor() 的影响。它有时仍然会产生错误的输出。我们如何测试这些情况并找出问题的真正原因？

3. 我需要在 cmd 中调用一些命令。但是，数组元素充当前一个元素的参数。例如，如果我使用 cmd /C dir whoami 的元素制作数组。然后这会产生错误的输出，因为 whoami 充当 dir 的参数。向cmd独立提供命令的正确方法是什么？

以下是有相同问题的示例代码：

   import java.io.BufferedReader;
   import java.io.InputStreamReader;
   import java.util.Arrays;
   import java.util.List;
   public class Sample {
       public static void main(String[] args) throws Exception {
        //List<String> commandList = Arrays.asList("cmd.exe","dir");
        String[] commandList = {"cmd.exe", "/C", "dir"};
        //String[] commandList = {"cmd.exe", "/C", "dir", "whoami"};
        //String[] commandList = new String[] {"cmd.exe", "/C", "dir"};
        ProcessBuilder processBuilder = new ProcessBuilder(commandList);
        Process process = processBuilder.start();
        //process.waitFor();
        BufferedReader iReader = new BufferedReader(
                             new InputStreamReader(process.getInputStream()));
        String tempStr= "";
        StringBuffer buffer = new StringBuffer();
        while((tempStr = iReader.readLine())!=null){
            buffer.append(tempStr+System.lineSeparator());
        }
        System.out.println(buffer.toString());
        process.destroy();
        int exitValue = process.exitValue();
        System.out.println(exitValue);
    }
   }

Answer 1

1. Arrays.asList("cmd.exe","dir") 有两个 List 元素。 ("cmd.exe", "dir") 与 ("cmd.exe", "/C", "dir") 不同。它可能挂起，因为您忽略了包含描述错误的错误消息的错误输出。您可以使用 processBuilder.redirectError(ProcessBuilder.Redirect.INHERIT); 解决此问题（这通常是一个好主意，除非您打算以其他方式读取错误流）。
2. 删除 process.destroy() 并将 process.exitValue() 替换为 process.waitFor()。 在阅读所有输出之前不要调用 process.waitFor()； waitFor() 等待进程结束，您无法读取不再运行的进程的输出。
3. 您可以在命令之间使用&& 链接命令；例如：new ProcessBuilder("cmd.exe", "/C", "dir && date /t")。如果这不起作用，您可以尝试创建一个临时 .bat 文件并将其传递给 cmd /c。如果这不可接受，您可能必须为要运行的每个命令创建一个单独的进程。

顺便说一下，StringBuffer 已经过时了。请改用 StringBuilder，因为它没有不必要的同步开销。