AngularJS缓存请求服务返回同一Promise是否错误？会引发内存泄漏吗？

嘿，这个实现思路其实挺实用的，咱们来仔细唠唠它的合理性和潜在问题：

一、这种实现方式本身是正确的吗？ #

答案是没问题，这是一种很常见的「请求防抖/共享Promise」优化方案。

核心逻辑是当多个组件或地方同时请求同一个资源时，复用同一个Promise，避免重复发起HTTP请求——这在AngularJS场景下完全合理，能有效减少不必要的网络请求，提升性能。

不过有几个细节需要注意，不然可能踩坑：

• 缓存键的准确性：要确保相同的资源对应同一个缓存键（比如你用请求URL当键的话，要考虑参数是否一致，比如/api/data?id=1和/api/data?id=2是不同资源，不能共用同一个键）。
• 失败请求的处理：如果第一次请求失败了，默认情况下后续调用会一直拿到这个失败的Promise。如果你的业务允许重试，最好在catch回调里把缓存的Promise删掉，这样下次调用会重新发起请求，比如：

  .catch(function(error) {
    delete pendingRequests[resourceKey];
    throw error;
  });
  

• 缓存的生命周期：如果你的资源是会更新的，要考虑是否需要设置缓存过期，或者提供手动清除缓存的方法，避免用户一直拿到旧数据。

二、会造成内存泄漏吗？ #

正常情况下不会有内存泄漏，但风险取决于你的缓存策略：

1. Promise本身的回收：当所有订阅这个Promise的.then()/.catch()都执行完毕，且没有其他代码持有对这个Promise的引用时，JS的垃圾回收机制会自动回收它。
2. 永久缓存的风险：如果你的缓存容器（比如服务里的pendingRequests对象）是和服务生命周期绑定的（AngularJS服务是单例，只要应用不销毁就一直存在），而且你无限期保留所有缓存的Promise/结果，那确实会积累内存占用——但这是缓存设计的选择，不是这个Promise复用方式的问题。

解决这个问题的办法很简单：

• 给缓存设置过期时间，定期清理不再使用的缓存项；
• 提供清除特定缓存或全部缓存的API，让业务代码可以主动释放资源。

举个典型的实现示例 #

app.service('resourceCacheService', function($http, $q) {
  // 存储待完成或已完成的请求Promise
  const requestCache = {};

  this.cachedRequest = function(resourceUrl, requestParams) {
    // 用URL+参数生成唯一缓存键
    const cacheKey = `${resourceUrl}_${JSON.stringify(requestParams)}`;

    // 如果已有缓存的Promise，直接返回
    if (requestCache[cacheKey]) {
      return requestCache[cacheKey];
    }

    // 发起新请求
    const requestPromise = $http.get(resourceUrl, { params: requestParams })
      .then(response => {
        // 请求成功后，把缓存替换为已resolve的Promise（可选，也可以直接存结果）
        requestCache[cacheKey] = $q.resolve(response.data);
        return response.data;
      })
      .catch(error => {
        // 请求失败时移除缓存，允许后续重试
        delete requestCache[cacheKey];
        return $q.reject(error);
      });

    // 缓存当前的请求Promise
    requestCache[cacheKey] = requestPromise;
    return requestPromise;
  };

  // 提供手动清除缓存的方法
  this.clearCache = function(cacheKey) {
    if (cacheKey) {
      delete requestCache[cacheKey];
    } else {
      // 清除全部缓存
      Object.keys(requestCache).forEach(key => delete requestCache[key]);
    }
  };
});

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