0

Функция ниже должна редактировать строку удаляя из неё лишние символы если этот символ не пробел, не латинская буква, или не цифра. Но почему-то даже если ввести русскую букву, или любой другой символ не из допустимого множества, то в этот if не заходит.

string editing(string sentence)
{
    int i;
    i = 1;
    while (i <= sentence.length())
    {
        if (!('0' <= sentence[i] <= '9' || 'A' <= sentence[i] <='Z' || 'a' <= sentence[i] <= 'z' || sentence[i] == ' '))
        {
            sentence.erase(i, 1);
            i--;
        }
        i++;
    }
    return sentence;
}

1 ответ 1

5

Вот это, например

'0' <= sentence[i] <= '9'

компилятор рассматривает как

('0' <= sentence[i]) <= '9'

так что больше sentence[i] чем 0 или меньше - мы все равно получим что-то (true или false), меньшее, чем 9 - так что весь результат получается true.

Еще учтите, что массивы и иже с ними ведут счет элементов с 0, так что элемент

sentence[sentence.length()]

не существует. Выход за границы элемента, UB...

5
  • Идиотское (нет) сравнение всплывает из математики. Там это абсолютно корректная запись условия принадлежности диапазону. Увы ,по всей видимости преподаватели забывают объяснить ученикам, что подавляющее большинство языков не поддерживает математическую нотацию 11 ноя 2020 в 5:10
  • Сравнение возникает не только из математики, но ещё и из питона, где такая запись вполне валидна
    – int3
    11 ноя 2020 в 7:56
  • 1
    если sentence - std::string, то sentence[sentence.length()] не является UB
    – KoVadim
    11 ноя 2020 в 8:14
  • @KoVadim Насколько я помню, до С++11 являлось.
    – Harry
    11 ноя 2020 в 8:36
  • да, но там были ограничения.
    – KoVadim
    11 ноя 2020 в 8:37

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

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