質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.84%

  • JavaScript

    14837questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • Node.js

    1682questions

    Node.jsとはGoogleのV8 JavaScriptエンジンを使用しているサーバーサイドのイベント駆動型プログラムです。

  • Express

    211questions

    ExpressはNode.jsのWebアプリケーションフレームワークです。 マルチページを構築するための機能セットおよびハイブリッドのWebアプリケーションを提供します。

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

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 411

jackfrost

score 12

 前提・実現したいこと

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
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+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);
  });
});

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/06/17 22:31

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

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

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

    キャンセル

  • 2018/06/17 22:54

    ■回答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 の出番になるわけです。

    キャンセル

  • 2018/06/17 23:36 編集

    yhgさま

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.84%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • JavaScript

    14837questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • Node.js

    1682questions

    Node.jsとはGoogleのV8 JavaScriptエンジンを使用しているサーバーサイドのイベント駆動型プログラムです。

  • Express

    211questions

    ExpressはNode.jsのWebアプリケーションフレームワークです。 マルチページを構築するための機能セットおよびハイブリッドのWebアプリケーションを提供します。