メソッドでの戻り値が適応されない

Question

let find = function(){
  let process = spawn("find",[DIR_COPY,"-name","*.pde"]);
   process.stdout.on('data', (data)=>{
    console.log("data :" + data);
    return data ;
});

let open = function(path){
  let process = spawn("open",[path]);
  process.on("exit",(code,signal)=>{
    console.log("file_open");
  });
}

let postProcess = function() {
let data = find() ;
  open(data);
}

上記のコードなのですが、findメソッドで得たdataという変数を
openメソッドの引数として扱いたいのですが、
dataの中身を見てみると、何も入っていません。

findメソッドの行なっている操作は、
「.pde」の名前がついたファイルのパスを所得しています。

openメソッドの行なっている操作は、
findで得たパスを開くといった操作なのですが、
どうもファイルが開かれません。

コンソールで戻り値で得たdataの
中身を見てみると何も入っていないようでした。

なぜ、戻り値が適応されないのでしょうか？
よろしくお願いいたします。

Comments

[takasima20]

find() が無いみたいなエラーは出てませんか?

[mote]

回答ありがとうございます。エラーは出ていない状況です。

Answer 1

spawnとともに用いるコールバック(onで登録する関数)の動作を勘違いしておられる気がします。

find関数は何もreturnしてませんので
let data = find()
としてもdataの中身はundefinedにしかなりません。

も少しfindでやっていることを補足すると

(1)子供プロセスでfindコマンドを実行する
(2)子供プロセスが何かをstdoutへ出力した際に呼び出されるコールバック関数を登録
(3)上記を行ったらサブプロセスからの出力や終了は待たずにすぐにreturn

となります。(2)で登録したコールバック関数から何かをreturnしてもそれはfind関数の戻り値とはなりません。

おそらく下記のようなことをしたかったのではないかと想像します。なお、spawnの用い方の変更以外に以下の点を変更しています。

• find echoの連携にした

単に動作の確認のしやすさのためですが、
find . -name '*.pde' -exec echo {} \;
と同様のことを-execをnode側で行うような動作にしてみてます。

• 関数定義

let name = function (...) {}ではなく単にfuntion name(...) {}とした

• let

letにすべきところがないと感じたので全部constにしました。

t.js

const { spawn } = require('child_process')

function find(callback) {
  const proc = spawn('find', ['.', '-name', '*.pde'])
  const pid = proc.pid
  console.log(`find: spawn pid=${pid}`)
  proc.stdout.on('data', data => {
    console.log(`data: ${data}`)
    for (const s of ('' + data).split('\n').filter(s => s != '')) {
      callback(s)
    }
  })
}

function echo(path) {
  const proc = spawn('echo', [path])
  const pid = proc.pid
  console.log(`echo: spawn pid=${pid}`)
  proc.stdout.on('data', data => {
    console.log(`echo: ${data}`)
  })
  proc.on('exit', (code, signal) => {
    console.log(`echo: ${pid} done code=${code}, signal=${signal}`)
  })
}

function doProcessPipeline() {
  find(echo)
}

doProcessPipeline()

結果例：

$ ls
a.pde  b.pde  c.pde  t.js
$ node t
find: spawn pid=4751
data: ./b.pde
./c.pde
./a.pde

echo: spawn pid=4752
echo: spawn pid=4753
echo: spawn pid=4754
echo: 4752 done code=0, signal=null
echo: 4753 done code=0, signal=null
echo: 4754 done code=0, signal=null
echo: ./b.pde

echo: ./c.pde

echo: ./a.pde

$

Comments

[mote]

とても分かりやすいです！まさにfindではそのような事がしたかったです。
もう少し疑問点があるのですが、差し支えなければ答えていただきたいです。
今回、echoではなく、子供プロセスでのpdeファイルの実行をしたいのですが、
その場合はopenを使う必要は無いのでしょうか!?

[KSwordOfHaste]

えーと、自分は拡張子pdeがどんなファイルなのか知りません。またopenコマンドって何かわかりません。ひょっとしてMacintoshユーザさんですか？調べてみるとMacintoshにはopenコマンドってのがあるらしいですが・・・

[mote]

そうです！
ありがとうございました。const s of ('' + data).split('\n').filter(s => s != '' "　ここがあまり理解出来ていないので、教えていただきたいです...

[KSwordOfHaste]

一つ一ついうと

('' + data)

dataは文字列ではなくBuffer型なのでまずは('' + data)で文字列型に変換します。

('' + data).split('\n')

一つの文字列を改行文字を区切りに複数の文字列へ分解しそれを要素とする配列に変換します。

('' + data).split('\n').filter(s => s != '')

配列要素から「空文字列」を取り除いた新たな配列を生成します。

こういうのは実際に小さなコードを書いて動かしながら動作を把握するのがよいでしょう。

for (s of 'a//c'.split('/')) {
　console.log(s)
}
for (s of 'a//c'.split('/').filter(s => s != '')) {
　console.log(s)
}

という具合に書いて動かしてみればよいのです。

[mote]

ありがとうございます。とても助かりました！

Answer 2

2点問題があります。

• dataはfind内のコールバックにローカルな変数なので、外部からは使えません。
• .onで受けているように、processの動作は非同期実行なので、同期的な返り値として受けることはできません。

ただファイルを探すためだけなら、node-globに同期モードがありますので、そちらのほうが便利かもしれません。