var express = require('express');
var GoogleUrl = require('google-url');
var favicon = require('serve-favicon');
var mongo = require('mongodb').MongoClient;
var app = express();
var db;
var googleUrl = new GoogleUrl({key: 'AIzaSyB5i1sdnt6HwzrvsPTBu0FbPiUZrM_BCsk'});
var PORT = 8080;
mongo.connect('mongodb://localhost:27017/url-shortener', function(err, newDb){
if(err){
throw new Error('Database failed to connect');
} else{
console.log('Successfully connected to MongoDB on port 27017');
}
db=newDb;
db.createCollection('sites', {
autoIndexID: true
});
});
app.use(favicon(__dirname+'/public/favicon.ico'));
app.get('/new/*', function(req, res){
var doc;
console.log('This is the url: '+req.params[0]);
googleUrl.shorten(req.params[0], function(err, shortUrl){
if(err){
console.log(err);
}else{
console.log(shortUrl);
}
doc = check_db(req.params[0], shortUrl, db);
});
下面的res.json语句在其他函数有可能返回值之前运行并返回一个未定义的变量。
res.json(doc);
});
app.listen(process.env.PORT, function(){
console.log('Express listening on: '+PORT);
});
function check_db(longUrl, shortUrl, db){
db.collection('sites').findOne({
'longUrl': longUrl,
'shortUrl': shortUrl
}, function(err, doc){
if(err){
console.log(err);
}if(doc){
在执行以下语句之前执行res.json语句,并且可以返回值。
console.log('This site already exists on the database');
return doc;
}else{
save_db(longUrl, shortUrl, db);
}
});
}
function save_db(longUrl, shortUrl, db){
db.collection('sites').insert({
'longUrl': longUrl,
'shortUrl': shortUrl
}, function(err, doc){
if(err){
throw err
}else{
console.log(doc);
}
db.close();
});
}
在上面的代码中,res.json语句在GET请求下面定义的函数有机会完成执行之前执行,结果是res.json返回一个未定义的变量。我知道我必须在我的应用程序中实现异步功能(潜在的承诺?),但我完全不知道如何这样做!
答案 0 :(得分:1)
回调只是函数调用中的一个参数,所以
googleUrl.shorten(req.params[0], function(err, shortUrl){
if(err){
console.log(err);
}else{
console.log(shortUrl);
}
doc = check_db(req.params[0], shortUrl, db);
});
res.json(doc);
表现得像
foo(a, b);
bar();
您对#googleUrl.shorten()的电话会立即拨打#res.json(),就像调用 #foo一样()之后会立即调用 #bar()。
您的回调函数:
function(err, shortUrl){
if(err){
console.log(err);
}else{
console.log(shortUrl);
}
doc = check_db(req.params[0], shortUrl, db);
}
异步执行,这意味着它不会中断常规控制流。当你需要推迟执行像#res.json(doc)这样的语句时,你必须确保它的执行也是从常规控制流中取出的。为此,您需要在 #check_db()和 #save_db()中接受回调参数。新功能签名如下所示:
function save_db(longUrl, shortUrl, db, callback)
和
function check_db(longUrl, shortUrl, db, callback)
然后传递一个回调函数,它接受一个参数' doc ',并执行 res.json( doc )到你的数据库功能。例如:
doc = check_db(req.params[0], shortUrl, db, function(doc){
res.json(doc);
});
注意:您只需将<回调'从 #check_db() #save_db() >
最后,一旦异步函数完成执行,就调用回调,而不是返回一个值。例如:
db.collection('sites').findOne({
'longUrl': longUrl,
'shortUrl': shortUrl
}, function(err, doc){
if(err){
console.log(err);
}if(doc){
console.log('This site already exists on the database');
callback(doc);
}else{
save_db(longUrl, shortUrl, db, callback);
}
});
我已将更改保留到 #save_db()。祝你好运!
答案 1 :(得分:0)
将check_db
和save_db
更改为异步
function check_db(longUrl, shortUrl, db, cb){
db.collection('sites').findOne({
'longUrl': longUrl,
'shortUrl': shortUrl
}, function(err, doc){
if(err){
console.log(err);
process.nextTick(function(){cb(err)});
}else if(doc){
console.log('This site already exists on the database');
process.nextTick(function(){cb(null, doc)});
}else{
save_db(longUrl, shortUrl, db, cb);
}
});
}
function save_db(longUrl, shortUrl, db, cb){
db.collection('sites').insert({
'longUrl': longUrl,
'shortUrl': shortUrl
}, function(err, doc){
db.close();
process.nextTick(function(){cb(err, doc)});
});
}
并像这样使用:
app.get('/new/*', function(req, res){
var doc;
console.log('This is the url: '+req.params[0]);
googleUrl.shorten(req.params[0], function(err, shortUrl){
if(err){
console.log(err);
}else{
console.log(shortUrl);
}
check_db(req.params[0], shortUrl, db, function(err, doc){
if(err) return res.json(err);
res.json(doc);
});
});
});