Node.js中持续出现'cant set headers after they are sent'错误求助

Hey there! Let's squash that frustrating can't set headers after they are sent error you're dealing with. I've gone through your code, and the root cause is simple to spot and fix.

Why This Error Happens #

This error pops up when your server tries to send two responses for a single HTTP request. HTTP rules only allow one response per request—once you send headers (like when you render a page or redirect), you can't send another set of headers afterward.

Looking at your routes.js file's POST /contact handler, you're doing exactly that:

router.post('/contact', [/* validation rules */], (req,res) => {
  const errors = validationResult(req);
  res.render('contact', { data: req.body, errors: errors.mapped() }); // First response sent here
  const data = matchedData(req);
  console.log('Sanitized: ', data);
  req.flash('success', 'Thanks for the message! I\'ll be in touch :) ');
  res.redirect('/'); // Second response attempt triggers the error
});

The Fix #

You need to add a condition: only render the contact page if there are validation errors. If everything checks out, process the data, set your flash message, and redirect. Here's the corrected code for the route:

router.post('/contact', [
  check('message')
    .isLength({ min:1 })
    .withMessage('Message is required')
    .trim(),
  check('email')
    .isEmail().withMessage('That Email doesn\'t look right')
    .trim()
    .normalizeEmail()
], (req,res) => {
  const errors = validationResult(req);
  
  // If there are errors, render the form with error messages and stop execution
  if (!errors.isEmpty()) {
    return res.render('contact', { data: req.body, errors: errors.mapped() });
  }

  // No errors? Proceed with processing and redirect
  const data = matchedData(req);
  console.log('Sanitized: ', data);
  req.flash('success', 'Thanks for the message! I\'ll be in touch :) ');
  res.redirect('/');
});

Bonus: Show the Success Flash Message #

While you're at it, update your index.ejs to display the success flash message so users know their submission worked:

<h1>Working With Forms in Node.js</h1>
<% if (req.flash('success').length > 0) { %>
  <div class="success-notice"><%= req.flash('success')[0] %></div>
<% } %>

Quick Recap #

By adding the if (!errors.isEmpty()) check and using return to exit the function early when there are errors, you ensure only one response is sent per request. This eliminates the header conflict that was causing your error.

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