运行Node.js服务器时出现CastError问题求助

Looks like the root cause here is a leading space in your blog ID (the value " 5f04cc85d5db3f0d582fb51a" has a space at the start). MongoDB's ObjectId can't handle whitespace, so when you pass this untrimmed ID to findById(), it throws that CastError. Let's break down how to fix this:

Step 1: Track down where the space comes from #

The most common culprit is the "Read more" link in your blogindex.ejs file. If you wrote something like this:

<a href="/blogs/ <%= blog._id %>">Read more</a>

The space between /blogs/ and <%= blog._id %> gets included in the URL. When the request hits your server, req.params.id ends up with that leading space attached.

Step 2: Fix the issue (two solid options) #

Option 1: Fix the frontend link (recommended) #

Remove the extra space in your blogindex.ejs link to ensure the URL has a clean ID:

<a href="/blogs/<%= blog._id %>">Read more</a>

This eliminates the whitespace at the source, so you won't have to handle it on the backend at all.

Option 2: Clean the ID on the backend #

If you can't modify the frontend right now, trim the whitespace from the ID before querying the database. Also, avoid using Blog as your callback variable name—it conflicts with your model:

// SHOW ROUTE
app.get('/blogs/:id', (req,res) => {
  // Trim leading/trailing whitespace from the ID
  const cleanBlogId = req.params.id.trim();
  
  Blog.findById(cleanBlogId, (err, foundBlog)=>{
    if(err){
      console.error(err);
      res.send(`Error: ${err}`);
    }else{
      res.render('blogshow', {blog: foundBlog})
    }
  });
});

Bonus: Add validation for better error handling #

To make your code more robust, check if the trimmed ID is a valid ObjectId before querying the database. This helps you return clearer errors to users:

const mongoose = require('mongoose');

// SHOW ROUTE
app.get('/blogs/:id', (req,res) => {
  const cleanBlogId = req.params.id.trim();
  
  // Validate the ID format
  if(!mongoose.Types.ObjectId.isValid(cleanBlogId)){
    return res.status(404).send('Invalid blog ID - please check your link');
  }
  
  Blog.findById(cleanBlogId, (err, foundBlog)=>{
    if(err){
      console.error(err);
      res.status(500).send('Failed to fetch blog');
    }else if(!foundBlog){
      res.status(404).send('Blog not found');
    }else{
      res.render('blogshow', {blog: foundBlog})
    }
  });
});

Extra suggestion: Use async/await instead of callbacks #

Callbacks can get messy as your app grows. Switching to async/await will make your code more readable and easier to debug:

app.get('/blogs/:id', async (req,res) => {
  try {
    const cleanBlogId = req.params.id.trim();
    
    if(!mongoose.Types.ObjectId.isValid(cleanBlogId)){
      return res.status(404).send('Invalid blog ID');
    }
    
    const foundBlog = await Blog.findById(cleanBlogId);
    
    if(!foundBlog){
      return res.status(404).send('Blog not found');
    }
    
    res.render('blogshow', {blog: foundBlog});
  } catch (err) {
    console.error(err);
    res.status(500).send('Error fetching blog');
  }
});

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