1

Я нашёл вот это

Но это не совсем то, что мне надо. Мне надо что-бы он удалял только сообщения с матом, а без мата оставлял.Вот что у меня получилось:

bot.on('message', async message => {
  if(message.author.id == "748788241867603979"){
    let filterwordslist = ["лох", "пидор", "сука", "блять"];
    let messageArray = message.content.split(" ");
    let args = messageArray.slice(0);
    const [...restArgs] = args;
    const text = restArgs.join(' ');
    let edit = text;

    filterwordslist.forEach(item => {
      edit = edit.replace(
        new RegExp(`(^|[^а-я])${item}([^а-я]|$)`, "gmi"),
        "$1`*цензура*`$2"
      );
    });
    if(сообщение с матом){
      message.delete();
      message.channel.send({
        embed: new Discord.MessageEmbed()
          .setColor("RANDOM")
          .setAuthor('FilterWords', message.guild.iconURL())
          .setDescription(`${message.author}: ` + edit)
      });
    }
  }
})

Вопрос у меня в этом условии:

if(сообщение с матом){
3
  • Спасибо за правку вопроса, я отозвал голос за закрытие и минус сменил на плюс (теперь вопрос вполне может быть полезен).
    – yar85
    Commented 8 июл. 2021 в 9:10
  • Комментарии не предназначены для расширенной дискуссии; разговор перемещён в чат.
    – Qwertiy
    Commented 8 июл. 2021 в 9:59
  • Границы слов проверять надо.
    – Qwertiy
    Commented 8 июл. 2021 в 10:09

3 ответа 3

1

Судя по коду, переменная edit в результате выполнения метода forEach содержит текст с заменами; а переменная text (выше) - похоже, содержит нетронутый исходный текст.
Значит, сравнив значение этих переменных, можно определить наличие замен (а следовательно, и мата) в исходном тексте:

/*....*/
filterwordslist.forEach(item => {
  edit = edit.replace(
    new RegExp(`(^|[^а-я])${item}([^а-я]|$)`, "gmi"),
    "$1`*цензура*`$2"
  );
});
if (edit !== text) {
  message.delete();
  /*....*/
}
7
  • спасибо тебе, как я сам не додумался. Commented 8 июл. 2021 в 9:11
  • Неужели нельзя проверить вхождение без замены?
    – Qwertiy
    Commented 8 июл. 2021 в 10:17
  • меня и так устраивает, а как говорится, работает не трогай. Commented 8 июл. 2021 в 11:29
  • @Qwertiy, можно. Заменив forEach на метод some, в коллбэке которого возвращая результат (new RegExp(....)).test(edit). Это даст незначительный выигрыш в производительности. Стоит ли он простоты изменений (образовательной цели) - по-моему, нет. Оптимизации уже существующего кода, желательно выполнять при необходимости в них. Когда алгоритм реализуется с нуля - другое дело.
    – yar85
    Commented 8 июл. 2021 в 11:32
  • Еще пару спичек сэкономить, поможет однократное инстанцирование массива слов и регулярок из них при запуске приложения. В этом уже начинает играть роль общая архитектура, а не только один отдельно взятый обработчик... и с высокой вероятностью появляется еще больше векторов оптимизации.
    – yar85
    Commented 8 июл. 2021 в 11:41
1

Вынеси список запрещённых слов в константу вне всех методов:

const blacklist = new Set(["слово", "второе", "третье"]);

А потом проверяй:

if (message.content.match(/[а-яё]/gi)?.some(word => blacklist.has(word))) {

Это одно из самых эффективных решений.


Если версия ноды не позволяет использовать ?., то можно вынести в переменную и сделать 2 проверки, либо написать так:

if ((message.content.match(/[а-яё]/gi) || []).some(word => blacklist.has(word))) {
0
if (message.content.match(/(?<![а-яё])(плохоеслово|другоеслово|ещё)(?![а-яё])/i)) {
10
  • Используя для определения границ слова \b (если этот токен в JS работает с кириллицей при флаге u) или символьный класс юникода (типа [^\p{L}]), можно сделать регулярку попроще и побыстрее.
    – yar85
    Commented 8 июл. 2021 в 12:01
  • Точнее, с классами юникода тут уместнее будет \P{L} (инверсный \p{L}) - то есть, выражение вида /(^|\P{L})(слово|ещеслово|...)($|\P{L})/iu при использовании с методом test. Простые проверки буквального соответствия токенам не создают такого количества внутренних циклов как look<behind|ahead>'ы, и в некоторых случаях (без |) они выполняются быстрее даже чем одиночный indexOf (быстрейшего из методов поиска подстроки в JS).
    – yar85
    Commented 8 июл. 2021 в 12:30
  • ответ без пояснений хуже ответа с пояснениями. Commented 8 июл. 2021 в 13:52
  • @yar85, \b не работает даже с u. По крайней мере, у меня не получилось заставить работать, а кроме u подходящих флагов вроде нет. С юникодными категориями хорошая идея - как-то даже не подумал, что так можно. Предпросмотры могут создавать циклы, но в данном случае я не вижу причин, по которым они должны возникнуть и думаю, что существенного влияния на производительность нет, зато мой вариант находит ровно слово - без лишних символов по бокам - в большинстве случаев это хороший стиль.
    – Qwertiy
    Commented 8 июл. 2021 в 16:15
  • @yar85, Например, если понадобится список слов, то при добавлении флага g вариант с предпросмотрами будет корректен, а вариант с захватом лишних символов не сможет найти оба слова, если они разделены пробелом. Впрочем, можно отказаться от предпросмотра назад и использовать только вперёд, что решит эту проблему, и понизит совместимость кода до ES5. А в то, что регулярка может быть быстрее indexOf я что-то не верю. Есть пример?
    – Qwertiy
    Commented 8 июл. 2021 в 16:17

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.