[javascript / node.js]request-promiseとasync/awaitの使い方

Question

お世話になってます。
javascript (node.js) でアプリを作ろうとしており、ハマっているので質問させてください。

やりたいこと

• 画像提供サービスからAPI経由で画像情報を取得
• 取得した画像をFirebase Storageに保存

できずにいること

• 画像情報は取得できたがソースコード上で変数に代入するとundefinedとなってしまう

環境

node : v8.7.0

直接関係ないとは思いますが補足すると、最終的にFirebase Functions上で動かしたいので、
今はMacにfirebase-toolsをインストールし、ローカル（エミュレータ）で動作確認しています。

書いたソースコード

const rp = require('request-promise');

const options = {
    uri: xxx, // 省略
};

exports.api = functions.https.onRequest((request, response) => {
    async function main(){
        // 画像情報を取得
        const photoInfo = await getPhotoInfo();
        // （２）ここで変数photoInfoに(1)で取得した画像情報が入っていてほしいが、
        // undefinedとなってしまう
        console.log(photoInfo);

        // 画像をダウンロード

        // 画像をアップロード
    }

    main();

    function getPhotoInfo() {
        rp(options)
        .then(res => {
            // (1)ここでは変数resに取得した画像情報がjson形式で格納される
            let response = JSON.parse(res);
        })；
    }

    response.send("Send successfully");
});

request-promiseは処理に成功したらpromiseを返すのかと思い、photoInfo.then(res => {console.log(res)})としたのですがUnhandledPromiseRejectionWarningが出てエラーとなってしまいます。

promiseやasync/awaitの使い方を間違えているだけだと思いますが、request-promiseでawaitを使って同期的に処理を記述するやり方にたどり着けていません。

アドバイスを頂けると幸いです。よろしくお願いします。

Answer 1

プロミスチェーン内のどこかで例外が出て(多分rpかJson.parseか)、それを処理できていないというようなエラーかと思います。下記のように、例外をキャッチして、その情報を確認してみてください。
（※回答ではないのですが、コードを示しやすいので。）

    function getPhotoInfo() {
        rp(options)
        .then(res => {
            // (1)ここでは変数resに取得した画像情報がjson形式で格納される
            let response = JSON.parse(res);
        })
        // !!追加!!
        .catch(error => {
            // promiseのエラーをキャッチします
            console.log(error);
        })；
    }

######追記

次のような感じでうまくいきました。コメントの部分を見ていただければわかりますが、「プロミス」および「JSONをパースした結果」、その2つをreturnできていなかったのかなと思います。
⇒回答したコードにpromiseとasync/awatiが混在しちゃってたので、直しました。

ただ、エラーに関しては別問題だと思います。
それについてはまた別途、エラー分を質問に乗せたうえでご質問されてみては？

const rp = require('request-promise');

const options = {
    uri: 'https://api.syosetu.com/novelapi/api/?out=json',
};

// asyncがいります
async function getPhotoInfo() {
    // awaitします
    const json = await rp(options);
    const info = JSON.parse(json);
    return info;
}

async function main() {
    // awaitして画像情報を取得します
    const photoInfo = await getPhotoInfo();
    console.log(photoInfo);
}

main();

Comments

[yoshiky]

ありがとうございます！
ご提示いただいた箇所ではエラーは出なかったため、catchに入りませんでした。
JSONのパースも正しくできているようで、変数responseにパースされた値が入りました。

[退会済みユーザー]

あ、getPhotoInfoのメソッド内で、プロミスをreturnしてなくないですか？

[yoshiky]

return しても undefinedになってしまいます。。

async function main(){
// 画像情報を取得
const photoInfo = await getPhotoInfo();
// photoInfo.then(res => { console.log(res)});
console.log(photoInfo);

// 画像をダウンロード

// 画像をアップロード
}

main();

function getPhotoInfo() {
return rp(options)
.then(res => {
let response = JSON.parse(res);
})
.catch(err => {
console.log("[E001]Fail to get image info: " + err);
});
}

[退会済みユーザー]

回答に追記しておきました。

[yoshiky]

確かに両方returnすれば、awaitの戻り値でrequest-promiseのレスポンスを得ることができました！
ありがとうございました！
普通こういう使い方をしないのか、ネット上には二重にreturnするような紹介がなかったのが少しモヤっとしますが、とりあえずは助かりました！

[退会済みユーザー]

ギャーごめんなさい。見直したんですが、promiseとasync/awaitを混在させちゃってますね。直しておきます。

[yoshiky]

修正していただきありがとうございました。
まだpromiseとasync/awaitの使い方が把握できて違いがよく分かってませんが。。

些末なことですが、この程度ならgetPhotoInfoで別関数に切り分ける必要ないな、と思いましたw

[退会済みユーザー]

うまく説明できてるかわからないですが、async function xxx() {} はpromiseを返し、await somePromiseと書いたらそのプロミスの完了を待つ、という感じでしょうか。結果が必要な際はreturnしないと、何を返していいかわからないので、returnしておくという感じかと。

確かに、ちょろっとそこだけで使うのに書く分には分けなくてもよさそうですね。JSON.parse(await getPhotoInfo())と書けちゃいますし。仕事で使う際はよく考えたほうがよさそうですが。