JavaScriptの非同期処理？の方法について。

Question

Discordのボットを作ろうとしており、スチームIDを元に画像を取得した後にメッセージを送信するようにしたいのですがうまくいきません。
おそらくPromiseをうまく使うことができれば解決すると思うのですが、その方法がわかりません。どなたかご教授お願いいたします。
また、Promise, resolve のあたりが煩雑になっているのでこれをasyncを使って解決できるのであればそちらも教えていただきたく思います。

require('dotenv').config();

const SteamAPI = require('steamapi');
const steam = new SteamAPI('API_KEY');
const Discord = require('discord.js');
const client = new Discord.Client();

const chid_main = 'CHANNELL_ID';

function heyheyhey() {
    steam.getUserSummary("USER_ID").then(summary => {
        console.log('getsummary');
        var content = {
            embed: {
                author: {
                    name: "Author name",
                    icon_url: summary.avatar.small
                },
                description: 'description'
            }
        };
        console.log('set url');
        return content;
    });
}
client.on('ready',() => {
    var promise = new Promise(function (resolve, reject) {resolve(heyheyhey());})
    promise.then((content) => {
        client.channels.cache.get(chid_main).send(content);
        console.log('send message');
    })
    
})

client.login();

実行結果

send message
getsummary
set url
(node:2364) UnhandledPromiseRejectionWarning: DiscordAPIError: Cannot send an empty message
    at RequestHandler.execute (C:\Users\kent\AppData\Roaming\npm\node_modules\discord.js\src\rest\RequestHandler.js:154:13)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
    at async RequestHandler.push (C:\Users\kent\AppData\Roaming\npm\node_modules\discord.js\src\rest\RequestHandler.js:39:14)
(Use `node --trace-warnings ...` to show where the warning was created)
(node:2364) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or 
by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:2364) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Comments

[m.ts10806]

質問タグあります。
https://teratail.com/tags/Discord

Answer 1

単純にheyheyhey関数に戻り値(return)が無いだけなのでは？

蛇足ですが、僕が書くなら以下のようにしますかね。（動作確認まではしていませんが）
極端な話、asyncもPromiseオブジェクトをreturnしているのでたいした書き方に変わりはないですよ。

client.on('ready', () => {
    steam
        .getUserSummary("USER_ID")
        .then(summary => {
            console.log('getsummary');
            var content = {
                embed: {
                    author: {
                        name: "Author name",
                        icon_url: summary.avatar.small
                    },
                    description: 'description'
                }
            };
            console.log('set url');
            return content;
        })
        .then(content => {
            client.channels.cache.get(chid_main).send(content);
            console.log('send message');
        });
});

--
追記

async function heyheyhey() {
    return steam.getUserSummary("USER_ID").then(summary => {
        console.log('getsummary');
        var content = {
            embed: {
                author: {
                    name: "Author name",
                    icon_url: summary.avatar.small
                },
                description: 'description'
            }
        };
        console.log('set url');
        return content;
    });
}

client.on('ready',() => {
    heyheyhey().then(content => {
        client.channels.cache.get(chid_main).send(content);
        console.log('send message');
    });
});

Comments

[Avent]

return cotent; の直前で console.log(content) とすると中身はきちんと入っているんですよね...
実行結果でsend message が先に来ているので、非同期処理がきちんとできていないからだと思うのですが、どのようにすればよいのでしょう？

[deo_deo]

あーなるほど。実行結果を見ていませんでした。失礼。
それはheyheyhey関数の中身が非同期処理だからsend messageが先に出力されているだけです。
(厳密にはgetUserSummaryが非同期)
ドキュメント
https://github.com/xDimGG/node-steamapi
こちらを見ましたが、getUserSummaryメソッドは非同期関数になってます（Promiseを返している）。
なので、heyheyhey関数が終了するのを待たずに完了したと評価されてresolveが即時に実行されpromise.thenが先に実行されているということです。

上記に書いたようにclient.on('ready')の中に処理が書けない理由があるのであれば以下にすれば期待通りの結果になると思います。

追記しました。