はてなjs教科書 promiseのコードの問題

[voidsatisfaction]

WHY?

https://github.com/hatena/Hatena-Textbook-JavaScript/blob/master/async/promise.md

上のリンクでは、次のようなpromiseコードが書いています。

let db;
MongoClient.connect('mongodb://localhost:27017/rssDataBase')
    .then((_db) => {
        db = _db;
        return fs.readFile('./feed.xml', 'utf8');
    })
    .then(rssString => parseString(rssString))
    .then((result) => {
        const items = result.rss.channel[0].item;
        return Promise.all(items.map(item => request(item.link[0])));
    })
    .then((responses) => {
        return responses.map((res) => {
            return {
                url  : res.request.href,
                body : res.body,
            };
        });
    })
    .then((data) => {
        return db.collection('entries').insertMany(data);
    })
    .then(() => {
        db.close();
    });

ここで、fs.readFile('./feed.xml', 'utf8')をreturnしていますが、これは、「https://nodejs.org/api/fs.html#fs_fs_readfile_path_options_callback」 の仕様によると、callback関数を第３引数としてもらっていますし、promiseを使ってラップされてないので、実装に混同する余地があり、コードを修正する必要があると思います。

parseStringという関数も同様です。

HOW?

プロミスでラップした正確なコードを書く