値を返すときは未定義です

Question

私はノードプロジェクトを持っています。 ルートファイルがindex.jsで、ファイルがhelper.jsの場合、ここではヘルパー機能がいくつかあり、それをindex.jsにインポートしました。 helper.jsで関数を使用してデータを取得しようとしていますが、index.jsで呼び出すとundefinedが返されます。 しかし、helper.jsには、私が必要とするデータを示すconsole.logが表示されます。 この問題をどのように修正できますか？

index.jsファイルの内容：

const helper = require('./helper'); 

let data = helper.getData(); 
console.log(data); // undefined 

helper.jsファイルの内容：

const fs = require('fs'); 

module.exports = { 
    getData:() => { 
    fs.readFile('data.json', 'utf8', (err, data) => { 
     const allData = JSON.parse(data); 
     console.log(allData); // IS OK! 
     return allData; 
    }); 
    } 
} 

Answer 1

あなたはPromiseを使用することができます。

const fs = require('fs'); 

module.exports = { 
    getData:() => { 
    return new Promise(function(resolve, reject){ 
     fs.readFile('data.json', 'utf8', (err, data) => { 
      if(err){ 
       reject(err); 
      } else { 
       try { 
        resolve(JSON.parse(data)); 
       } catch(ex){ 
        reject(ex); 
       } 
      } 
     }); 
    }); 
    } 
} 

、その後：

helper.getData().then(function(data){ 
    console.log(data); 
}, function(err){ 
    // here something failed 
}); 

を

fs.readFileメソッドは非同期であり、結果として任意のデータを与えることはありませんcheck the documentation here

だから一つの選択肢は、私が行ったようにPromiseを使用するか、@Tatsuyukiイシの答えで提案されているように、あなたがcallback実装についてのドキュメントを確認することができますcallbackを使用することです。

Answer 2

fs.readFileは非同期関数なので、何も返されません。

返品に本当に必要な場合は、同期バージョンfs.readFileSyncを使用することができます。それ以外の場合は

- と、それを行うには良い方法は - つまり、あなたがallDataとresolve、その後することができますpromiseを返すgetData持っているだろう。

Answer 3

readFileは、コールバックを受け付ける非同期関数です。 2つのオプションがあります：

1。 getData()のパラメータとしてコールバックを取得します。

getData: (callback) => { 
    fs.readFile('data.json', 'utf8', (err, data) => { 
     const allData = JSON.parse(data); 
     console.log(allData); // IS OK! 
     callback(allData); 
    }); 
    } 

2。同期バージョンを使用してください。もちろん

getData:() => { 
    var data = fs.readFileSync('data.json', 'utf8'); 
    const allData = JSON.parse(data); 
    console.log(allData); // IS OK! 
    return allData;   
    } 

、あなたは物事をチェーンでより美しいですが、それはしばしばBluebirdのような依存関係で使用されるの約束を使用することができます。

Answer 4

allDataは、getData関数ではなく、コールバック関数から返されます。 getDataに明示的なreturnがないので、helper.getData()関数はundefinedを返し、この値は必要なものの代わりに出力されます。

@ sandの答えのように、Promiseを使用してデータを正しく返すことをお勧めします。