node.js + express + MySQLで undefined is not function と怒られた

Question

前提・実現したいこと

現在、expressのappを使って
当月と前月以前のデータと動画の表示の処理をし、
またHTML側の月の遷移により
post と get で同じSQLを動かしたいため、
app.useで'/*'にて共通処理としました。

ところがリダイレクトにより app.use('/*')が
複数回処理されていることがわかりました。

この事から、SQLを無駄に動かさないように
app.useの共通処理をやめ、
get post ともに1度だけSQLを実行すべく
functionでモジュール化しましたが、
知識が乏しいので、undefined is not a function と
怒られました。

どのようにモジュール化すればよいか、ご教示いただけると嬉しいです。

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

const express = require('express');
const app = express()
const mysql = require('mysql2/promise');
// ～省略～
const ddate = new Date();// 今日を取得
var yymm = ddate.toFormat("YYMM"); // 2101形式でセット

function getColumnData(num, result, column){
// ここで連想配列から配列にセットし直してreturn
}
// ここにfunction getSQLData(sql)を設置

app.set('ejs', ejs.renderFile);
const pool = mysql.createPool({
	host: 'localhost',
	user: 'root',
	password: 'test',
	database: 'test',
});
app.use('/', express.static(path.join(__dirname, 'stream')));// Movie
app.use('/', express.static(path.join(__dirname, 'image')));// mp4ファイルのサムネイル用
app.post('/views/', async(request, response, next) => {
	yymm = ${request.body.test}`; // postで年月情報取得
	next();
});
app.get('/', ,async( request, response, next ) => {
	moveImage();// 取得写真を別フォルダに移動
	makeMovie();// 写真から動画生成 exec(`ffmpeg ...
	console.log("app.get");
	next();
});
app.use('/*',async( request, response ) => {
	try{
		// SQLによるDBデータ取得
		sql = `select * from test where yymm = '${yymm}'`;
		var [data] = await pool.query(Sql);
		// var data = await getSQLData(sql); とセット
		// ここでは1個ですが、実際には5個のSQLを動かしていて
		// getColumnDataモジュールで配列に置き換えた処理を4個動かしています
		response.render('test.ejs',{
			data: data,
			item: item,
			...
		});
	}
	catch(err){
		console.log(`catch: ${err}`);
	};
	console.log("Successful");
});

https.createServer(options, app).listen(port,() => {
	console.log('server start');
});

// console

node test.js
app.get
server start
Successful
Successful

やった事

function getSQLData(sql){
	var [data] = pool.query(sql);
	return data;
}

また、こうした方がいいんじゃない？という上記ソースを全く変えてしまう
お答えでも、全然かまいません。
すいませんが、よろしくお願い致します。

解決後の備忘録

今回の変更箇所を記載しておきます。
function から const 変数名 に変更。

const getSQLData = async()=>{
	sql = `select * from test where yymm = '${yymm}'`;
	// グローバル変数にセット
	var [Gl_Data] = await pool.query(Sql);
	...
};

変更後がこちら↓

const express = require('express');
const app = express()
const mysql = require('mysql2/promise');
// グローバル宣言
var tmpData;
var gl_Data;
var gl_Item;
...
// ～省略～
const ddate = new Date();// 今日を取得
var yymm = ddate.toFormat("YYMM"); // 2101形式でセット

function getColumnData(num, result, column){
// ここで連想配列から配列にセットし直してreturn
}
// SQLの実行関数
const getSQLData = async()=>{
	// SQLによるDBデータ取得
	sql = `select * from test where yymm = '${yymm}'`;
	var [Gl_Data] = await pool.query(Sql);
	...
};

app.set('ejs', ejs.renderFile);
const pool = mysql.createPool({
	host: 'localhost',
	user: 'root',
	password: 'test',
	database: 'test',
});
app.use('/', express.static(path.join(__dirname, 'stream')));// Movie
app.use('/', express.static(path.join(__dirname, 'image')));// mp4ファイルのサムネイル用
app.post('/views/', async(request, response, next) => {
	yymm = ${request.body.test}`; // postで年月情報取得
	// SQLによるDBデータ取得
	await getSQLData();
	next();
});
app.get('/', async( request, response, next ) => {
	moveImage();// 取得写真を別フォルダに移動
	makeMovie();// 写真から動画生成 exec(`ffmpeg ...
	console.log("app.get");

	// SQLによるDBデータ取得
	await getSQLData();
	next();
});
app.use('/*',async( request, response, next ) => {
	try{
		// getColumnDataモジュールで配列に置き換えた処理を4個動かしています
		var data = getColumnData(hogeNum, gl_Data, 'data');// 配列に置き換え
		var item = getColumnData(hogeNum, gl_Item, 'item');// 配列に置き換え
		response.render('test.ejs',{
			data: data,
			item: item,
			...
		});
	}
	.catch(err){
		// html側にエラー状況を通告
		response.status(404).send('404 error, Page Not found');
		// コンソール側にエラーを表示
		console.log(`catch: ${err}`);
	};
	console.log("Successful");
});

https.createServer(options, app).listen(port,() => {
	console.log('server start');
});

うん、スッキリしました。

Answer 1

undefined is not function

まずこのエラーそのものに関して回答します。
JSに於いて「値()」という風に値の後に丸括弧()を付けると、
値を関数と認識して実行しようとします。

()は関数実行の証と考えてください。

しかし、関数実行が出来るのは関数だけ。
undefined is not functionと叱られるのはすなわち、
「undefinedの値に向かって関数実行しとるやろ！関数やないから実行できへんで」という意味です。

ざっと見た感じ、それが起こりそうな箇所は見つけられませんでした。
行番号とかを見ながら関数実行している箇所を探しましょう。

var [data] = await pool.query(Sql);

ん？JSの変数は大文字・小文字を区別します。
まぁ、コードの貼り付け時のミスということでスルーしますが、

実践でやると大ハマリする原因です。
エディタにLinter等のツールを入れて自動的にチェックしてくれるような仕組みを導入しましょう。

エディタがVSCode等のモダンなエディタなら
導入方法を記載した記事がゴロゴロヒットするので簡単に入れられると思います。

app.use('/*',async( request, response ) => {

この追加した関数はミドルウェアですが、
第三引数のnextを定義して引き継いでいないということは、
最終的にこの第三引数のnextは実行していないってことですよね？

app.useだろうが、app.getだろうが
一度該当する箇所に入ればルーティングは「終わった」と判断する作りになり、
以降の実行が打ち切られる事になります。

参考サイト: ミドルウェアの使用 - Express

それを防ぐ為の第三引数nextだと言うことをちゃんと理解しておいてください。
nextが実行されて初めて「そうなんだ、これはあくまでミドルウェアでメインの処理がまだあるねんな？」と判断してくれます。

console.log("Successful");

なーにがsuccessfulやねん。
SQLの実行に失敗してもエラーを握りつぶしているのでここ通過しちゃいますよ。

内部でasync関数版try-catchの仕組みを利用してSQL文を発行していますが、
失敗した時はresponseで直接ステータスコードの400番なりを返して終了させるような作りにしましょう。

Comments

[HYDESA]

miyabi-sunさん、いつもお世話になっています。
ご丁寧なご回答、ありがとうございました。

`app.use` のところはnext()を入れていったら、上記で正常に稼働しましたので、そのままにしておりました。このご教示で、nextを追記しておきました。

`console.log("Successful");` はブラウザ側の話だったんですね。
お陰様で理解できまして、
`response.status(404).send('404 error, Page Not found');`
を`catch`の中に組み込みました。
確かにエラー時はタブ内でグルグル回っていたので、それが無くなりスッキリしています。

あとはfunction内でのquery()が出来ないので、もう少し調べています。
ただ、`catch`で`response.state(404)`を付けたおかげなのか、呼び出しは1回で済むようになりました。

nodeの経験値ゼロですので、呼び出し1回になったのが、かなり？？？状態ですが、何とか理解したいと思います。

[HYDESA]

その後、調査をしておりましたが
async function getSQLData(sql){
var [data] = await pool.execute(sql);// const pool = mysql.createPool{}で宣言してます
return data;
}
上記に変えてdataの中身を見るべく、console.log('%j',pool.execute(sql)); としたところ
{} だけしか表示されませんでした(query()も同様でした)。

どうやら関数化すると、execute()もしくはquery()は実行されていない？ようです。
もう少し探ってみます。

[HYDESA]

関数化のところ、自己解決しました。

function にこだわっていましたが、
下記のサイトがとても参考になりました。
https://teratail.com/questions/118317

async function getSQLData(sql){...
これを
const getSQLData = async()=>{...
実行はtry{}の中で
await getSQLData();
としたら、無事にできるようになりました。

なお、ベストアンサーはいろいろ教えてくださったmiyabi-sunさんにしました。
miyabi-sunさん、ありがとうございましたm(_ _)m