【Node.js+EJS+SQLite3】異なるテーブルからデータを抜き出し、EJSに表示させたい。

Question

初めて質問させていただきます。初学者のため、足りない情報などあるかと思いますので、その都度ご指摘いただければと思います。

実現したいこと

フォーム画面を作成したい。

• プルダウンの中身をデータベースから引っ張って表示させたい。
• 引っ張るテーブルはそれぞれ異なる。（2つ以上のテーブルから参照する
• プルダウンの個数は2個以上

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

    31|                         </select>
    32|                     </div>
 >> 33|                     <% for(let i in check_data){ %>
    34|                         <div class="mb-2">
    35|                             <label class="form-label" for="date"><%= check_data[i].item %></label>
    36|                             <div class="form-check form-switch">

check_data is not defined

該当のソースコード

var express = require('express');
var router = express.Router();

//load the sqlite
const sqlite3 = require('sqlite3');
const db = new sqlite3.Database('./MEIS.db');

/* Join to User*/
router.get('/form', function(req, res,next) {
    db.all("SELECT * FROM "+'"'+req.query.user+'"', (err, rows) =>{
      if (err) {
          console.log("DBのユーザー情報に異常があります。");
          return;
      }
      let form_opt ={form_data:rows}
      res.render('pages/form',form_opt);
    });
    db.all("SELECT * FROM "+'"'+req.query.check+'"', (err, rows) =>{
      if (err) {
          console.log("DBの点検項目に異常があります。");
          return;
      }
      let check_opt ={check_data:rows}
      res.render('pages/form',check_opt);
    });
});

module.exports = router;

<%- include('../partials/head'); %>
    <%- include('../partials/header'); %>
        <main>
            <div class="container">
                <form action="" method="post">
                    <div class="mb-2">
                        <label class="form-label" for="date">日時を選択してください。</label>
                        <input class="form-control" type="date" id="today">
                        <script>
                            var date = new Date();
                            var yyyy = date.getFullYear();
                            var mm = ("0"+(date.getMonth()+1)).slice(-2);
                            var dd = ("0"+date.getDate()).slice(-2);
                            document.getElementById("today").value=yyyy+'-'+mm+'-'+dd;
                        </script>
                    </div>
                    <div class="mb-2">
                        <label class="form-label" for="date">ユーザーを選択してください。</label>
                        <select class="form-select">
                            <% for(let i in form_data){ %>
                                <option value="<%= form_data[i].id %>"><%=form_data[i].name%></option>
                                <%}%>
                        </select>
                    </div>
                    <% for(let i in check_data){ %>
                        <div class="mb-2">
                            <label class="form-label" for="date"><%= check_data[i].item %></label>
                            <div class="form-check form-switch">
                                <input class="form-check-input" type="checkbox" id="flexSwitchCheckDefault">
                                <label class="form-check-label" for="flexSwitchCheckDefault">Default switch checkbox input</label>
                            </div>
                        </div>
                    <%}%>
                </form>
            </div>
        </main>
<%- include('../partials/footer'); %>

個人的に以下の箇所が問題であると考えています。

router.get('/form', function(req, res,next) {
    db.all("SELECT * FROM "+'"'+req.query.user+'"', (err, rows) =>{
      if (err) {
          console.log("DBのユーザー情報に異常があります。");
          return;
      }
      let form_opt ={form_data:rows}
      res.render('pages/form',form_opt);
    });
    db.all("SELECT * FROM "+'"'+req.query.check+'"', (err, rows) =>{
      if (err) {
          console.log("DBの点検項目に異常があります。");
          return;
      }
      let check_opt ={check_data:rows}
      res.render('pages/form',check_opt);
    });
});

試したこと

• db.serializeでの同期化
• form.ejsの記載ミスの確認。（片方を消して、動作を確認しましたが、片方ずつなら表示されます。

res.renderが2箇所記載されているのが問題でしょうか？db.allの外でできれば良いのですが、外で記述してしまうとそれもエラーになってしまい困っております・・・

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

• Node.js : 18.12.1
• EJS : 3.1.8
• Express : 4.18.2
• SQLite3 : 5.1.4

Comments

[hoshi-takanori]

Node.js の DB アクセスは非同期なので、コールバックで処理するなら最初のコールバックの中で次の DB アクセスを行う必要があります。

追加。これをコールバック地獄と言って、何が起きてるかはこの記事が参考になるかと。(今は promise よりも async/await を使うとすっきり書けそう…。)
https://qiita.com/yusuke_ten/items/7cb87fed011cffbf2d4f

[uuchu]

コメントありがとうございます！！
以下のような形で、良いのでしょうか？・・・
```form.js
router.get('/form', function(req, res,next) {
db.all("SELECT * FROM "+'"'+req.query.user+'"', (err, rows) =>{
if (err) {
console.log("DBのユーザー情報に異常があります。");
return;
}
let form_opt ={form_data:rows}
res.render('pages/form',form_opt)
db.all("SELECT * FROM "+'"'+req.query.check+'"', (err, rows) =>{
if (err) {
console.log("DBの点検項目に異常があります。");
return;
}
let check_opt ={check_data:rows}
res.render('pages/form',check_opt);
});
});
});
```
これでも、エラーになってしまいますね・・・
```
31| </select>
32| </div>
>> 33| <% for(let i in check_data){ %>
34| <div class="mb-2">
35| <label class="form-label" for="date"><%= check_data[i].item %></label>
36| <div class="form-check form-switch">
check_data is not defined
```

```res.render```自体は、何度も書いて良いものなんでしょうか？

[hoshi-takanori]

res.render は最後に一回だけです。ってか、よく見たら SQL 文もやばいですね。入力値をチェックせずに文字列連結するのは SQL インジェクションの脆弱性です。しかもテーブル名が可変な時点で DB 設計も怪しい…。さらに、各 route で同じ DB をそれぞれ開いてそうですが、それって大丈夫でしたっけ？

[uuchu]

やばいですか汗
それぞれ細かくご教授お願いしたいくらいです・・・が
学習におすすめの書籍などありますでしょうか？