Правильно ли задал операции сравнения?

Question

[Андрей Любименко]

Задача состоит в том, что беру переменную String(Содержит ФИО сотрудника) и вытаскиваю по порядку каждую букву, и сверяю её по таблице кодировок типа 1251, если это буква НЕ RUS(маленькая,большая, и отдельно стоящие ё), НЕ ENG(маленькая,большая), и это НЕ пробел - то меняю boll на значение false; Почему значение не меняется? а остаётся true

Comments

[Ivan Bogachev]

Вставляйте код с помощью тега code, а не картинкой.

Answer 1

[Roman]

Дословно

#include<locale>
//...
locale loc(".1251");
if(!isalpha(c, loc) && !isspace(c, loc))
{
  //...
}

win10 VS2017

Comments

[jcmvbkbc]

сверяю её по таблице кодировок типа 1251, если это буква НЕ RUS

isalpha

Но есть нюанс в использовании isalpha, состоящий в том, что для всего за пределами ASCII нужна рабочая локаль. И нужно знать, в какой кодировке строка. И если кодировка не совпадает с локалью, то всё становится очень сложно.

[Roman]

jcmvbkbc, полезное замечание
https://en.cppreference.com/w/cpp/locale/isalpha

Answer 2

[devalone]

> если это буква НЕ RUS(маленькая,большая, и отдельно стоящие ё), НЕ ENG(маленькая,большая), и это НЕ пробел - то меняю boll на значение false

Здесь ты пишешь "И", а в коде почему-то стоит "ИЛИ".

вдумайся в смысл выражения (c != 32) || (c != 184), оно всегда истинно

И сравнивай стандартными функциями вроде isalpha, в крайнем случае c >= 'a' && c <= 'z', но не с числами, во первых не читаемо, во вторых нет гарантий, что кодировка ascii.

А ещё лучше упростить код так:

bool isValidChar = isAlpha(c) || isDigit(c) || isSomethingElse(c);

сразу понятно, что символ нам подходит только, если удовлетворяет хотя бы одному из условий. А чтоб совсем хорошо было, используй регулярные выражения.