Express中Mongoose find条件判断报错（URL短链项目）

我来帮你搞定这个问题！你遇到的property userUrl undefined错误，根源在于当查询新链接时，数据库返回的doc是一个空数组，这时候doc[0]就是undefined，你再去访问doc[0].userUrl自然会报错。而且你的条件判断逻辑也有问题，导致“创建新文档”的分支根本不会触发。

问题分析 #

原来的代码里，你直接用else if(doc[0].userUrl==req.params.url)来判断，但如果没有找到匹配的记录，doc是空数组，doc[0]不存在，这行代码直接抛出错误，后续的else分支根本没机会执行。

修正后的代码（回调版本） #

我们需要先判断查询结果是否为空，再执行对应的逻辑：

app.get('/:url', (req,res) => { 
  const targetUrl = req.params.url;
  Url.find({userUrl: targetUrl},(err,doc)=>{ 
    // 先处理数据库错误
    if(err){ 
      console.error(err);
      return res.status(500).send('服务器错误'); // 别忘了给客户端返回响应
    }
    // 判断是否找到匹配记录：doc是数组，长度大于0表示存在
    if(doc.length > 0) { 
      console.log('hi');
      // 这里可以加上跳转到原链接的逻辑，比如res.redirect(doc[0].userUrl)
    }else { 
      // 未找到则创建新文档
      const url = new Url({ userUrl: targetUrl }); 
      url.save()
        .then(() => {
          console.log('新链接已创建');
          res.send('新链接创建成功');
        })
        .catch(saveErr => {
          console.error(saveErr);
          res.status(500).send('创建链接失败');
        });
    } 
  }) 
})

更推荐的Async/Await版本（代码更清晰，避免回调地狱） #

app.get('/:url', async (req,res) => { 
  try {
    const targetUrl = req.params.url;
    // 用findOne更高效，因为我们只需要判断是否存在一条记录
    const existingUrl = await Url.findOne({userUrl: targetUrl});
    if(existingUrl) { 
      console.log('hi');
      // res.redirect(existingUrl.userUrl);
    }else { 
      const newUrl = new Url({ userUrl: targetUrl }); 
      await newUrl.save();
      console.log('新链接已创建');
      res.send('新链接创建成功');
    }
  } catch(err) {
    console.error(err);
    res.status(500).send('服务器错误');
  }
})

关键优化点 #

• 优先判断查询结果是否存在：通过doc.length > 0（回调版）或直接判断existingUrl是否为null（findOne版），避免访问undefined的属性。
• 完善错误处理：不仅要打印错误，还要给客户端返回对应的HTTP响应，避免请求挂起。
• 使用findOne代替find：因为我们只需要检查是否存在匹配的单条记录，findOne性能更高，返回的是单个文档而非数组，代码更简洁。
• Async/Await写法：相比嵌套回调，异步/等待的写法更易读、易维护，也更容易处理错误。

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