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

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

新規登録して質問してみよう
ただいま回答率
85.48%
Node.js

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

JavaScript

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

Q&A

解決済

1回答

1109閲覧

fs.readdirを飛ばして先の処理が実行されてしまう

amie8mie

総合スコア14

Node.js

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

JavaScript

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

0グッド

0クリップ

投稿2019/04/24 00:40

こちら
node.js ファイル一覧を取得する
を参考にNode.jsでファイル検索をし、見つかったら何かしらの処理をしたいのですが

JavaScript

1const fs = require('fs') 2 3const exists = ( reg ) => { 4 5 fs.readdir('.', function(err, files){ 6 7 const fileList = files.filter(function(file){ 8 9 return fs.statSync(file).isFile() && reg.test(file) 10 }) 11 12 if ( 0 < fileList.length ) { 13 14 console.log('あった') 15 return true 16 } 17 else { 18 19 console.log('なかった') 20 return false 21 } 22 }) 23} 24 25//テキストファイルがあるかどうか調べる 26if( !exists( /.*.txt$/ ) ) { 27 28 console.log('なかったので終わり') 29} 30else { 31 32 console.log('あったときの処理') 33} 34

これを実行してみると、たとえ目的のファイルがあったとしてもコンソールには

なかったので終わり あった

と表示されてしまいます
おそらく同期・非同期に関する問題が発生しているのだとは思いますが…
どうすればよいでしょうか
async/awaitとかでしょうか?

気になる質問をクリップする

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

回答:
変数[exists]に格納された関数で実行される処理が「fs.readdir」の非同期処理の実行開始を行うだけで戻り値が無いため。

改善案:
1.async/awaitを利用した処理として実装する(モダン)
2.[exists]に格納された関数内で判定条件の出力を行うように実装する(レガシー)

下記3つの前提条件がそろうのであれば、上記の改善案1.を提示したいところです。
ただ、前提条件のが揃いませんと理解に時間がかかると思われますので改善案2.のみ回答いたします
改善案1.の前提条件:
a.利用環境Nodeのバージョンが7.6以降である
b.非同期処理を理解し、処理を自分で書くことができる
c.Promiseを理解し、処理を自分で書くことができる

javascript

1// 改善案2.V1 2// 関数内で処理を完結させる 3const fs = require('fs') 4const exists = ( reg ) => { 5 fs.readdir('.', function(err, files){ 6 const fileList = files.filter(function(file){ 7 return fs.statSync(file).isFile() && reg.test(file) 8 }) 9 if ( 0 < fileList.length ) { 10 console.log('あったときの処理') 11 } else { 12 console.log('なかったので終わり') 13 } 14 }) 15} 16// 実行 17exists( /.*.txt$/ );

上記ですと決まった処理しかできませんので,コールバックを利用するバージョン

javascript

1// 改善案2.V2 2// exists 処理後に行いたい処理を渡す 3const fs = require('fs') 4 5// exists 処理後に行いたい処理を定義した関数 6const existsCallbackFunc = (isFileExists)=>{ 7 if(isFileExists) { 8 console.log('なかったので終わり') 9 }else { 10 console.log('あったときの処理') 11 } 12} 13const exists = ( reg , callback ) => { 14 fs.readdir('.', function(err, files){ 15 const fileList = files.filter(function(file){ 16 return fs.statSync(file).isFile() && reg.test(file) 17 }) 18 const isFileExists = 0 < fileList.length 19 callback(isFileExists); 20 }) 21} 22// 実行後に行いたい処理を引数に渡してから実行 23exists( /.*.txt$/ , existsCallbackFunc );

なお、関数自体に改善する余地が多々ありますが、今回の質問とは逸脱し過ぎる為、割愛いたします。

不明点などがあれば、レスください。

投稿2019/04/24 02:03

編集2019/04/24 02:04
kanimaru

総合スコア1013

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問