Node.js async使用時にreturn値がnullになってしまう

Question

表題の件でのご質問投稿となります。

以下のコードにて、DynamoDBから取得した値を成型し、戻り値として返す事を考えています。
ですがデバックをしてみると、どうもDBからアイテムをゲットする処理が動いておらず、
return値がNullになってしまいました。

asyncを使用した場合はどこにawaitを入れればいいのかも分からず、
asyncを使用しない場合は最後まで処理されますがreturn値が返ってこずで力尽きてしまいました。

const AWS = require("aws-sdk");
require('date-utils');
const DynamoDB = new AWS.DynamoDB.DocumentClient({ region: "ap-northeast-1" });

exports.handler = async (event) => {
//以下にて、DynamoDBの"No"列から、YYYYMMDD形式の本日の値をparams1に保存。
 
 var dt = new Date();
  var Todaytmp = dt.toFormat("YYYYMMDD");
    var Today = Number(Todaytmp);
    console.log(Today);
  var params1 = {
    TableName: 'TableName',
    Key: {
      'No': Today,
    }
  };

//移行処理されない。。。
//処理内容としては、上記で得たYYYYMMDDのデータをプライマリキーとしている、DynamoDBに問い合せ、その行を取得する。
DynamoDB.get(params1, function(err, data){
    if(err){
        console.log(err);
    }else{
        console.log(data);
//色々成型ここから
        var DBData = JSON.stringify(data);
        var Call1tmp = DBData.replace(/}|{|"|Item|Call2(.*)|Call1|:|,/g,'');
        var Call2tmp = DBData.replace(/}|{|"|Item|No(.*)|:|Call1(.*)Call2|,/g,'');
        var Call3tmp = DBData.replace(/}|{|"|Item|Call4(.*)|:|Call1(.*)Call3|,/g,'');
        var Call4tmp = DBData.replace(/}|{|"|Item(.*)Call4|:/g,'');
//色々成型ここまで

        console.log(Call1tmp);
        console.log(Call2tmp);
        console.log(Call3tmp);
        console.log(Call4tmp);

//戻り先の処理しやすいString形式に変換        
        var Call1 = String(Call1tmp);
        var Call2 = String(Call2tmp);
        var Call3 = String(Call3tmp);
        var Call4 = String(Call4tmp);
        console.log(Call1);
        console.log(Call2);
        console.log(Call3);
        console.log(Call4);
        return Call1;
    }    
});
};

どうしたら最後まできちんと処理されますでしょうか。

Comments

[p19ljk]

「return値がNull」とありますが、return値とはどれのことを指しますか？
このソース内では何も実行していないので、returnなどないようにみえますが。。。
これを呼び出している別のソースがあるのであれば、それも記述するとよいかと。
あと、インデントがぐちゃぐちゃなので修正してみてください。

[Ochonansan]

ありがとうございます。
今後は質問の際、インデントも整理する事を心がけます。

Answer 1

実際にDynamoDBを使ったことがあるわけではないので詳しいことは分かりませんが、DynamoDB.get(params1, function(...のfunctionはコールバック関数だと思うので、この関数内でreturnを使っても、それは親の関数exports.handlerのreturnではなく、あくまでもコールバック関数にとってのreturnにしかならないと思います。

Call1を外に出すには、Promiseとawaitでコールバック関数の返り値をキャッチする必要があるように思います。

exports.handler = async event => {
    //以下にて、DynamoDBの"No"列から、YYYYMMDD形式の本日の値をparams1に保存。
    var dt = new Date();
    var Todaytmp = dt.toFormat('YYYYMMDD');
    var Today = Number(Todaytmp);
    console.log(Today);
    var params1 = {
        TableName: 'TableName',
        Key: {
            'No': Today,
        }
    };

    // (!) PromiseとawaitでdataをJSON.stringifyしながらキャッチ
    var DBData = await new Promise((resolve, reject) => {
        DynamoDB.get(params1, (err, data) => {
            if (err) {
                console.log(err);
                reject(err);
            } else {
                console.log(data);
                resolve(JSON.stringify(data));
            }
        });
    });

    //色々成型ここから
    var Call1tmp = DBData.replace(/}|{|"|Item|Call2(.*)|Call1|:|,/g, '');
    var Call2tmp = DBData.replace(/}|{|"|Item|No(.*)|:|Call1(.*)Call2|,/g, '');
    var Call3tmp = DBData.replace(/}|{|"|Item|Call4(.*)|:|Call1(.*)Call3|,/g, '');
    var Call4tmp = DBData.replace(/}|{|"|Item(.*)Call4|:/g, '');
    //色々成型ここまで

    console.log(Call1tmp);
    console.log(Call2tmp);
    console.log(Call3tmp);
    console.log(Call4tmp);

    //戻り先の処理しやすいString形式に変換        
    var Call1 = String(Call1tmp);
    var Call2 = String(Call2tmp);
    var Call3 = String(Call3tmp);
    var Call4 = String(Call4tmp);

    console.log(Call1);
    console.log(Call2);
    console.log(Call3);
    console.log(Call4);

    return Call1;
};

（蛇足ですが、折角JSONで受け取っているので、String化して正規表現でバラすのではなく、JSONのまま加工した方が良いように思います。）

Comments

[Ochonansan]

ありがとうございます。
はい、ご指摘の通り親の関数へのreturnが行いたかったのですが、配下のコールバック関数に返そうとしているようなコードになってしまっておりました。(まだその部分の知識もままなっておりませんでした。)

ご教示頂いた通りのコードで実現したい事を達成できそうです。
また、加工の方法も自分が知っている内容に逃げてしまっていましたが、
今後の為定石通り遠回り無しの方法を考えてみます。
ありがとうございました。