Promise写法问题
发布于 9 年前 作者 im-here 5493 次浏览 最后一次编辑是 8 年前 来自 问答

假如我有一个查询用户数据的需求,逻辑是这样的:先从缓存查,如果缓存里没有,再从数据库查,我用promise实现如下:

exports.getUserData=function(){
	return redis.getAsync().then(function(val){
		if(val)
			return db.findFromDB(sql).then(function(val){
				return val;
			});
		else return val;
	}).catch(function(error){
		//....
	});
}

我怎么感觉我这样的写法和多层callback没区别了,是我写法不对吗?

7 回复

你这写法不能说不对,但是可读性有点尴尬。。。可以这样组织逻辑

Promise.resolve()
.then(function(){
	return redis.getAsync();
})
.then(function(val){
	if(val) return db.findFromDB(sql);
	return val;
})
.then(function(val){
	//TODO: sth
})
.catch(function(err){
	//TODO: err
});

@lgyhitler 这样的话 上面的代码能封装成一个函数么? 我试了下 假如上面代码封装函数名为getUserData(),应该如何调用? 其实我要的就是上面的代码是一个函数,另外一个函数调用该函数,拿到该函数返回的值做输出!

你应该是没搞明白Promise得真正意图,整个流程都是异步的所以会出现层层callback,Promise相当于用一种封装的形式来代替callback,每个then都是异步的,但是每个then()返回的都是一个Promise对象,这样的话就允许实现类似链式调用的方式进行流程控制。你若是想将上面的封装成一个函数,那就调用它的方式还得是Promise。 比如上面的封装成 函数getUserData,那调用方式就是:

function Fn(){
	Promise.resolve()
			.then(function(){
				return getUserData();
			})
			.then(function(userData){
				//TODO: userData
			})
}

@lgyhitler 我这样封装了下,应该没问题吧? 测了下 ,运行正常!

function Fn(){
  return Promise.resolve()
  .then(function(){
	  return redis.getAsync();
  })
  .then(function(val){
	  if(val) return db.findFromDB(sql);
	  return val;
  });
}
//调用
Fn().then(function(val){
	//输出结果
}).catch(function(error){
	//处理错误
})

是的,没错

@imhered redis.getAsync() 是返回 Promise 么? 如果是的话, 我觉得其实就不需要再写一遍 Promise.resolve() 了. Fn 直接从 return redis.getAsync() 开始就行.

另外, 你的 if(val) 是不是写反了? 有缓存的时候去执行 sql ?

@Chunlin-Li 嗯,是的。代码已经改过了,直接从return redis.getAsync()开始的,if(val)这个确实是写反了,不过只是提问的时候写反了的

回到顶部