fs.existsは非推奨なのでs.statまたは、fs.accessで、実行したい。

Question

前提・実現したいこと

javascript,node.jsの基本をかじって分かったつもりの初心者です。

### Node.jsで、serverを使って、指定フォルダ内のHTML、CSS、JAVASCRIPTをブラウザに表示させたい。

前提にcontentフォルダを作成
その中にhtml、css、JavaScriptファイルを作成。　

実現したいこと
fs.existsは非推奨なのでs.statまたは、fs.accessで、実行したい。

fs.existsやexistの部分をstatまたは、accessに変更しただけでは、実装中に以下のエラーメッセージが発生。

発生している問題・エラーメッセージ

ページが見つかりません！ 

該当のソースコード

server.js

var http = require('http');
var path = require('path');
var fs = require('fs');

var mimeTypes = {
	'.js': 'text/javascript',
	'.html': 'text/html',
	'.css': 'text/css'
};

http.createServer(function (request, response) {
	var lookup = path.basename(decodeURI(request.url)) || 'index.html',
	f = 'content/' + lookup;
	fs.exists(f, function (exists) {
		if (exists) {
			fs.readFile(f, function(err, data) {
				if (err) {
					response.writeHead(500);
					response.end('Server Error!');
					return;
				}
				var headers = {'Content-Type': mimeTypes[path.extname(f)] + ';charset=utf-8'};
				response.writeHead(200, headers);
				response.end(data);
			});
			return;
		}
		response.writeHead(404);
		response.end('ページがみつかりません！');
	});
}).listen(8080);

試したこと

参考サイトをまねして例外処理を適用してみたが、変化なし。

https://www.tech-tech.xyz/nodejs-writefile-mkdir-stat-access.html#fsstatfsaccess

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

node v10.15.0
npm 4.0.5

ソースコード元：

oreilly 「Nodeクックブック」

    レシピ 1.1　URL ルーティングを設定する

https://www.amazon.co.jp/dp/4873116066?_encoding=UTF8&isInIframe=0&n=465392&ref_=dp_proddesc_0&s=books&showDetailProductDesc=1#product-description_feature_div

Answer 1

こちらにありますように、fs.existsには競合状態（存在チェックの直後にファイルの作成・削除などが行われると想定どおりに進まない）問題があります。

Using fs.exists() to check for the existence of a file before calling fs.open(), fs.readFile() or fs.writeFile() is not recommended. Doing so introduces a race condition, since other processes may change the file's state between the two calls. Instead, user code should open/read/write the file directly and handle the error raised if the file does not exist. （Node.jsドキュメントより）

fs.existsでのチェックを行わずにfs.readFileを行って、errorの際にerror.codeを見てファイルが存在しない（ENOENT）かそれ以外のエラー化を確認するようにしましょう。

エラーコードの一覧

Comments

[maisumakun]

つまり、ファイルがあるかを事前にチェックすること「そのもの」がよくない、ということです。

[takusabuwp]

maisumakunさん、早速のアドバイスありがとうございます。

むむむ、If文内、全体を変えないといけなさそうですね。
まだまだ、勉強が必要なようです。
参考サイトや参考資料が、あれば教えていただけませんか
？英文は苦手なので、日本語がいいですが、、、

[takusabuwp]

so87さん、これは、回答ですね。
ヒントをもとに問いてないのに回答をもらってしまって恐縮です。
新しい、命令語が続々と、、、
勉強になります。

Answer 2

なぜfs.existsが非推奨になったのか知る必要があります。
そしてfs.exists 非推奨でググれば、すぐに知れます。
その内容はmaisumakunさんの回答のとおりなのですが、じゃあどうすれば？ということですよね。
初心者さんということで、足がかりになればと思い、ざっくりリファクタリングしておきます。（わからない時は右も左も知ってるはずなのにわからないですから＾＾）

const http = require('http')
const path = require('path')
const fs = require('fs')
const util = require('util')

const readFile = util.promisify(fs.readFile)

const mimeTypes = {
	'.js': 'text/javascript',
	'.html': 'text/html',
	'.css': 'text/css',
};

http.createServer(async function (request, response) {
	const lookup = path.basename(decodeURI(request.url)) || 'index.html'
	const filepath = 'content/' + lookup
	
	try {
		const data = await readFile(filepath)
		const headers = {'Content-Type': mimeTypes[path.extname(filepath)] + ';charset=utf-8'}
		response.writeHead(200, headers)
		response.end(data)
	} catch (err) {
		response.writeHead(404)
		response.end('ページがみつかりません！')
	}

}).listen(8080);

fs.existsした後にfs.readFileは意味ない、ダサいということです。
ファイルを読み込みたいのだから、素直にfs.readFileしちゃえってことです。
あと関係ないですが、node.js v10をご利用ということですので、varは極力避けてconstで書いていく（後々身にしみてvarの気持ち悪さを感じるようになります）。そしてなるべくPromise化を行い、async/await構文を使ってコーディングできるようになると、脱初心者です。
頑張ってください。