配列の中身が空になってしまいます

Question

現在、JavaScriptでアクセスしてきたらCSVファイルの中身を検索して
検索結果をJsonで返すWebアプリを作成しています。

CSVファイルの中身の検索はうまくいっていて、
ループの中では配列（lineArray1）の中身がちゃんと入っているのですが
ループから出たところで配列（lineArray1）の中身が空になってしまいます。

配列の中身がちゃんと入る実装方法を教えていただけないでしょうか？

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

const app = express();


app.get('/',(req,res) =>{

var startDateTime = req.query.StartDateTime;
var endDateTime = req.query.EndDateTime;
var lineArray1 = new Array();


var startDateTime2 = new Date(startDateTime.substr(0,4) , Number(startDateTime.substr(4,2))-1, Number(startDateTime.substr(6,2)), 0, 0,0 );
var endDateTime2 = new Date( endDateTime.substr(0,4) , Number(endDateTime.substr(4,2))-1, Number(endDateTime.substr(6,2)), 0, 0,0 );
var todayDateTime = new Date();
todayDateTime.setHours(0);
todayDateTime.setMinutes(0);
todayDateTime.setSeconds(0);

for(startDateTime2;startDateTime2<=endDateTime2;startDateTime2.setDate( startDateTime2.getDate() + 1 ))
{

var csvFileName='test.csv';


if(todayDateTime == startDateTime2)
{
csvFileName='test.csv'
}
else
{
csvFileName=GetFileName(startDateTime2)
}



fs.readFile(csvFileName, 'utf8', function (err, data) {
  var dataArray = data.split(/\r?\n/);
  
  for(var line of dataArray)
  {
  	var lineArray2 = line.split(',');
  	var lineDateTime=lineArray2[0];

  	if(CheckOverTime(startDateTime,lineDateTime))
  	{
  		if(CheckUnderTime(endDateTime,lineDateTime))
  		{

		  	lineArray1.push(lineArray2);
  		
  		}  	
  	}

  }
});

}
  
  var json_text = JSON.stringify(lineArray1);

  res.header('Content-Type', 'application/json; charset=utf-8');
  res.set({ 'Access-Control-Allow-Origin': '*' });
  res.send(json_text);
console.log(json_text);
});



app.listen('3000',() =>{
  console.log('Application started');
});

Answer 1

🍉
fs.readFile(csvFileName, 'utf8', function (err, data) {
  🍈
});
🍏

こういうコードがあった場合、コールバックは非同期に実行されるため、実行順は🍉🍏🍈です。
なので、🍏をコールバックの中に移動しましょう。

🍉
fs.readFile(csvFileName, 'utf8', function (err, data) {
  🍈
  🍏
});

Comments

[tamina0624]

ありがとうございます。
何故動かなかったのかはっきりと分かりました！

[int32_t]

> すべてのファイルの中身を読んでから値を返したい

簡単に readFileSync() を使ってもいいですね。
https://nodejs.org/api/fs.html#fsreadlinksyncpath-options

または、Promise を使って await しましょう。
https://nodejs.org/api/fs.html#fspromisesreadfilepath-options

[tamina0624]

ありがとうございます。
readFileSyncを使ってやりたいことができました