async/await 如何优美的处理异常？

本着一种学习的态度写了此话题,如有不妥之处还请各位大牛多多指点.

Purpose: 此解决方案,可以避免业务代码中出现大量的try/catch, 众所周知try/catch对性能方面有一定的影响, 另一方面try/catch在Node.js >= v8.3.0以后对性能的损耗是可以忽略不计的.

try/catch具体文献: However for Node 8.3+ the performance hit of calling a function inside a try block is negligible.

1. try/catch方法处理异常

async updateArticleById(ctx, next){
		 let id = ctx.params["id"];
       let body = ctx.request.body;
       try {
           let ret = await Services.admin.updateArticle(id, body);
           ctx.body = {ret: 0, data: ret};
       } catch(e) {
           ctx.body = {ret: 1, data: null, err: e.message || e.stack};
       }
}

以上捕获异常是使用try/catch的方式来处理，因为await后面跟着的是Promise对象，当有异常的情况下会被Promise对象的内部
catch捕获，而await就是一个then的语法糖，并不会捕获异常, 那就需要使用try/catch来捕获异常，并进行相应的逻辑处理。

2. 封装异常处理函数

知道了上面异常会被Promise对象自身的catch捕获异常，可以使用下面的解决方案 to.js

module.exports = (promise) => {
   if(!promise || !Promise.prototype.isPrototypeOf(promise)){
       return new Promise((resolve, reject)=>{
           reject(new Error("requires promises as the param"));
       }).catch((err)=>{
           return [err, null];
       });
   }
   return promise.then(function(){
       return [null, ...arguments];
   }).catch(err => {
       return [err, null];
   });
};

格式 [error, …result]

采用类似Golang风格的错误返回方式, 这里指定第一个参数为错误参数，后面为正常返回结果
if块是用来处理非法参数，并返回错误[err, null]
await后面如果是一个promise对象，那么await的任务就是在等待promise.resolve，而to.js就是主动去调用then和catch，主动处理并重新封装结果，并且在then或是catch里面继续返回封装后的数据，返回值对于await来说仍然是一个promise对象，然而resolve的值却是一个可迭代的对象[error, …result]

这个可迭代的对象如何使用 ?

async updateArticleById(ctx, next){
       let body = ctx.request.body;
       let id = ctx.request.params["id"];
       let [err, ret] = await ctx.app.utils.to(Services.admin.updateArticleById(id, body));
       if(err) {
           return ctx.body = {ret: 1, data: null, err: err};
       }

       ctx.body = {ret: 0, data: ret};
   }

参考:how-to-write-async-await-without-try-catch-blocks-in-javascript

yuu2lee4 1楼•7 个月前

arguments可直接用扩展运算符 ctx.app.utils.to(Services.admin.updateArticleById(id, body));没加await

danielsss 2楼•7 个月前作者

@yuu2lee4 我以为arguments不会和array一样直接使用扩展运算符. 非常感谢，已经改正

151263 3楼•7 个月前

新版本的V8引擎 try/catch 对性能没有影响

danielsss 4楼•7 个月前作者

@151263 确实，我上面也说了！

brickyang 5楼•7 个月前

try/catch 和 if(err) 都不优雅，满屏捕获代码看着眼花，还是尽量在产生异常的地方处理掉……

axetroy 6楼•7 个月前

可以使用这个, 参考了go的defer，在defer中捕获错误，这样也可以消除try catch

const godefer = require('godefer');

const getUserInfo = godefer(async function(name, defer){
  defer(function(err){
  	if (err){
		console.error(err);
	}
  })
  
  // ...
  await doSomthingAsync();
  await doAnotherSomthingAsync();
})

当前只是看起来好看一些，内部还是用try catch

JsonSong89 7楼•7 个月前

不太明白,try catch 不是很自然的做法吗? 用if判断,如果漏掉呢?如果忘记return呢?

danielsss 8楼•7 个月前作者

@178220709

目前Node.js < 8.3.0 try/catch 是多多少少会有性能影响
如果你忘记return那返回的结果就是第二次set body的值
如果你不想使用if，那么你完全可以在to.js中的then和catch中直接处理掉

return promise.then(function(){
       ctx.body = {...}//normally handle
	   //return [null, ...arguments];
   }).catch(err => {
       ctx.body = {...}//exception handle
       //return [err, null];
});

如果使用try/catch代码效果是下面的样子，如果你能忍受代码这么写，确实使用try/catch更舒服，所谓萝卜青菜各有所爱，我就是有强迫症的人，完全受不了这种代码风格.

async function asyncTask(cb) {  
    try {
       const user = await UserModel.findById(1);
       if(!user) return cb('No user found');
    } catch(e) {
        return cb('Unexpected error occurred');
    }

    try {
       const savedTask = await TaskModel({userId: user.id, name: 'Demo Task'});
    } catch(e) {
        return cb('Error occurred while saving task');
    }

    if(user.notificationsEnabled) {
        try {
            await NotificationService.sendNotification(user.id, 'Task Created');  
        } catch(e) {
            return cb('Error while sending notification');
        }
    }

    if(savedTask.assignedUser.id !== user.id) {
        try {
            await NotificationService.sendNotification(savedTask.assignedUser.id, 'Task was created for you');
        } catch(e) {
            return cb('Error while sending notification');
        }
    }

    cb(null, savedTask);
}

danielsss 9楼•7 个月前作者

@brickyang 确实是这样。但是如果有些业务场景确实需要将异常发送给客户端，那就避免不了这种情况.

如果实在嫌弃这种代码不够优雅，其实可以在to.js中直接response 8楼有代码示例

xiaozhongliu 10楼•7 个月前

@danielsss 非得每一步都try catch吗, 你上我项目里找找, 能找到超过5个算你赢: https://github.com/xiaozhongliu/node-api-seed

danielsss 11楼•7 个月前作者

@XiaozhongLiu 如果下面promisify(hget)出现异常，你如何处理错误呢？

async hget(key, field) {
        const value = await promisify(redis.hget)(key, field)
        try {
            return JSON.parse(value)
        } catch (e) {
            return value
        }
 }

xiaozhongliu 12楼•7 个月前

代码拉下来跑跑试试就知道了 https://github.com/xiaozhongliu/node-api-seed/blob/master/router.js