错误封装axios配置,导致响应出现叠加

业务需求

公司有多个项目共同用到了已封装好的axios(funcation http()),基于去掉复制粘贴这种重复工作。

  • 统一封装所有项目axios方法(\color{red}{本文问题重点}
  • 把封装好的方法打包发布到公司npm私服(不涉及)
问题重现

Vue项目页面Create(){}的时候调用了一个接口,相当于调用了http()方法,此时响应response还是正常的。
但是当再调用任何接口的时候,response出现了多次(我在响应拦截器中console了response,调用了几次http(),就多出现几次console)

// axios 封装的方法(关键代码)
// httpClient是axios重命名、因为每个项目的baseURL不一样,所以每次需要传一个base进来、url是接口地址
export default function http(base, url, data = {}, type = "get", header = {}) {
  httpClient.defaults.baseURL = base;
  httpClient.interceptors.response.use(
    response => {
      console.log("response", response);
      return response
    },
    error => {
      console.log("error", error.response);
      if (error.response.status === 401 && error.response.data.code === 'nosso') {
        window.location.href = base + '/redirect/to/frontend?page=' + encodeURIComponent(window.location.href)
      }
      return error.response
    }
  );
  type = type.toLowerCase();
  let promise;
  return new Promise((resolve, reject) => {
    if (type === 'post') {
      promise = httpClient.post(url, data, {headers: header})
    } else if (type === 'put') {
      promise = httpClient.put(url, data, {headers: header})
    } else if (type === 'delete') {
      promise = httpClient.delete(url, {data: data, headers: header})
    } else {
      promise = httpClient.get(url, {params: data, headers: header})
    }
    promise.then(response => {
      resolve(response.data);
    }).catch(error => {
      reject(error)
    })
  })
}
Create后--正常情况
第一次调用退出方法--错误情况
第二次调用退出方法--错误情况

如果再点退出,response的数量会继续累加。当时看到这个情况后很不理解。首先能确定的是两个请求格式和参数完全一致,那么问题只能在响应和后端数据上了(排除后台数据问题)。
百度了一圈,基本没有发现这个问题的解决办法(本来是没有问题的,这个封装的方法最后是由我自己改动的。详细改动:将baseURL和响应拦截器放在了http()里面,就出现以上错误。之所以会放在里面是因为base位置问题,后面细说)。

为什么会出现这个问题 && 问题原因

项目以前http()的响应拦截器里的地址是写死的,封装后,需要根据不同的项目给不同的请求前缀地址。而axios默认的请求配置是在发生请求之前定义的。

原项目axios逻辑顺序

如图,调用http()的时候,需要将一个base传入baseURL和响应拦截器里面。因为是配置项,有点违背代码逻辑,所以将baseURL和响应拦截器放在http()里面,如下图:

有问题的axios逻辑顺序

回到上面的问题,我猜测把响应拦截器放在http()后,因为每次都有个new Promise的过程,相当于在项目里,每调一次http(),axios就多一个响应拦截器配置,每个响应都依次依赖,所以才导致调用一个接口,就多一个response结果。

解决

找到问题就好解决,最开始没有想到,httpClient.default.baseURL可以当做axios服务的全局变量。也就是说,我只需要把这个default.baseURL放在http()里面,响应拦截器放在外面就可以了

httpClient.interceptors.response.use(
  response => {
    return response.data
  },
  error => {
    if (error.response.status === 401 && error.response.data.code === 'nosso') {
      window.location.href = httpClient.defaults.baseURL + '/redirect/to/frontend?page=' + encodeURIComponent(window.location.href)
    }
    return error.response
  }
);
/**
 * 封装请求方法
 * @params(url)
 * @params(data)
 * returns { Promise }
 * @user NIGangJun <nigangjun@aeotrade.com>
 */
export default function http(base, url, data = {}, type = "get", header = {}) {
  httpClient.defaults.baseURL = base;
  type = type.toLowerCase();
  let promise;
  return new Promise((resolve, reject) => {
    if (type === 'post') {
      promise = httpClient.post(url, data, {headers: header})
    } else if (type === 'put') {
      promise = httpClient.put(url, data, {headers: header})
    } else if (type === 'delete') {
      promise = httpClient.delete(url, {data: data, headers: header})
    } else {
      promise = httpClient.get(url, {params: data, headers: header})
    }
    promise.then(response => {
      resolve(response);
    })
  })
}

就是把响应拦截器放在外面,然后base替换成baseURL就行了。

有个优化的地方:响应拦截器resolve的数据换成了response.data,reject的数据换成了error.response。promise就直接去掉了catch(如果要catch,响应拦截器中reject的数据要return Promise.reject(error.response),并且http()方法里不能再使用 new Promise封装)

总结

虽然最后解决结果很简单,但是中间这个过程不可忽略。

  • axios配置应该符合规范;
  • 查找代码问题应该按照功能进行拆分,依次查询;

问题千变万化,解决思路如出一辙——用心;

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 158,560评论 4 361
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 67,104评论 1 291
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 108,297评论 0 243
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 43,869评论 0 204
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 52,275评论 3 287
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 40,563评论 1 216
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 31,833评论 2 312
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 30,543评论 0 197
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 34,245评论 1 241
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 30,512评论 2 244
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 32,011评论 1 258
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 28,359评论 2 253
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 33,006评论 3 235
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 26,062评论 0 8
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 26,825评论 0 194
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 35,590评论 2 273
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 35,501评论 2 268