Node.js如何捕获促销码不存在的Promise错误并优化代码？

Hey there! Let's break down your problem and fix this step by step. First, let's look at the issues in your current code, then I'll show you a cleaner, more maintainable solution.

当前代码的问题 #

1. 手动Promise封装冗余：Mongoose从4.8版本开始就支持原生Promise了，没必要用q库手动封装，这会增加不必要的复杂度。
2. 错误处理逻辑混淆：当促销码不存在时，用reject(new Error(...))会抛出完整的错误栈——因为Node.js的Error对象默认会生成调用栈信息，这会导致服务器返回一堆冗余的栈数据给客户端。另外如果把那行代码注释掉，这个Promise会一直处于pending状态，永远不完成，这也是个bug。
3. 查询方法选择不当：Promotion.find()返回的是数组，但促销码一般是唯一的，用findOne()更高效，直接返回单个文档，不用额外判断数组长度。

优化后的代码实现 #

我们会用async/await简化异步代码，利用Mongoose的原生Promise，同时区分业务逻辑里的“不存在”情况和真正的服务器异常错误。

var mongoose = require('mongoose');
var Promotion = mongoose.model('Promotion');

var PromotionController = {
  // 使用async函数让异步代码更易读，逻辑更清晰
  applyPromo: async function(promoCode) {
    // 先处理空促销码的边界情况
    if (!promoCode) {
      return null; // 也可以返回结构化对象，比如 { valid: false, reason: "促销码不能为空" }
    }

    try {
      // 用findOne查询唯一促销码，比find更高效直接
      const promotion = await Promotion.findOne({ code: promoCode });
      
      if (promotion) {
        console.log('有效促销码:', promotion);
        return promotion._id; // 返回有效促销码的ID
      } else {
        // 促销码不存在属于预期的业务情况，返回null而非抛出错误
        return null;
      }
    } catch (err) {
      // 这里捕获真正的服务器错误，比如数据库连接失败
      console.error('验证促销码时数据库出错:', err);
      // 抛出友好的错误信息，供上层处理
      throw new Error('验证促销码失败');
    }
  }
};

module.exports = PromotionController;

上层调用示例（路由处理） #

为了避免把错误栈返回给客户端，在路由/请求处理层这样处理结果：

// 假设在Express路由中
app.post('/apply-promo', async (req, res) => {
  try {
    const promoId = await PromotionController.applyPromo(req.body.promoCode);
    
    if (promoId) {
      res.status(200).json({ 
        success: true, 
        message: '促销码验证成功',
        promoId 
      });
    } else {
      // 促销码不存在，返回友好的客户端提示，无错误栈
      res.status(400).json({ 
        success: false, 
        message: '该促销码不存在' 
      });
    }
  } catch (err) {
    // 处理服务器端异常，返回通用提示
    res.status(500).json({ 
      success: false, 
      message: err.message 
    });
  }
});

关键优化点说明 #

• 区分业务逻辑与异常错误：促销码不存在是预期内的业务场景，所以我们用返回null的方式处理，而Promise的reject只留给数据库连接失败这类意外的服务器错误。
• Async/Await简化代码：消除了手动封装Promise的繁琐，让异步代码的逻辑更像同步代码，可读性和可维护性大幅提升。
• 用findOne替代find：直接返回匹配的文档（如果存在），不用再处理数组长度判断的逻辑，代码更简洁高效。

如果你坚持保留原Promise写法（不推荐） #

如果想维持原有的Promise结构，可以调整错误逻辑避免返回完整栈：

var mongoose = require('mongoose'), Promotion = mongoose.model('Promotion'), q = require('q'), Promise = q.Promise;
var PromotionController = {
  applyPromo: function(promoCode) {
    return new Promise(function(resolve, reject) {
      if(promoCode === null) {
        resolve(false);
      }
      Promotion.findOne({code: promoCode}, function (err, promotion) {
        if (err) {
          // 捕获数据库错误，返回友好信息而非原始错误对象
          return reject(new Error('验证促销码失败'));
        }
        if(promotion) {
          console.log('OK');
          console.log(promotion);
          resolve(promotion._id);
        } else {
          // 用resolve返回标识而非reject，避免触发错误栈
          resolve(null);
          // 如果你一定要用reject，上层必须过滤掉栈信息，只返回错误消息
          // reject(new Error('该促销码不存在'));
        }
      });
    });
  }
};
module.exports = PromotionController;

注意这种情况下，上层调用时绝对不要把完整的Error对象返回给客户端，只返回错误消息即可。

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