別ファイルの関数呼び出し

Question

前提・実現したいこと

node.js初心者です。
local:8080にアクセスがあったら、ソースコードのあるディレクトリ内の.jsか.htmlのファイル一覧を表示する。
という機能を実装したいのですが、node.jsで関数の結果がundifinedになってしまうようです。
原因がわからず困っております。

※タスク
1.カレントディレクトリの末尾が.jsか.htmlのファイルを取得する関数をモジュールとして別ファイルに作成
2.function.js内の関数をmain.jsから引数を渡して実行
3.結果をmain.jsのdataに入れる
4.Vueでテンプレに表示

該当のソースコード

const Vue = require('vue')
const server = require('express')()
const renderer = require('vue-server-renderer').createRenderer()
const list = require("./function.js");

server.get('*', (req, res) => {
  var test = list("./");
  const app = new Vue({
    data: {
    url: req.url,
    items: test
    },
    template: require('fs').readFileSync('./template.html', 'utf-8')
})

  renderer.renderToString(app, (err, html) => {
    res.send(html)
  })
});
server.listen(8080)

const fs = require('fs');
console.log('module load!');

//ファイル一覧取得関数
var readdir = function (dir) {
  fs.readdir(dir, function (err, files) {
    if (err) {
      throw err;
    }
    files.filter( function( value ) {
      //特定の拡張子を含むものだけ抽出
      return value.match( /.js$|.html$/g );
    })
  });
}
module.exports = readdir;

試したこと

1.テンプレにはdataの値が入っていれば表示される(ベタ書きでテストしました)
2.function.js自体は動いている(module.exportせずにfunctins.jsだけで実行しテストしました)
3.main.js側のconsole.logで関数の結果がundefinedと表示される。

補足情報（FW/ツールのバージョンなど）

bash-3.2$ node -v
v9.5.0

Answer 1

readdir 関数が値を何も返してないですね。
せっかくなのでこのまま非同期にファイル一覧取得できるようにしてみましょう。

const util = require('util');
const fs = require('fs');

// fs.readdir を Promise インスタンスを返すようにラッピング
// Node.js v10 の場合は require('fs').promises.readdir があるのでそちらを使えば良い
const readdir = util.promisify(fs.readdir);

// 正規表現のパターンを作るときはパースに少し時間かかる処理なので、
// なるべくループの外側で作るのが良い
const FILTER_REGEX = /.(js|html)$/;

module.exports = function (dir) {
  // 読み込んだファイル一覧をフィルタリングして返す
  return readdir(dir).then(files => files.filter(value => FILTER_REGEX.test(value)));
};

使うときは、下記のように async/await を使うとネストが一段深くならないで済みます。

server.get('*', async (req, res) => {
  const test = await list("./");

  const app = new Vue({
    data: {
      url: req.url,
      items: test
    },
    template: require('fs').readFileSync('./template.html', 'utf-8')
  });

  renderer.renderToString(app, (err, html) => {
    res.send(html);
  });
});

Comments

[jackfrost]

yhgさま
ご回答ありがとうございます。
教えていただいたコードが動くことは確認できたので、目的は達成できそうなのですが、当方jsとnode.js自体が初心者のためコードの理解に時間がかかりそうです。
特に以下の部分が理解できないため、解説していただけると助かります。

■質問1
```
value => FILTER_REGEX.test(value)
```
上記の部分でフィルタリングしているのだろうと推測できるのですが、なぜtestでフィルタリングできるのか理解できません。なぜbooleanが返らないのか。。。

■質問２
最初に私が書いたコードのどこに不備があったのか分からないと、また同じ間違いをすると思うので、間違いを指摘して欲しいです。
教えていただいたコードを参考に作り替えたりしてテストしましたが、やはりうまくいきませんでした。
なぜ結果が返らないのか教えていただけるとありがたいです。

[yhg]

■回答1
その部分は普通に boolean 返っていますが、その外側の filter メソッドでフィルタリングしてます。
files => files.filter(value => FILTER_REGEX.test(value)) を ES5 記法で書くと

function (files) {
return files.filter(function (value) {
return FILTER_REGEX.test(value);
});
}

のようになります。

■回答2

var readdir = function (dir) {
fs.readdir(dir, function (err, files) {
if (err) {
throw err;
}
files.filter( function( value ) {
//特定の拡張子を含むものだけ抽出
return value.match( /.js$|.html$/g );
})
});
}

この部分ですが、一番外側の readdir 関数は何も値を返していませんので、readdir 関数を呼び出しても何も返ってきません。

また、

var readdir = function (dir) {
let results;

fs.readdir(dir, function (err, files) {
if (err) {
throw err;
}
results = files.filter( function( value ) {
//特定の拡張子を含むものだけ抽出
return value.match( /.js$|.html$/g );
});
});

return results;
}

のように変数に入れればいけるだろみたいな発想になってしまう人もいると思いますが、fs.readdir 関数は非同期関数なので、return results; に到達した時点では変数 results に何も入っていないので空の値を返してしまいます。というわけで、今回のように Promise の出番になるわけです。

[jackfrost]

yhgさま

分かりやすい解説をありがとうございます。
同期・非同期はまったく意識していなかったため、大変勉強になりました。
fs.readdirSyncにしたらPromiseと同じ結果が得られました。
ご指摘ありがとうございました。