替换和修剪消息

Question

我正在尝试 toLowerCase，替换然后修剪该替换，例如 Hello [- the re _- wo rl d. 将变成一个没有空格或标点符号的字符串 hellothereworld

我正在使用这个

    let msg = message.content.toLowerCase()
    let originalMessage = msg.split(" ");
    let removePunctuation = originalMessage.toString().replace(/[.,\/#!$%\^&\*;:{}=\-_`~()]/g,"");
    let checkMessage = removePunctuation.toString().trim();

然后这个来过滤消息：

for (let i = 0; i < checkMessage.length; i++) {
    if(allowed.includes(checkMessage[i])) {
        continue;
    }
    if(curse.includes(checkMessage[i])) {
        const filterViolation = new Discord.RichEmbed()
            .setColor('#ff0000')
            .setAuthor(message.author.username, message.author.avatarURL)
            .setTitle('Filter Violation')
            .setDescription(`${rule} **Profanity.** \n${ifcont}\n${ifmistake}`)
            .setTimestamp()
            .setFooter(copyright);
        message.delete()
        message.author.send(filterViolation)
        message.channel.send(msgvio).then(msg => {msg.delete(15000)})
        logger.write(logviolation)
    }
//More code
}

^{感谢 Tarazed 对过滤器的总体帮助。}

但是当我输入任何违反过滤器的消息时，什么都不会发生，控制台中也不会抛出任何错误。关于我做错了什么有什么想法吗？

新代码：

let checkMessage = message.content.toLowerCase().replace(/[^\w ]/g,"");

    console.log(checkMessage);

    // let checkMessage = msg.split(" ");
    for (let i = 0; i < checkMessage.length; i++) {
    if(checkMessage.includes(allowed[i])) {
        continue;
    }
//--------------------------------------- CURSE
    if(checkMessage.includes(curse[i])) {
        const filterViolation = new Discord.RichEmbed()
            .setColor('#ff0000')
            .setAuthor(message.author.username, message.author.avatarURL)
            .setTitle('Filter Violation')
            .setDescription(`${rule} **Profanity.** \n${ifcont}\n${ifmistake}`)
            .setTimestamp()
            .setFooter(copyright);
        message.delete()
        message.author.send(filterViolation)
        message.channel.send(msgvio).then(msg => {msg.delete(15000)})
        logger.write(logviolation)
    }
}

它正确记录了检查消息console.log(checkMessage);，但它没有通过过滤器，既不允许也不允许。

它正确记录了消息，但单词违反了过滤器，但什么也没做。

新代码 2：

if(curse.some(word => checkMessage.includes(word) && !allowed.some(allow => allow.includes(word) && checkMessage.includes(allow)))) {
        const filterViolation = new Discord.RichEmbed()
            .setColor('#ff0000')
            .setAuthor(message.author.username, message.author.avatarURL)
            .setTitle('Filter Violation')
            .setDescription(`${rule} **Profanity.** \n${ifcont}\n${ifmistake}`)
            .setTimestamp()
            .setFooter(copyright);
        message.delete()
        message.author.send(filterViolation)
        message.channel.send(msgvio).then(msg => {msg.delete(15000)})
        logger.write(logviolation)
        return;
    }

Answer 1

由于checkMessage 是一个字符串，您的for 循环正在循环并测试单个字符。

例如：

let checkMessage = "hellothereworld";
for (let i = 0; i < checkMessage.length; i++) {
    console.log(checkMessage[i]);
}

也许您宁愿保留字符串中的空格并对单个单词进行操作：

let originalMessage = "Hello [-the world"


// Use regex that keeps spaces
let removePunctuation = originalMessage.toString().replace(/[^\w ]/g,"");
let checkMessage = removePunctuation.toString().trim();

// Split into words
let messageArray = checkMessage.split(' ');

// Loop over words
for (let i = 0; i < messageArray.length; i++) {
    console.log(messageArray[i]);
}

Answer 2

您当前的问题是，您实际上只是在迭代像 "test" 这样的字符串，并检查每个字母是否是诅咒词。

也就是说，checkMessage[0] 将是 't'，checkMessage[1] 将是 'e'，等等。我猜没有一个字符可以匹配您的 curse 甚至 allowed 数组中的任何内容。

您可以完全摆脱循环，只需检查if(curse.includes(checkMessage))，整个消息...但要小心，这可能很容易返回误报。天堂禁止有人发送像 "it was a nice pic until i looked closer" 和 pic until 这样的消息会在您的过滤器中触发某个 c-word。

无论如何，我还想指出，您用于去除空格和标点符号的代码做了一些相当奇怪的事情。我将评论我期望在每个阶段发生的事情。假设输入消息为" Hello - world. "

let originalMessage = msg                    //" hello - world. "
                       .split(" ");          //["", "hello", "-", "world.", ""]
let removePunctuation = originalMessage
                         .toString()         //",hello,-,world.," (why even split?)
                         .replace(/[.,\/#!$%\^&\*;:{}=\-_`~()]/g,""); //"helloworld"
let checkMessage = removePunctuation
                    .toString() //"helloworld" (does nothing ever)
                    .trim();    //"helloworld" (does nothing ever)

您可以通过使用匹配所有非单词字符的正则表达式类\W 轻松实现相同的目的：

let checkMessage = message.content.toLowerCase().replace(/\W/g,""); //"helloworld"

---

看起来仍然存在与循环相关的问题。我建议使用Array#some 来测试消息是否包含任何脏话：

let checkMessage = message.content.toLowerCase().replace(/[^\w ]/g,"");

if(curse.some(word =>                //search through the curse words
     checkMessage.includes(word) &&  //if message has this curse word
     !allowed.some(allow =>          //and there's no allowed word which:
       allow.includes(word) &&       //1. contains this curse word
       checkMessage.includes(allow)  //2. is in the message
     )
  )) {
  //then send violation
}