-1

我试着做一个掷硬币的命令,用户.cf heads输入,机器人向他展示他的答案、结果以及他们是赢还是输。

我尝试使用args和不使用它,但它不起作用;我的代码中有一个错误:

bot.on('message', message => {
  const prefix = '.';
  if (!message.content.startsWith(prefix)) return;
  const args = message.content.slice(prefix.length).trim().split(/  +/g)
  const cmd = args.shift().toLowerCase();
  var choice = 'h';
  if (args[1] != undefined)
    args[1] = args[1].toLowerCase();
  if (args[1] == 'heads' || args[1] == 'h' || args[1] == 'head')
    choice = 'h';
  else if (args[1] == 'tails' || args[1] == 't' || args[1] == 'tail')
    choice = 't';
  if (cmd === 'cf' || cmd === 'coin' || cmd === 'flip' || cmd ===
    'coinflip') {
    var coins = [
      "heads",
      "tails"
    ];
    coinz = coins[Math.floor(Math.random() * coins.length)];
    if (choice != coinz) {
      message.channel.send(`Your bet: \`${args[1]}\`, 
               outcome: \`${coinz}\` you lose`);
    } else {
      message.channel.send(`Your bet: \`${args[1]}\`, 
                outcome: \`${coinz}\` you win`);
    };
  };
});

代码有效,但它给了我 100% 的失败,有时${args[1]}虽然我输入了,但它是未定义的headshhead${coinz}每次都是尾巴。

4

2 回答 2

1

回答收集我的原始评论。

关于 100% 的损失率: 要决定是赢还是输,您可以将您的选择变量与 coinz 进行比较。但是,使用您的代码,“choice”只能是“h”“t”,而“coinz”将是“heads”“tails”。这种比较总是会返回假,告诉你这是一个损失。

关于 undefined $args[1] : args [1] undefined来自您的args.shift()电话。移动一个数组会从这个数组中删除第一个元素,所以一旦你提取了“cmd”变量,你的参数现在存储在 args[0] 中。我在这里解决这个问题的方法是存储一个cmdvar asargs[0]和一个choicevar as args[1],不移动。另请注意,您仍然可能遇到错误,因为您的测试:

if (args[1] != undefined)
  args[1] = args[1].toLowerCase();

没有括号,这意味着无论条件是否通过,以下行都将运行,这意味着您将尝试访问 args[1] 即使它不存在。您应该包含所有以下代码,{ }因为它取决于args[1]变量:

client.on('message', message => {
const prefix = '.';
if (!message.content.startsWith(prefix)) return;
const args = message.content.slice(prefix.length).trim().split(/ +/g)
const cmd = args[0].toLowerCase();

console.log(`CMD : ${cmd}`);
// Little optimization here, check for the command before doing anything else.
if (cmd === 'cf' || cmd === 'coin' || cmd === 'flip' || cmd === 'coinflip') {

    if (args[1] != undefined) {
        let choice = args[1].toLowerCase();
        console.log(`choice : ${choice}`);

        if (choice == 'heads' || choice == 'h' || choice == 'head') {
            choice = 'heads';
        }

        else if (choice == 'tails' || choice == 't' || choice == 'tail') {
            choice = 'tails';
        }


        var coins = [
            "heads",
            "tails"
        ];
        coinz = coins[Math.floor(Math.random() * coins.length)];

        if (choice != coinz) {
            message.channel.send(`Your bet: \`${choice}\`,
            outcome: \`${coinz}\` you lose`);
        } else {
            message.channel.send(`Your bet: \`${choice}\`,
                outcome: \`${coinz}\` you win`);
        };
    }
    else {
        message.reply("please choose *heads* or *tails* before the coin flip.");
    }
}


});
于 2019-07-17T15:48:36.793 回答
0

似乎决定以太是赢还是输,您将您的选择变量与coinz进行比较。但是,使用您的代码,“choice”只能是“h”或“t”,而“coinz”将是“heads”或“tails”。这种比较总是会返回 false,告诉你这是 @Gruntzy 的损失

于 2019-07-17T04:40:02.267 回答