Node.js使用Moment.js处理日期报非ISO格式警告问题

• 未选择结束日期时，前端会给后端传递endDate = undefined，将undefined传入moment()时，输入不符合RFC2822/ISO格式要求，就会触发你看到的弃用警告，此时moment会回退到原生Date()做解析，不同浏览器/版本下解析结果不可靠。
• 自定义的isValid()函数逻辑完全不生效：new Date(moment(undefined))返回的是Invalid Date类型对象，和字符串"Invalid Date"做相等判断永远返回false，永远走不到你设置的默认2100-11-11分支。
• 额外逻辑问题：聚合查询里套了无意义的$or（数组内只有一个匹配条件，不需要$or包裹）；日期边界处理有缺陷，传入yyyy-mm-dd格式字符串时默认解析为当天0点，会漏掉endDate当天0点之后的所有数据；catch分支返回404状态码不符合规范，服务端执行错误应该返回500类状态码。

后端代码调整 #

不要等生成Invalid Date对象后再做判断，直接在解析前校验参数是否存在，使用moment内置的严格格式解析避免警告，同时修正日期边界逻辑：

router.get("/searchDate", async (req, res) => {
  try {
    const { startDate, endDate } = req.query;
    const manufacturingDateQuery = {};

    // 处理起始日期：参数合法则取当天0点，否则默认取最早时间戳
    if (startDate && moment(startDate, "YYYY-MM-DD", true).isValid()) {
      manufacturingDateQuery.$gte = new Date(moment(startDate, "YYYY-MM-DD").startOf('day'));
    } else {
      manufacturingDateQuery.$gte = new Date(0);
    }

    // 处理结束日期：参数合法则取次日0点（覆盖所选日期全天数据），否则默认取远期时间
    if (endDate && moment(endDate, "YYYY-MM-DD", true).isValid()) {
      manufacturingDateQuery.$lt = new Date(moment(endDate, "YYYY-MM-DD").add(1, 'day').startOf('day'));
    } else {
      manufacturingDateQuery.$lt = new Date(moment("2100-11-11"));
    }

    const containers = await Container.aggregate([
      {
        $match: {
          manufacturingDate: manufacturingDateQuery
        },
      },
    ]);

    res.status(200).json(containers);
  } catch (err) {
    res.status(500).json({ success: false, msg: "Date search failed" });
  }
});

调整点说明：

• 解析日期时显式指定YYYY-MM-DD格式，传入第三个参数true开启严格解析，从根源上避免格式不识别的警告，也不会出现跨浏览器解析不一致的问题
• 参数校验前置，不会给moment传入undefined这类非法值
• 结束日期做加1天处理，例如传入endDate=2022-06-06时，实际匹配小于2022-06-07 0点的所有数据，覆盖6月6日全天记录，不会漏数据
• 移除多余的$or包裹，简化查询结构
• 修正错误状态码为500，符合HTTP状态码语义

前端代码小调整 #

现有convert函数在日期参数为空时，会生成NaN-NaN-NaN格式的非法字符串传给后端，加一层空值判断即可：

onChangeDate() {
  function convert(str: any) {
    if (!str) return null;
    const date = new Date(str),
      mnth = ("0" + (date.getMonth() + 1)).slice(-2),
      day = ("0" + date.getDate()).slice(-2);
    return [date.getFullYear(), mnth, day].join("-");
  }

  this.containersService
    .getContainersByDateRange(convert(this.from), convert(this.to))
    .subscribe((containers: any) => {
      this.containers = containers;
    });
}

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