用Promise改写传统异步方法回调函数及多个请求并发等待返回

前言

上周做了一个需求,其实很简单,但也不是那么理所当然的简单。

根据后台返回的数据,在高德地图上把数据中的点根据经纬度都标注出来。

这个过程中遇到了两个问题:

  1. 数据量很大,测试数据10000条以上,单次展示。(先不要纠结着10000个点在地图上能不能展示的清楚。)
  2. 数据并不是全的,测试数据中都没有经纬度,只提供了点所在区域的adcode,这里调用高德的DistrictSearch来返回该区域中心点的经纬度。(这里后期其实做了缓存,不再每次都去获取了。但是这里假设依然需要每次都异步的去获取数据。)

高德的DistrictSearch是一个异步的回调方法,所以也就引出来接下来要描述的问题。

callback function

说两个典型的用到回调函数的场景:

const arr = [1,2,3]
arr.forEach(value => value + 1);

Array.prototype.forEach里面的就是一个callback function啊,只是,它是同步的。如果你不想用这个,比如想在forEach中提前中断掉循环,大可以用forofforin……各种的for循环来替代。

$.ajax('yourUrl', response => {
    // do whatever you want here
})

其实不用jQuery很久了,这里举这个例子,只是因为它够典型,也遇到的最多。一个异步的回调函数,这里的回调似乎无法避免,因为说实话你也不能控制它什么时候执行不是?

问题:

所以有些逻辑我要等请求数据返回了之后再做该怎么办呢?

答案:

So easy, 放在回调函数callback里面嘛!

结果:

asyncFuncion1(function() {
  asyncFuncion2(function() {
    asyncFuncion3(function() {
      asyncFuncion4(function() {
        //...infinite
      });
    });
  });
});

That's cool, no problem?

Of course not!!! 这就是传说中的“回调金字塔”“回调黑洞”

前言中的问题,用callback来解

首先,看一下第一版代码:

// step 1: get all marker first
for (const element of data) {
  for (const value of element.details) {
    // if neither longitude nor latitude, get them from adminAreaCode
    let { longitude, latitude } = value;
    const { adminAreaCode } = value;
    if ((!longitude || !latitude) && adminAreaCode) {
      const districtSearch = new IMAP.DistrictSearch();

      districtSearch.search(adminAreaCode, (status, result) => {
        // return status is string 0, WTF!!!!!!
        if (status === '0') {
          const paths = result.results;

          const resultPath = paths[0];

          const { center: { lng, lat } } = resultPath;
          if (lng && lat) {
            addMarker(longitude, latitude);
          }
        } else {
          // return error
        }
      });
    } else if (longitude && latitude) {
      addMarker(longitude, latitude);
    }
  }
}

// step 2: mark the marker on the map
// .......

看到这里,先思考下上面的代码能不能满足需求呢?

这样写明显是有问题的:

首先,地图不太可能取回来了一个点就标记一次,所以step 2里面的代码是没办法写到每一次执行的回调里面去的。

所以这里就要等待所有的异步执行完毕,那么如何等待呢?

设置一个全局变量,然后setInterval轮询?或者在每次异步方法的回调函数里面去判断?

来看看修改后的代码,我似乎并没这么写过,为了说明这个问题写了下:

// step 1: get all marker first
for (const element of data) {
  for (const value of element.details) {
    // if neither longitude nor latitude, get them from adminAreaCode
    let { longitude, latitude } = value;
    const { adminAreaCode } = value;
    if ((!longitude || !latitude) && adminAreaCode) {
      const districtSearch = new IMAP.DistrictSearch();

      // callbackCount plus 1 when async callback function runs
      callbackCount = callbackCount + 1;

      districtSearch.search(adminAreaCode, (status, result) => {
        // callbackFinishCount plus 1 when async callback function returns        
        callbackFinishCount = callbackFinishCount + 1;
        if (callbackCount === callbackFinishCount) {
            // step 2: mark the marker on the map
            // .......
        }

        // return status is string 0, WTF!!!!!!
        if (status === '0') {
          const paths = result.results;

          const resultPath = paths[0];

          const { center: { lng, lat } } = resultPath;
          if (lng && lat) {
            addMarker(longitude, latitude);
          }
        } else {
          // return error
        }
      });
    } else if (longitude && latitude) {
      addMarker(longitude, latitude);
    }
  }
}

写这代码的我?过来,保证不打死我。。。

把callback function 改写成Promise

刚刚的“回调金字塔”能不能解? Promise是一个很好的选择。第一步,用Promise重写刚刚的callback函数:

const searchPromise = new Promise((resolve, reject) => {
  districtSearch.search(adminAreaCode, (status, result) => {
    // return status is string 0, WTF!!!!!!
    if (status === '0') {
      const paths = result.results;

      const resultPath = paths[0];

      const { center } = resultPath;
      resolve(center);
    } else {
      reject(result);
    }
  });
});

searchPromise就是通过new Promise创建的Promise对象,到这里,我们就有了一个处于pending状态的Promise对象。

Promise & async await

这一个部分,首先强烈推荐ES7语法中的async & await,这两个方法出现,极大的简化了Promise的写法,当然依然可以选择Promise.resolve或者Promise.then来写。开心就好啦。async & await,以后有空会来详细介绍的,这里可以跟着链接了解下先。

兼容性babel已经支持,不要犹豫,赶紧尝试使用这项新特性吧。

定义好searchPromise之后就是调用它了:

getMarkers = async (data = []) => {
  const markers = [];

  const opts = new IMAP.MarkerOptions();

  const addMarker = (longitude, latitude) => {
    const lnglat = new IMAP.LngLat(longitude, latitude);
    const marker = new IMAP.Marker(lnglat, opts);
    markers.push(marker);
  };

  for (const element of data) {
    for (const value of element.details) {
      // if no longitude nor latitude, get them from adminAreaCode
      let { longitude, latitude } = value;
      const { adminAreaCode } = value;
      if ((!longitude || !latitude) && adminAreaCode) {
        const districtSearch = new IMAP.DistrictSearch();

        const searchPromise = new Promise((resolve, reject) => {
          districtSearch.search(adminAreaCode, (status, result) => {
            // return status is string 0, WTF!!!!!!
            if (status === '0') {
              const paths = result.results;

              const resultPath = paths[0];

              const { center } = resultPath;
              resolve(center);
            } else {
              reject(result);
            }
          });
        });
        const { lng, lat } = await Promise.resolve(searchPromise);
        longitude = lng;
        latitude = lat;
      }
      if (longitude && latitude) {
        addMarker(longitude, latitude);
      }
    }
  }
  return markers;
};

写法一下子就简单了。async function,和同步方法无异的写法。注意下asyncawait的用法。

但是这个依然是有问题的,因为,每一次的请求都等待着上一次的请求完成,回到开始的前言,10000个点依次执行,简直不敢想。

一个优化,Promise.all

本来这篇文章只是想说说回调如何改写成Promise的,结果写的比与集中要多,这里写一个小小的优化吧。

先创建promise对象,然后把这些promise对象放入一个数组,用Promise.all去调用它,这样所有请求就是异步,然后等所有请求返回结果之后再去地图上标点。

getMarkers = async (directoryData = []) => {
  const markers = [];

  const opts = new IMAP.MarkerOptions();

  const addMarker = (longitude, latitude) => {
    const lnglat = new IMAP.LngLat(longitude, latitude);
    const marker = new IMAP.Marker(lnglat, opts);
    markers.push(marker);
  };

  const searchPromises = [];

  for (const element of directoryData) {
    for (const value of element.behaviorDetails) {
      // if no longitude nor latitude, get them from adminAreaCode
      const { longitude, latitude } = value;
      const { adminAreaCode } = value;
      if ((!longitude || !latitude) && adminAreaCode) {
        const districtSearch = new IMAP.DistrictSearch();

        const searchPromise = new Promise((resolve, reject) => {
          districtSearch.search(adminAreaCode, (status, result) => {
            // return status is string 0, WTF!!!!!!
            if (status === '0') {
              const paths = result.results;

              const resultPath = paths[0];

              const { center } = resultPath;
              resolve(center);
            } else {
              reject(result);
            }
          });
        });
        searchPromises.push(searchPromise);
      } else if (longitude && latitude) {
        addMarker(longitude, latitude);
      }
    }
  }

  const results = await Promise.all(searchPromises);
  for (const result of results) {
    const { lng, lat } = result;
    addMarker(lng, lat);
  }

  return markers;
};

Next

异步方法其实还能用Generator函数来写,这里就不废话了,下次写一篇专门的文章来介绍吧。

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

推荐阅读更多精彩内容

  • 弄懂js异步 讲异步之前,我们必须掌握一个基础知识-event-loop。 我们知道JavaScript的一大特点...
    DCbryant阅读 2,681评论 0 5
  • 异步编程对JavaScript语言太重要。Javascript语言的执行环境是“单线程”的,如果没有异步编程,根本...
    呼呼哥阅读 7,283评论 5 22
  • 欢迎阅读专门探索 JavaScript 及其构建组件的系列文章的第四章。 在识别和描述核心元素的过程中,我们还分享...
    OSC开源社区阅读 1,139评论 1 10
  • Promise 对象 Promise 的含义 Promise 是异步编程的一种解决方案,比传统的解决方案——回调函...
    neromous阅读 8,675评论 1 56
  • 本文首发在个人博客:http://muyunyun.cn/posts/7b9fdc87/ 提到 Node.js, ...
    牧云云阅读 1,675评论 0 3