如何验证无返回值的localStorage登录函数是否执行成功？

你当前的代码确实存在两个核心问题：一是没处理localStorage.setItem可能抛出的异常，二是仅通过存在性判断存储成功，无法确保内容准确。以下是具体的优化方案：

核心问题分析 #

1. 未捕获异常：localStorage.setItem在存储空间不足、浏览器禁用本地存储、或者存储内容过大时，会抛出DOMException，直接导致函数中断，根本执行不到后续的isUserLoggedIn调用。
2. 验证不严谨：就算setItem没抛错，也可能出现存储内容不完整或异常的情况，仅判断“存在值”无法保证存储的是正确的用户数据。

优化方案：try/catch + 内容一致性验证 #

利用try/catch捕获存储异常，同时在存储完成后验证内容是否与原数据一致，确保操作真正成功：

logInUser(user: IPerson): boolean {
  try {
    const userStr = JSON.stringify(user);
    localStorage.setItem(this.userID, userStr);
    
    // 验证存储内容是否完全一致
    const storedUserStr = localStorage.getItem(this.userID);
    if (storedUserStr !== userStr) {
      // 清理错误存储的数据
      localStorage.removeItem(this.userID);
      return false;
    }
    
    return true;
  } catch (err) {
    // 捕获存储失败的异常（如空间不足、禁用存储等）
    console.error('用户登录存储失败:', err);
    return false;
  }
}

isUserLoggedIn(): boolean {
  return Boolean(localStorage.getItem(this.userID));
}

细节说明：

• 异常捕获：try/catch能覆盖所有localStorage操作可能抛出的错误，包括权限问题、空间不足等。
• 内容验证：对比存储的字符串与原序列化字符串，避免出现存储不完整数据的情况（比如存储过程中意外中断）。如果你的IPerson包含undefined、函数等JSON.stringify会忽略的属性，可以改用深度对象对比（手写对比逻辑或工具函数均可）。

可选：包装为Promise异步接口 #

如果需要统一异步代码风格，可以把函数包装成Promise，方便用await调用：

logInUser(user: IPerson): Promise<boolean> {
  return new Promise((resolve) => {
    try {
      const userStr = JSON.stringify(user);
      localStorage.setItem(this.userID, userStr);
      
      const storedUserStr = localStorage.getItem(this.userID);
      if (storedUserStr !== userStr) {
        localStorage.removeItem(this.userID);
        resolve(false);
        return;
      }
      
      resolve(true);
    } catch (err) {
      console.error('用户登录存储失败:', err);
      resolve(false);
    }
  });
}

调用示例：

const loginSuccess = await logInUser(yourUser);
if (loginSuccess) {
  // 执行登录成功逻辑
} else {
  // 提示用户存储失败，请检查浏览器设置或存储空间
}

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