Node.jsコマンドが間違えた場合メッセージを送信

Question

前提・実現したいこと

Discord.jsでコマンドが存在しなかった時にコマンドが存在しません。とメッセージを送信させたいです。

発生している問題・エラーメッセージ

なし

###ソースコード

const discord = require("discord.js");
const client = new discord.Client();
const prefix = ‘.’

client.on('command_error', () => {
  if ('error') return CommandNotFound
    message.channel.send(
        {embed: {
    color: 0x0fc0303,
    description: 'コマンドが存在しません'
  }}
);
})

client.on("message", async message => {
  if (message.content.startsWith(`${prefix}ping`)) {
message.channel.send(‘ping’)
  }}
  );

client.login(process.env.TOKEN);

###試してみたらエラーが出ました

Answer 1

client.on(String, function)

ドキュメントのEvent項目を見る限り、"command_error"というイベントは存在しないですね。

私はDiscord.js自体は利用していなくてドキュメントを見ただけの推測ですが、
onメソッド使っている所からNode.jsに提供されているEvents、またはEventEmitterを利用しながら
Discord.jsの開発者陣が用意したEvent処理の仕組みですね。
(イベント定義して待つ仕組みを作る場合、開発者がこの流れに乗っからない理由はない)

なので、都合よく開発者陣が"command_error"なるイベントを作っているなら使えますが……
まぁ無いですね。

そもそも「コマンド」だなんて概念はDiscord.jsにはありません。
BOTを作っている質問者さんが「受け取ったメッセージを解析して、自作のルールに一致する文字列をコマンド扱いにしよう！」と言ってるだけの話ですからね。
仮に同名のイベントが存在していたとして、質問者さんの意図通りの動きをするとは思えません。

---

さて、上記を踏まえつつどうすれば良いか？

質問者さんがコマンドと呼んでいるものは、
全てmessageイベントで受け取っているはずです。

DiscodeというSNSに接続するNode.jsのクライアントは、
参加者の発言を受け取ったら兎に角messageイベントを叩いて、client.on("message")で受けるような実装になっているはずですからね。

client.on("message", async message => {
  // メッセージを解析してpingという文字列を抜き出してコマンドが存在したと言っている
  if (message.content.startsWith(`${prefix}ping`)) {
message.channel.send(‘ping’)
  }}
);

この中でコマンドが存在するか否かを判別するしかありません。
私だったらこんな感じの実装するという体で進めてみましょうか。

// キー名: 関数という形式でコマンド一覧を定義
const commands = {
  ping: message => {
    // pingにはpongで返すのがお約束なのでこちらに変更
    message.channel.send("pong");
  },
  // 配列一覧を展開して返すだけのしょぼいコマンド付け足してみた
  show: (message, args) => {
    message.channel.send(JSON.stringify(args));
  }
};

client.on("message", async message => {
  if (!message.content.startsWith(prefix)) return;
  const [command, ...args] = message.content.replace(prefix, "").split(" ");

  // 上記のcommandsで定義されているキーで無ければ
  // コマンドが存在しないものとして処理
  if (commands[command] == null) {
    // 質問文冒頭のエラー処理部分をコピペしてきた
    message.channel.send({
      embed: {
        color: "0x0fc0303",
        description: 'コマンドが存在しません'
      }
    });
    return;
  }

  commands[command](message, args);
});

• 関係のない状況になったら即returnで逃げる早期リターン
• String.replaceでプレフィクスを除去
• String.splitを使って、半角スペースでぶった切りつつ配列に変換
• 分割代入で配列をコマンド部と引数部に分解

この流れは半角スペースを引数の区切りとして使いたくない場合に禍根になるので
やりたい内容からするとノイズになるから今回の回答からは省いているけど、工夫の余地はあるね。

• val == nullはnullやundefined値を見分ける時によく使うイディオム

Comments

[akkiyy]

試してみるとエラーが出ました。

[akkiyy]

写真ここで送らないんで質問に出します

[miyabi-sun]

message.channelは文字列じゃないですね
message.content.startsWithに修正しました

[akkiyy]

出来ました！ありがとうございます!!