Обещания с fs и bluebird

В настоящее время я изучаю, как использовать обещания в nodejs

поэтому моей первой задачей было перечислить файлы в каталоге, а затем получить содержимое каждого с помощью обоих шагов, используя асинхронные функции. Я придумал следующее решение, но у меня есть сильное ощущение, что это не самый элегантный способ сделать это, особенно в первой части, где я «превращаю» асинхронные методы в обещания.

// purpose is to get the contents of all files in a directory
// using the asynchronous methods fs.readdir() and fs.readFile()
// and chaining them via Promises using the bluebird promise library [1]
// [1] https://github.com/petkaantonov/bluebird 

var Promise = require("bluebird");
var fs = require("fs");
var directory = "templates"

// turn fs.readdir() into a Promise
var getFiles = function(name) {
    var promise = Promise.pending();

    fs.readdir(directory, function(err, list) {
        promise.fulfill(list)
    })

    return promise.promise;
}

// turn fs.readFile() into a Promise
var getContents = function(filename) {
    var promise = Promise.pending();

    fs.readFile(directory + "/" + filename, "utf8", function(err, content) {
        promise.fulfill(content)
    })

    return promise.promise
}

Теперь свяжите оба обещания:

getFiles()    // returns Promise for directory listing 
.then(function(list) {
    console.log("We got " + list)
    console.log("Now reading those files\n")

    // took me a while until i figured this out:
    var listOfPromises = list.map(getContents)
    return Promise.all(listOfPromises)

})
.then(function(content) {
    console.log("so this is what we got: ", content)
})

Как я писал выше, он возвращает желаемый результат, но я уверен, что есть более элегантный способ.


node.js javascript asynchronous fs bluebird
person Henrik    schedule 17.10.2013    source источник

Ответы (1)


arrow_upward
46
arrow_downward

Код можно сделать короче, используя общее обещание и Метод .map:

var Promise = require("bluebird");
var fs = Promise.promisifyAll(require("fs")); //This is most convenient way if it works for you
var directory = "templates";

var getFiles = function () {
    return fs.readdirAsync(directory);
};
var getContent = function (filename) {
    return fs.readFileAsync(directory + "/" + filename, "utf8");
};

getFiles().map(function (filename) {
    return getContent(filename);
}).then(function (content) {
    console.log("so this is what we got: ", content)
});

На самом деле вы могли бы сократить это еще больше, так как функции больше не тянут свой вес:

var Promise = require("bluebird");
var fs = Promise.promisifyAll(require("fs")); //This is most convenient way if it works for you
var directory = "templates";

fs.readdirAsync(directory).map(function (filename) {
    return fs.readFileAsync(directory + "/" + filename, "utf8");
}).then(function (content) {
    console.log("so this is what we got: ", content)
});

.map должен быть вашим методом хлеба с маслом при работе с коллекциями - он действительно мощный, поскольку он работает для чего угодно, от промисов до массива промисов, которые сопоставляются с дальнейшими обещаниями, до любого сочетания прямых значений между ними.

person Esailija    schedule 20.10.2013
comment
Приятно также отметить, что будущие читатели должны использовать функции Async: readdirAsync, readFileAsync. - person Wtower; 09.08.2016