回答編集履歴

createConnectionの使い方の変化に追従出来るよう修正

2021/11/12 06:16

投稿

miyabi-sun

スコア21194

test CHANGED Viewed

@@ -164,7 +164,9 @@
-// ここは基本的にこのままで構わない
+// ここはこのままで構わない
+// ただし返り値がPromiseのインスタンスになっており、一度thenを叩いて本物のコネクションを取り出す必要がある
 const connection = mysql.createConnection({
@@ -186,13 +188,15 @@
+  // connection.thenを基点に展開する
-  connection.query(
+  connection.then(c => c.query(
     'SELECT date, temperature, humidity FROM data WHERE device = ? ORDER BY date ASC;',
     [req.body.device]
-  ).then(results => {
+  )).then(results => {
     console.log(results);
@@ -200,25 +204,163 @@
-    return connection.query(
+    return connection.then(c => c.query(
       'SELECT * FROM data where date=(select max(date) from data where device = ?);',
       [req.body.device]
+    ));
+  }).then(results => {
+    console.log(results);
+    variables.news = results;
+    res.render('index.ejs', valiables);
+  });
+});
+```
+Promiseのインスタンスは`.then`メソッドを使う事で数珠つなぎに出来ます。
+コールバック関数を引数に渡すやり方に比べればネストが一段階下がった事がわかりますね。
+「で？このPromise使った書き方の何が嬉しいんだい？」
+そうなんですよね、Promiseはコールバック地獄を解決しただけで
+アプリ開発者目線では別に何も嬉しくなってないんですよね。
+Promiseはasync/await構文と併用して、初めて凄まじい威力を発揮するようになります。
+```js
+const mysql = require("mysql2/promise");
+const connection = mysql.createConnection({
+  host: 'localhost',
+  user: 'root',
+  password: '-------',
+  database: '********'
+});
+// await構文はasync関数の直下でないと使えない
+// 普通のアロー関数の前にasyncというワードを付与するだけ
+app.post('/select', async (req, res) => {
+  // awaitを使うとPromise.then(val => {})を実行してvalを取り出すコードに変換される
+  const c = await connection;
+  const items = await c.query(
+    'SELECT date, temperature, humidity FROM data WHERE device = ? ORDER BY date ASC;',
+    [req.body.device]
+  );
+  const news = await c.query(
+    'SELECT * FROM data where date=(select max(date) from data where device = ?);',
+    [req.body.device]
+  );
+  console.log(items);
+  console.log(news);
+  res.render('index.ejs', {items, news});
+});
+```
+はいめっちゃ読みやすい。
+でもまぁ、今度は`'SELECT * FROM data where date=(select max(date) from data where device = ?);'`が心配ですね。
+本当に動くのかい？
+try-catchで拾ってなんかエラー出したら怒るようにしておきましょうか。
+```js
+app.post('/select', async (req, res) => {
+  const val = {};
+  try {
+    const c = await connection;
+    const items = await connection.query(
+      'SELECT date, temperature, humidity FROM data WHERE device = ? ORDER BY date ASC;',
+      [req.body.device]
     );
-  }).then(results => {
-    console.log(results);
-    variables.news = results;
+    const news = await connection.query(
+      'SELECT * FROM data where date=(select max(date) from data where device = ?);',
+      [req.body.device]
+    );
-    res.render('index.ejs', valiables);
+    res.render('index.ejs', {items, news});
+  } catch (e) {
+    console.log("SQLクエリの発行に失敗");
+    console.error(e);
+    res.send('サーバーでの動作が大失敗しました。急いで直すのでしばしお待ちを。');
-  });
+  }
 });
@@ -226,138 +368,4 @@
-Promiseのインスタンスは`.then`メソッドを使う事で数珠つなぎに出来ます。
-コールバック関数を引数に渡すやり方に比べればネストが一段階下がった事がわかりますね。
-「で？このPromise使った書き方の何が嬉しいんだい？」
-そうなんですよね、Promiseはコールバック地獄を解決しただけで
-アプリ開発者目線では別に何も嬉しくなってないんですよね。
-Promiseはasync/await構文と併用して、初めて凄まじい威力を発揮するようになります。
-```js
-const mysql = require("mysql2/promise");
-const connection = mysql.createConnection({
-  host: 'localhost',
-  user: 'root',
-  password: '-------',
-  database: '********'
-});
-// await構文はasync関数の直下でないと使えない
-// 普通のアロー関数の前にasyncというワードを付与するだけ
-app.post('/select', async (req, res) => {
-  // awaitを使うとPromise.then(val => {})を実行してvalを取り出すコードに変換される
-  const items = await connection.query(
-    'SELECT date, temperature, humidity FROM data WHERE device = ? ORDER BY date ASC;',
-    [req.body.device]
-  );
-  const news = await connection.query(
-    'SELECT * FROM data where date=(select max(date) from data where device = ?);',
-    [req.body.device]
-  );
-  console.log(items);
-  console.log(news);
-  res.render('index.ejs', {items, news});
-});
-```
-はいめっちゃ読みやすい。
-でもまぁ、今度は`'SELECT * FROM data where date=(select max(date) from data where device = ?);'`が心配ですね。
-本当に動くのかい？
-try-catchで拾ってなんかエラー出したら怒るようにしておきましょうか。
-```js
-app.post('/select', async (req, res) => {
-  const val = {};
-  try {
-    const items = await connection.query(
-      'SELECT date, temperature, humidity FROM data WHERE device = ? ORDER BY date ASC;',
-      [req.body.device]
-    );
-    const news = await connection.query(
-      'SELECT * FROM data where date=(select max(date) from data where device = ?);',
-      [req.body.device]
-    );
-    res.render('index.ejs', {items, news});
-  } catch (e) {
-    console.log("SQLクエリの発行に失敗");
-    console.error(e);
-    res.send('サーバーでの動作が大失敗しました。急いで直すのでしばしお待ちを。');
-  }
-});
-```
 とりまこんな感じですかね。