Promise使用异常排查：无法读取then属性及异步调用无返回值问题

让我们一步步拆解你遇到的Promise相关问题，从根源上解决这些异步逻辑的bug：

1. 回顾最初isMock函数的问题（已修复，可进一步优化） #

你已经定位到最初的isMock函数没有返回内部的Promise实例，导致调用isMock().then()时拿到undefined，触发「Cannot read property 'then' of undefined」错误。其实你的修复版本可以更简洁，避免不必要的Promise嵌套（这是常见的Promise反模式）：

export function isMock(){
  // 直接返回getSharedPreferenceValue的Promise链，无需额外包裹new Promise
  return getSharedPreferenceValue(Globals.SHARED_PREFERENCES_KEY_IS_TEST_USER)
    .then(isTestUser => {
      consoleLog('isMock - ' + isTestUser);
      // 直接返回布尔值，Promise会自动包装为resolved状态的实例
      return isTestUser === 'true';
    })
    .catch(err => {
      consoleLog('error -' + err);
      // 抛出错误，让外部调用者能捕获到异常
      throw err;
    });
}

2. 当前login函数的核心问题：未返回Promise链 #

你的login函数内部执行了isMock().then(...)，但没有把这个Promise对象返回出去！这就导致当loginUser_legacy用await login(...)调用时，await拿到的是undefined，自然无法获取后续的返回值。

同时，else分支里的return Promise.resolve(user)其实可以简化为直接返回user——Promise的then回调中返回普通值时，会自动被包装为resolved状态的Promise实例。

修复后的login函数：

export function login (ABCD, XYZ) {
  // 关键：返回整个Promise链，让外部能等待异步操作完成
  return isMock().then((isMock) => {
    consoleLog('login Legacy API - isMock() - ' + isMock);
    if(!isMock){
      console.log('isMock is false');
      // 如果是真实登录逻辑，这里应该返回调用真实API的Promise
      // 暂时没有的话，返回一个resolved的Promise避免链中断
      return Promise.resolve(null);
    }else{
      consoleLog('mock data - ../mockData/login.json');
      const user = require('../mockData/login.json');
      // 直接返回user，自动包装为resolved Promise
      return user;
    }
  })
  .catch(error => {
    console.log('Login isMock - error - ' + JSON.stringify(error));
    // 抛出错误，让外部调用者能捕获到异常
    throw error;
  });
}

3. loginUser_legacy的冗余写法与错误处理 #

你在await login(...)之后又链式调用了.then()，这是冗余的——await已经会等待Promise完成并返回结果，直接接收即可。同时，要使用try/catch来捕获login可能抛出的错误。

修复后的loginUser_legacy：

loginUser_legacy = async (ABCD, XYZ) => {
  try {
    // 直接用await获取login返回的结果
    const res = await login(ABCD, XYZ);
    // 如果res是对象，用JSON.stringify才能打印完整内容
    consoleLog('res - ' + JSON.stringify(res));
  } catch (error) {
    // 捕获login抛出的错误
    console.log('loginUser_legacy error - ' + JSON.stringify(error));
  }
}

关键总结 #

• Promise必须返回才能被外部await/then捕获：所有异步函数如果要让外部获取结果，必须返回对应的Promise实例。
• 避免Promise嵌套反模式：不要用new Promise去包裹已经返回Promise的函数调用，直接返回原Promise链即可。
• await和then二选一：在async函数中，优先使用await配合try/catch处理异步逻辑，代码更简洁易读。

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