js(aws sdk)の非同期処理がうまく動作しない

Question

javascriptのAWS SDKを使用してDynamoのアイテムを非同期で取得したい

タイトルの通り、javascrtipt(Node.js)を使用してDynamoDBの値を取りに行く処理を書いています。
非同期処理、Javascriptに不慣れで以下のことについて教えてください。

教えていただきたいこと

下に期待通りに動く場合と動かない場合の２パターンのコードを載せました。

どちらのコードもawaitで処理を止めいているところに、promiseを返しているのでどちらも動くのではと思っているのですが、なぜか動かない場合のコードのmain.jsでawaitが動作していないように見えます。（mainのdb_resultがundefinedになってしまう。）なぜでしょうか。

非同期処理を初めて触るのであまりよくわかっておらず、awaitではpromiseが返ってくるまで処理を止めるという認識をしていてこのコードを書いています。この認識が間違いでしょうか。

また、今回のようにモジュールを分割（ファイルを分ける：main.jsとdynamo.js）してそれぞれの関数の中で処理を実施、すなわち

mainの中の関数f()
 +-- dynamo.jsの中の関数g()
      +-- AWS SDKの関数

の場合で、下の関数の結果が上の関数の呼び出し元で必ず必要となる場合はどのように記述すればよろしいでしょうか。

よろしくお願いいたします。

正しく動作する場合

//dynamo.js
var docClient = new AWS.DynamoDB.DocumentClient();
exports.dbRead = async function(param){
    var resultJSON = await docClient.get(param).promise();
    if (!Object.keys(resultJSON).length) {
        throw　new Error({"error":"no items"})
    }
    return resultJSON.Item;
}

//main.js(express)
var db = require('./dynamo');

app.get('/name',async(req, res) => {
    var id = req.query.id;
    console.log(id)
    var params ={
     //省略
    }
    try{
        var db_result = await db.dbRead(params)
......

失敗する場合

main.jsは同じです。

// dynamo.js
var docClient = new AWS.DynamoDB.DocumentClient();
exports.dbRead = async function(param){
    let jsonResponse
    try{
        var resp = await docClient.get(param).promise()
        jsonResponse = resp.Item
        if(!Object.keys(jsonResponse).length){
            throw Error({"error":"no items"})
        }
        Promise.resolve(jsonResponse)
    } catch(err) {
        console.error("Error(dbread) :", err)
        Promise.reject(err)
    }
}

Answer 1

「失敗する場合」のコードでは、dbRead関数内で明示的なreturnをしていないからではないでしょうか。

初期値以外の値を返すためには、返す値を指定する return 文が関数内になくてはなりません。return 文を持たない関数は初期値を返します。コンストラクターが new キーワードとともに呼び出された場合、その this 引数が初期値となります。それ以外のすべての関数が既定で返す値は undefined です。

※関数 from MDNより引用

例えば、以下のように書けばawait dbRead()の結果としてPromise.resolve(jsonResponse)=「jsonResponseとおそらく同じ」を返すので、db_resultがundefinedにはならないとは思います。

// dynamo.js
var docClient = new AWS.DynamoDB.DocumentClient();
exports.dbRead = async function(param){
    let jsonResponse
    try{
        var resp = await docClient.get(param).promise()
        jsonResponse = resp.Item
        if(!Object.keys(jsonResponse).length){
            throw Error({"error":"no items"})
        }
-        Promise.resolve(jsonResponse)
+        return Promise.resolve(jsonResponse)
    } catch(err) {
        console.error("Error(dbread) :", err)
        Promise.reject(err)
    }
}