在Java微服务中检测外部调用进程的完成状态

我把你这段代码整理成了更规范的格式，同时标注了几个关键细节和优化方向：

Process p = Runtime.getRuntime().exec("/bin/sh /var/task/bin/iTMSTransporter " + commandLine);
BufferedReader stdError = new BufferedReader(new InputStreamReader(p.getErrorStream()));

logger.log("StdErr:");
String s;
while ((s = stdError.readLine()) != null && p.isAlive()) {
    System.out.println("stderr: " + s);
    // 过滤掉带DBG-X的调试日志，仅收集有效错误信息
    if (s.indexOf("DBG-X") == -1) {
        result.stdErr += s + "\n";
    }
}
// 后续的日志记录逻辑
this.log...

关键逻辑说明 #

• 借助Runtime.getRuntime().exec()触发外部shell脚本调用，目标程序是iTMSTransporter
• 专门监听进程的错误输出流（ErrorStream），并通过p.isAlive()确保在进程存活期间持续读取
• 实时打印所有stderr内容到控制台，同时过滤掉调试性质的DBG-X日志，只将有效错误信息存入result.stdErr

几个优化建议 #

1. 提升字符串拼接性能
   循环里用+=拼接字符串会频繁生成新对象，建议改用StringBuilder：

StringBuilder errCollector = new StringBuilder();
// ...
if (s.indexOf("DBG-X") == -1) {
    errCollector.append(s).append("\n");
}
// 最后统一赋值
result.stdErr = errCollector.toString();

2. 改用ProcessBuilder更安全
   如果commandLine包含空格、特殊字符，直接拼接命令会出问题，ProcessBuilder支持参数拆分，还能灵活配置环境、工作目录：

ProcessBuilder pb = new ProcessBuilder("/bin/sh", "/var/task/bin/iTMSTransporter", commandLine);
pb.redirectErrorStream(false); // 保持stdout和stderr分离
Process p = pb.start();

3. 避免资源泄漏
   记得用try-with-resources或者finally块关闭流和销毁进程：

try (BufferedReader stdError = new BufferedReader(new InputStreamReader(p.getErrorStream()))) {
    // 你的读取逻辑
} finally {
    if (p != null && p.isAlive()) {
        p.destroy();
    }
}

内容的提问来源于stack exchange，提问作者ken