如何将同步样式代码转换为异步代码?

时间:2012-09-30 23:51:15

标签: node.js design-patterns asynchronous callback synchronous

我有一些看起来像这样的同步代码:

function bulk_upload(files, callback) { 
  for (file in files) {
   sync_upload(file); // blocks till file uploads
  }
  print('Done uploading files');
  callback();
}

我现在必须使用异步API async_upload(file, callback)而不是sync_upload(file)来执行相同操作。我有各种选择但不确定什么是最好的:

1)在for循环之后使用sleep - 这是一个黑客,因为我不得不希望我的时间是正确的

2)递归链接我的数组:

function bulk_upload(files, callback) {
  if (files.length == 0) {
    print('Done uploading files');
    callback();
  } else {
    async_upload(files.removeLast(), function() { bulk_upload(files, callback); });
  }
}

这不仅是hacky而且是次优的,因为我可以使用新的async_upload API并行上传我的文件,但我最终顺序上传。

3)使用全局计数器:

function bulk_upload(files, callback) {
  uploads = 0
  for(file in files) {
    async_upload(file, function() { uploads++; }); 
  }
  while(uploads < files.length) ; // Empty spin - this is stupid
  print('Done uploading files');
  callback();
}

4)稍微好一点的反击(但仍然很糟糕):

function bulk_upload(files, callback) {
  uploads = 0
  for(file in files) {
    async_upload(file, function() { 
      if (++uploads == files.length) { // this becomes uglier as I want to await on more stuff
        print('Done uploading files');
        callback();
      }; 
    }); 
  }
}

2 个答案:

答案 0 :(得分:3)

您可以使用async模块的forEach方法执行此操作:

function bulk_upload(files, callback) {
    async.forEach(files, async_upload(file, callback), function (err) {
        if (err) {
            console.error('Failed: %s', err);
        } else {
            console.log('Done uploading files');
        }
        callback(err);
    });
}

答案 1 :(得分:2)

继我的评论之后,看起来这个代码足以使用期货(未经测试)。

function aync_upload_promise(file) {
    // create a promise.
    var promise = Futures.promise();
    async_upload( file, function(err, data) {
        if (err) {
            // break it
            promise.smash(err);
        } else {
            // fulfill it
            promise.fulfill(data);
        }
    });
    return promise;
}

var promises = [];
for(var i=0; i<files.length; ++i )
{
   promises.push( aync_upload_promise( files[i] ) );
}

Futures
   .join( promises )
   .when( function() {
      print('Done uploading files');
      callback();
   } )
   .fail( function(err) { print('Failed :(', err); } )
相关问题