求优化JavaScript获取下一个指定星期几日期的函数

你的核心需求是找到下一个指定星期几的日期，原代码逻辑可行，但可以简化结构、消除潜在副作用，让代码更简洁健壮。

原代码的可优化点 #

• 副作用隐患：lessonDate = todayDateObj是引用赋值，调用setDate会直接修改传入的原Date对象，可能引发意外问题。
• 冗余辅助函数：三个辅助函数的逻辑可以合并，减少代码层级，提升可读性。
• 外部依赖风险：依赖外部的dow数组，一旦外部变量变化会直接影响函数运行，最好将映射关系内置。

简化后的实现 #

const getLessonDate = (lessonWeekdayString, todayDateObj) => {
  // 与Date.getDay()的返回值对应：周日=0，周一=1...周六=6
  const weekdayMap = ['周日', '周一', '周二', '周三', '周四', '周五', '周六'];
  const todayWeekday = todayDateObj.getDay();
  const targetWeekday = weekdayMap.indexOf(lessonWeekdayString);

  // 校验输入合法性
  if (targetWeekday === -1) {
    throw new Error('请传入合法的星期几字符串，如"周一"');
  }

  // 计算需要添加的天数：若今天≤目标则直接取差值，否则取下周的对应日期
  const daysToAdd = todayWeekday <= targetWeekday
    ? targetWeekday - todayWeekday
    : 7 - (todayWeekday - targetWeekday);

  // 创建原日期的副本，避免修改传入的原对象
  const nextTargetDate = new Date(todayDateObj);
  nextTargetDate.setDate(nextTargetDate.getDate() + daysToAdd);

  return nextTargetDate;
};

关键改进说明 #

1. 消除副作用：通过new Date(todayDateObj)创建原日期的副本，所有操作仅作用于副本，不会改变传入的原始Date对象。
2. 简化计算逻辑：直接用7 - (todayWeekday - targetWeekday)替代原有的多函数组合计算，逻辑更直观。
3. 内置映射关系：将星期字符串与数字的映射内置到函数中，提升函数独立性，无需依赖外部变量。
4. 增加输入校验：添加对星期字符串的合法性检查，避免因无效输入导致错误。

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