トップに関する質問 node.js + express + MySQLで undefined is not function と怒られた

編集履歴

質問編集履歴

変更メソッド追加っす

2021/01/29 07:44

投稿

HYDESA

スコア8

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -196,8 +196,28 @@
 今回の変更箇所を記載しておきます。
+`function` から `const 変数名` に変更。
 ```express
+const getSQLData = async()=>{
+	sql = `select * from test where yymm = '${yymm}'`;
+	// グローバル変数にセット
+	var [Gl_Data] = await pool.query(Sql);
+	...
+};
+```
+変更後がこちら↓
+```express
 const express = require('express');
 const app = express()

修正どす

2021/01/29 07:44

投稿

HYDESA

スコア8

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -46,6 +46,8 @@
 const app = express()
+const mysql = require('mysql2/promise');
 // ～省略～
 const ddate = new Date();// 今日を取得
@@ -200,6 +202,8 @@
 const app = express()
+const mysql = require('mysql2/promise');
 // グローバル宣言
 var tmpData;

修正どす

2021/01/29 07:38

投稿

HYDESA

スコア8

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -310,7 +310,7 @@
 	}
-	catch(err){
+	.catch(err){
 		// html側にエラー状況を通告

修正どす

2021/01/29 07:33

投稿

HYDESA

スコア8

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -300,7 +300,7 @@
 		response.render('test.ejs',{
-			data: data
+			data: data,
 			item: item,

修正どす

2021/01/29 07:31

投稿

HYDESA

スコア8

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -120,6 +120,186 @@
 		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');
+});
+```
+```javascript
+// console
+node test.js
+app.get
+server start
+Successful
+Successful
+```
+### やった事
+```express
+function getSQLData(sql){
+	var [data] = pool.query(sql);
+	return data;
+}
+```
+また、こうした方がいいんじゃない？という上記ソースを全く変えてしまう
+お答えでも、全然かまいません。
+すいませんが、よろしくお願い致します。
+### 解決後の備忘録
+今回の変更箇所を記載しておきます。
+```express
+const express = require('express');
+const app = express()
+// グローバル宣言
+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,
@@ -132,6 +312,12 @@
 	catch(err){
+		// html側にエラー状況を通告
+		response.status(404).send('404 error, Page Not found');
+		// コンソール側にエラーを表示
 		console.log(`catch: ${err}`);
 	};
@@ -150,190 +336,4 @@
 ```
-```javascript
-// console
-node test.js
-app.get
-server start
-Successful
-Successful
-```
-### やった事
-```express
-function getSQLData(sql){
-	var [data] = pool.query(sql);
-	return data;
-}
-```
-また、こうした方がいいんじゃない？という上記ソースを全く変えてしまう
-お答えでも、全然かまいません。
-すいませんが、よろしくお願い致します。
-### 解決後の備忘録
-今回の変更箇所を記載しておきます。
-```express
-const express = require('express');
-const app = express()
-// グローバル宣言
-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');
-});
-```
 うん、スッキリしました。

修正どす

2021/01/29 07:31

投稿

HYDESA

スコア8

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -294,15 +294,15 @@
 		// getColumnDataモジュールで配列に置き換えた処理を4個動かしています
-		gl_Data = getColumnData(...);// 配列に置き換え
+		var data = getColumnData(hogeNum, gl_Data, 'data');// 配列に置き換え
-		gl_Item = getColumnData(...);// 配列に置き換え
+		var item = getColumnData(hogeNum, gl_Item, 'item');// 配列に置き換え
 		response.render('test.ejs',{
-			data: gl_Data
+			data: data
-			item: gl_Item,
+			item: item,
 			...

修正どす

2021/01/29 07:30

投稿

HYDESA

スコア8

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -264,13 +264,13 @@
 	// SQLによるDBデータ取得
-	await getSQLData();// await欲しいっす
+	await getSQLData();
 	next();
 });
-app.get('/', ,async( request, response, next ) => {
+app.get('/', async( request, response, next ) => {
 	moveImage();// 取得写真を別フォルダに移動
@@ -282,7 +282,7 @@
 	// SQLによるDBデータ取得
-	await getSQLData();// await欲しいっす
+	await getSQLData();
 	next();

ソース内は太字にならず...

2021/01/29 07:26

投稿

HYDESA

スコア8

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -288,7 +288,7 @@
 });
-app.use('/*',async( request, response, **next** ) => {
+app.use('/*',async( request, response, next ) => {
 	try{

ソース内は太字にならず...

2021/01/29 07:18

投稿

HYDESA

スコア8

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -200,11 +200,13 @@
 const app = express()
-**var tmpData;** // グローバル宣言
+// グローバル宣言
+var tmpData;
-**var gl_Data;**
+var gl_Data;
-**var gl_Item;**
+var gl_Item;
 ...
@@ -224,7 +226,7 @@
 // SQLの実行関数
-**const getSQLData = async()=>{**
+const getSQLData = async()=>{
 	// SQLによるDBデータ取得
@@ -234,7 +236,7 @@
 	...
-**};**
+};
@@ -262,7 +264,7 @@
 	// SQLによるDBデータ取得
-	**await getSQLData();**// await欲しいっす
+	await getSQLData();// await欲しいっす
 	next();
@@ -280,7 +282,7 @@
 	// SQLによるDBデータ取得
-	**await getSQLData();**// await欲しいっす
+	await getSQLData();// await欲しいっす
 	next();
@@ -312,7 +314,7 @@
 		// html側にエラー状況を通告
-		**response.status(404).send('404 error, Page Not found');**
+		response.status(404).send('404 error, Page Not found');
 		// コンソール側にエラーを表示

備忘録追加

2021/01/29 07:16

投稿

HYDESA

スコア8

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -187,3 +187,151 @@
 お答えでも、全然かまいません。
 すいませんが、よろしくお願い致します。
+### 解決後の備忘録
+今回の変更箇所を記載しておきます。
+```express
+const express = require('express');
+const app = express()
+**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();**// await欲しいっす
+	next();
+});
+app.get('/', ,async( request, response, next ) => {
+	moveImage();// 取得写真を別フォルダに移動
+	makeMovie();// 写真から動画生成 exec(`ffmpeg ...
+	console.log("app.get");
+	// SQLによるDBデータ取得
+	**await getSQLData();**// await欲しいっす
+	next();
+});
+app.use('/*',async( request, response, **next** ) => {
+	try{
+		// getColumnDataモジュールで配列に置き換えた処理を4個動かしています
+		gl_Data = getColumnData(...);// 配列に置き換え
+		gl_Item = getColumnData(...);// 配列に置き換え
+		response.render('test.ejs',{
+			data: gl_Data
+			item: gl_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');
+});
+```
+うん、スッキリしました。

変数前後の[]を削除

2021/01/29 07:15

投稿

HYDESA

スコア8

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -112,7 +112,7 @@
 		var [data] = await pool.query(Sql);
-		// var [data] = await getSQLData(sql); とセット
+		// var data = await getSQLData(sql); とセット
 		// ここでは1個ですが、実際には5個のSQLを動かしていて

一か所だけの表記訂正

2021/01/28 08:27

投稿

HYDESA

スコア8

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -20,7 +20,7 @@
 この事から、SQLを無駄に動かさないように
-app.useの共通処理をやめ、
+`app.use`の共通処理をやめ、
 `get` `post` ともに1度だけSQLを実行すべく