NodeJS中EJS渲染数据库数组提示未定义错误的解决方案咨询

问题排查与修复步骤： #

1. 修正POST请求的逻辑错误
   你的POST方法里，res.redirect('/quotes')会直接结束当前响应，后面的数据库查询代码根本不会执行。而且POST请求插入数据后，直接重定向到GET /quotes即可，GET请求会负责查询数据并渲染页面，所以POST代码可以简化：

app.post('/quotes', (req, res) => {
  datacollection.insertOne(req.body)
    .then(result => {
      console.log(result)
      // 插入成功后直接重定向，由GET请求处理数据查询和页面渲染
      res.redirect('/quotes')
    })
    .catch(error => console.error(error))
});

2. 完善GET请求的错误处理
   你的GET请求里catch块是空的，如果数据库查询失败，会导致res.render根本不会执行，页面无法正确接收datacollection变量，从而报错“is not defined”。必须在catch里处理错误，同时保证页面能接收到变量：

app.get('/quotes', (req, res) => {
  db.collection('datacollection').find().toArray()
    .then(results => {
      // 即使结果为空，也要传递变量给EJS
      res.render('quotes.ejs', { datacollection: results || [] })
    })
    .catch(error => {
      console.error(error)
      // 查询失败时，传递空数组避免页面报错
      res.render('quotes.ejs', { datacollection: [] })
    })
})

3. 修正EJS页面的数据渲染方式
   直接用<%= datacollection %>输出数组，只会显示[object Object]之类的无效字符串，而且如果数组为空仍可能引发报错。正确的做法是遍历数组并先做存在性判断：

<!-- quotes.ejs -->
<% if (datacollection && datacollection.length > 0) { %>
  <ul>
    <% datacollection.forEach(item => { %>
      <li>
        <!-- 根据你的数据结构输出对应字段，比如item.name、item.quote等 -->
        <%= item.name %>: <%= item.quote %>
      </li>
    <% }) %>
  </ul>
<% } else { %>
  <p>暂无数据</p>
<% } %>

额外检查点 #

• 确认数据库连接正常，db和datacollection变量已正确初始化，无拼写错误
• 检查EJS文件是否放在默认的views目录下，文件名是否与代码中一致（quotes.ejs）

内容的提问来源于stack exchange，提问作者LCD1