31

我试图将此函数的输出作为字符串返回,但它一直以未定义的形式返回。我哪里错了?

function run(cmd){
    var spawn = require('child_process').spawn;
    var command = spawn(cmd);
    var result = '';
    command.stdout.on('data', function(data) {
            result += data.toString();
    });
    command.on('close', function(code) {
            return result;
    });
}
console.log(run('ls'));
4

4 回答 4

49

您的函数在command.on语句后立即返回。return事件回调中的语句close无处返回。return属于事件回调,而不是run().

console.logcall 而不是return result.

一般来说,您应该编写如下内容:

function run(cmd, callback) {
    var spawn = require('child_process').spawn;
    var command = spawn(cmd);
    var result = '';
    command.stdout.on('data', function(data) {
         result += data.toString();
    });
    command.on('close', function(code) {
        return callback(result);
    });
}

run("ls", function(result) { console.log(result) });
于 2013-03-20T04:53:31.087 回答
5
var spawn = require('child_process').spawn,
    command  = spawn('ls', ['/tmp/']);
command.stdout.pipe(process.stdout);

以下链接与您的问题完全相同。

于 2013-03-20T04:52:10.320 回答
3

你总是可以将你的函数包装在一个 Promise 中并返回它。我发现比@fuwaneko 的回调解决方案更有效

function run(cmd) {
    return new Promise((resolve, reject) => {
        var spawn = require('child_process').spawn;
        var command = spawn(cmd)
        var result = ''
        command.stdout.on('data', function(data) {
             result += data.toString()
        })
        command.on('close', function(code) {
            resolve(result)
        })
        command.on('error', function(err) { reject(err) })
    })
}
于 2018-10-02T15:25:42.733 回答
1

干净的方式正在使用async/await,所以试试这样:

const spawn = require('child_process').spawnSync;
 
try {
    const child = spawn(cmd)
    return { stdout: child.stdout.toString(), stderr: child.stderr.toString() }
} catch (error) {
    console.log(error);
    return error
}
于 2021-10-10T06:33:10.323 回答