Node.js登录调用parseData后跳转页面仅半数成功问题排查

跳转偶发失效的核心原因是异步逻辑写法错误，await没有真正等待异步操作执行完成，叠加多处逻辑bug导致运行不稳定：

• fs.readFile 调用方式错误：你同时传入了回调参数和await，Node.js中fs.readFile只要传入回调就不会返回Promise，外层await会直接拿到undefined并立刻执行后续逻辑，完全不会等待文件读取完成。此时data变量为undefined，执行data.length时会直接抛出类型错误，中断请求流程导致跳转不执行。你加750ms固定延迟后，大概率能赶上文件读取完成，因此成功率提升到90%，但本质是碰运气，没有解决根本问题。另外回调内的return data完全无效，返回值会被直接丢弃，读文件的错误也只是打印日志没有向外抛出，会导致请求直接挂起超时。
• 数组存在性判断错误：getMaxStreak中用in操作符判断日期是否在数组内，in仅能检查对象的键名，数组的键是数字索引，传入日期字符串做判断永远返回false，会重复向数组插入数据，极端场景下可能触发循环异常。
• 日期计算逻辑错误：convert函数手写日期偏移没有处理跨年、大小月、闰年场景，比如1月的上一月会被计算为00月，3月1日的前一天固定返回上月30/31日，会导致日期匹配逻辑异常，极端情况可能造成循环越界。
• 并发安全隐患：代码中用到的mostRecentName未在函数内声明，如果是全局变量，多用户并发请求时会互相覆盖值，导致文件解析逻辑错乱。所有异步操作都没有错误捕获，一旦抛出异常请求会直接挂死，既不会跳转也不会返回错误提示。

1. 修正文件读取逻辑，使用Node.js官方提供的Promise版fs接口，去掉回调，确保await能正确等待文件读取完成：
   把原有读文件代码替换为 const data = await fs.promises.readFile(__dirname + "/mem.txt", "utf8")，同时将函数内的var声明替换为const/let，避免作用域混乱。
2. 修正数组存在性判断，将if (!(testDate in dates))替换为if (!dates.includes(testDate))。
3. 给路由逻辑加全局错误捕获，出现异常时直接返回错误响应，避免请求挂死。
4. 替换手写日期偏移逻辑，直接用原生Date对象计算前一天日期，避免手动处理日期的边界错误。
5. 排查mostRecentName变量来源，禁止用全局变量存储用户相关的请求数据，应该从请求对象req中取对应用户信息，作为参数传入parseData调用链路，避免并发请求数据串扰。
6. 修复完成后可以删除之前加的固定等待逻辑await sleep(750)，异步逻辑正确的情况下会在所有处理完成后立刻执行跳转，不需要靠延迟碰运气。

const fs = require('fs/promises'); // 直接引入promise版fs

app.post("/login.html", urlencodedParser, async (req, res) => {
  try {
    // 把用户信息作为参数传入，不要用全局变量，替换为你实际取当前登录用户名的逻辑
    await parseData(mostRecentName);
    res.redirect(__dirname + "/homescreen.html");
  } catch(err) {
    console.error('登录处理失败:', err);
    res.status(500).send('服务端处理异常');
  }
});

async function parseData(userName) {
  const dates = await findCommon(userName);
  const maxStreak = await getMaxStreak(dates);
  // 若需要使用maxStreak结果可在此处做存储/返回逻辑，不影响跳转执行
}

async function findCommon(userName) {
  const dates = new Set();
  // 正确等待文件读取，不传回调
  const data = await fs.readFile(__dirname + "/mem.txt", "utf8");
  const targetName = userName;

  for (let i = 0; i < data.length; i++) {
    if (data[i] === "*" && i + targetName.length < data.length) {
      if (data.slice(i + 1, i + targetName.length + 1) == targetName) {
        while (data[i] != "\n") {
          i++;
        }
        if (i < data.length - 1) {
          i++;
        }
        while (data[i] != "*" && i < data.length) {
          let curr = "";
          let count = 10;
          while (count > 0) {
            count--;
            curr += data[i];
            i++;
          }
          while (data[i] != "\n") {
            i += 1;
          }
          if (i < data.length - 1) {
            i++;
          }
          dates.add(curr);
        }
      }
    }
  }
  const datesArr = Array.from(dates);
  // 若bubbleSort是自定义异步方法，确保其返回Promise即可正常await
  return bubbleSort(datesArr);
}

async function getMaxStreak(dates) {
  const today = new Date();
  // 直接用原生方法格式化日期，避免手动拼接出错
  const testDate = today.toISOString().slice(0,10);

  // 用includes判断数组成员
  if (!dates.includes(testDate)) {
    dates.push(testDate);
  }
  let streak = 1;
  for (let i = dates.length - 1; i > 0; i--) {
    const currentDateStr = i == dates.length - 1 ? testDate : dates[i];
    const prevValidDates = await getPrevValidDates(currentDateStr);
    if (prevValidDates.includes(dates[i-1])) {
      streak++;
    } else {
      return streak;
    }
  }
  return streak;
}

// 替换原有错误的convert函数，用Date对象计算日期，自动处理跨年/大小月/闰年
async function getPrevValidDates(dateStr) {
  const current = new Date(dateStr);
  // 计算前一天日期
  current.setDate(current.getDate() - 1);
  const prevDay = current.toISOString().slice(0,10);
  // 如果业务需要匹配上月最后1-2天，可通过Date对象取当月第一天再减一天拿到上月最后一天，不要硬写30/31
  return [prevDay];
}

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