我试图围绕Node.js和一些异步操作。在下面的代码中,我获取了一些RSS提要并存储了以前找不到的文章。代码工作并存储新文章。但是,我不知道如何改变这一点,以便我知道所有文章何时完成解析。例如,每次满足限制时(每10篇文章或5篇文章后)都会调用每个async.eachLimit
的回调。那么我怎么知道它们何时完成?
var FeedParser = require('feedparser');
var request = require('request');
var mysql = require('mysql');
var async = require('async');
var connection = mysql.createConnection({
host : 'localhost',
user : 'someuser',
password : 'somepass',
database : 'somedb'
});
connection.connect();
connection.query('SELECT * FROM rssfeed', function(err, rows, fields) {
if(err == null){
async.eachLimit(rows, 5, parseFeed, function(err) {
if(! err) {
//horray
} else {
console.log(err);
}
});
}
});
function parseFeed(feed, callback) {
var articles = [];
request(feed.link)
.pipe(new FeedParser())
.on('error', function(error) {
callback(error);
})
.on('meta', function(meta) {
})
.on('readable', function() {
var stream = this, item;
item = stream.read();
if(item != null) {
item.rssfeed_id = feed.id;
articles.push(item);
}
})
.on('finish', function() {
async.eachLimit(articles, 10, parseArticle, function(err) {
if(! err) {
console.log('article each callback');
} else {
callback(error);
}
});
});
callback();
}
function parseArticle(item, callback) {
if(item.hasOwnProperty('rssfeed_id') && item.hasOwnProperty('guid') && item.hasOwnProperty('link') && item.hasOwnProperty('title')){
connection.query('SELECT * FROM rssarticle WHERE rssfeed_id = ? AND guid = ?', [item.rssfeed_id, item.guid], function(err, rows, fields) {
if(rows.length == 0){
connection.query('INSERT INTO rssarticle SET ?', {
rssfeed_id: item.rssfeed_id,
link: item.link,
title: item.title,
description: item.description,
publish_date: item.pubDate,
guid: item.guid
}, function(err, result){
if(err != null){
console.log(err);
}
});
}
});
}
callback();
}
答案 0 :(得分:6)
首先,你过早地预先调用你的回调。
function parseFeed(feed, callback) {
request
.streamStuff()
.streamStuff()
.streamStuff();
callback();
}
在完成之前,您不应该致电callback
。否则,将调用您的“已完成”方法,但您的异步代码实际上仍将运行。
所以不要这样做:
.on('finish', function() {
async.eachLimit(articles, 10, parseArticle, function(err) {
if(! err) {
console.log('article each callback');
} else {
callback(error);
}
});
});
callback();
只做
.on('finish', function() {
async.eachLimit(articles, 10, parseArticle, function(err) {
if(! err) {
// assuming this is a stub and really ends up doing `callback();`
console.log('article each callback');
} else {
callback(error);
}
});
});