Фильтрация плохих слов в MUD Game

Question

Фильтрация плохих слов в MUD Game

0

В моей игре C++ MUD я хочу, чтобы убрать игрока из возможности вводить проклятые слова. Вы все знаете, каковы они, нет необходимости в примерах. Поэтому я попробовал что-то вроде этого:

vector<string> vulger = { "You know what goes in here");

void Player::SendString(const std::string& p_string)
    {
for (vector<string>::iterator it = vulger.begin(); it != vulger.end(); ++it)
    {
        if (!p_string.compare(*it) || !p_string.find(*it))
        {
            //Testing what words are been neglected by the if statement.
            //cout << *it;
                Conn()->Protocol().SendString(*Conn(), p_string + newline);
        }
    }

    }

Но все это - это петлевые строки, которые отправляются в сеть. Включая анонсы.

Может ли кто-нибудь увидеть, что я делаю неправильно или что-то предлагаю?

Conor Pendlebury 01 дек. 2014, в 19:40

Источник

3

Ваш параметр называется p_string , но вы используете !p_name внутри вашего цикла. Что такое p_name ?
Ken White 01 дек. 2014, в 18:20
0

Извините, они просто опечатки, я буду редактировать их сейчас.
Conor Pendlebury 01 дек. 2014, в 18:25
0

Не перепечатывайте код здесь. Невозможно обнаружить проблемы в вашем коде, если это не ваш настоящий код. Повторное определение может скрыть проблему (или ввести новые, как здесь произошло). Скопируйте и вставьте вместо.
Ken White 01 дек. 2014, в 18:28
0

Будьте осторожны, чтобы не блокировать дискуссии о Джо Кокере.
molbdnilo 01 дек. 2014, в 18:29
1

Ваш код написан для циклического прохождения строк, которые отправляются в сеть. Что еще вы ожидаете от этого?
Ken White 01 дек. 2014, в 18:30
0

@KenWhite Я хочу, чтобы сообщение отправлялось в сеть, когда используется оскорбительное слово. Затем я создам функцию removeWord, чтобы справиться с этим.
Conor Pendlebury 01 дек. 2014, в 18:45
0

Все, что отправлено в сеть, помечается как оскорбительное, поэтому при сравнении должно быть проблема.
Conor Pendlebury 01 дек. 2014, в 18:49

Показать ещё 5 комментариев

Теги:

c++

validation

mud

2 ответа

0

Вы обрабатываете возвращаемое значение std::string::find как bool, когда на самом деле оно возвращает смещение в строку со специальным магическим значением npos если оно не найдено.

if (my_string.find("foo") == std::npos) {
   // foo is NOT in my_string
} else {
   // foo is in my_string
}

Adrian McCarthy 01 дек. 2014, в 19:55

Ещё вопросы

Ваш параметр называется p_string , но вы используете !p_name внутри вашего цикла. Что такое p_name ?
Извините, они просто опечатки, я буду редактировать их сейчас.
Не перепечатывайте код здесь. Невозможно обнаружить проблемы в вашем коде, если это не ваш настоящий код. Повторное определение может скрыть проблему (или ввести новые, как здесь произошло). Скопируйте и вставьте вместо.
Будьте осторожны, чтобы не блокировать дискуссии о Джо Кокере.
Ваш код написан для циклического прохождения строк, которые отправляются в сеть. Что еще вы ожидаете от этого?
@KenWhite Я хочу, чтобы сообщение отправлялось в сеть, когда используется оскорбительное слово. Затем я создам функцию removeWord, чтобы справиться с этим.
Все, что отправлено в сеть, помечается как оскорбительное, поэтому при сравнении должно быть проблема.

Thomas Matthews · Accepted Answer · 2014-12-01T16-11-00.000Z

Измените || в выражении if в &&:

    void Player::SendString(const std::string& p_string)
    {
      for (vector<string>::iterator it = vulger.begin(); it != vulger.end(); ++it)
      {
#if 0
// Original:
        if (!p_name.compare(*it) && !p_name.find(*it))
        {
          Conn()->Protocol().SendString(*Conn(), p_string + newline);
        }
        else
        {
          cout << "Vulgar word found: " << *it << "\n";
        }
#else
// Edit 1:  Changed ordering.
        if ((p_name == *it) || (p_name.find(*it) != std::string::npos))
        {
          cout << "Vulgar word found: " << *it << "\n";
        }
        else
        {
          Conn()->Protocol().SendString(*Conn(), p_string + newline);
        }
#endif

      }
    }

В результате тестирования я обнаружил, что весь список печатается, а не только более простое слово при его вводе.
Смотрите мои изменения, «Редактировать 1». Я упростила логику, если слово вульгарно, печатать, иначе отправлять слово
Используя обновленную версию, все, что отправлено в сеть, зацикливается столько раз, сколько содержится в векторе.
Тем не менее, теперь он улавливается, когда вульгурское слово отправляется в сеть.
Вам нужно удалить вульгарное слово из переменной p_string перед отправкой p_string в сеть. Оператор if идентифицирует только вульгарное слово, но не удаляет его из p_string , поэтому вульгарное слово все равно будет отправлено в сеть.
Вы думаете, что это также вызывает проблему с зацикливанием?
Какой самый дешевый способ удалить из середины вектора.