строка — C ++ :: toupper не разрешает сравнение на равенство?
Я пытаюсь преобразовать строку в верхний регистр, чтобы я мог манипулировать ею, но, хотя я могу успешно манипулировать естественными строчными буквами, а также преобразовывать строчные в верхний регистр, использование этого метода преобразования не позволяет манипулировать.
Например, если я передаю «привет» через шифрование, моя зашифрованная строка становится «ПРИВЕТ», но когда я передаю «ПРИВЕТ» через (естественно заглавные буквы), она корректно сдвигается.
Есть ли другой способ использования прописных букв, который мне нужно использовать, или я делаю что-то не так?
int Caesar::encrypt (const std::string &message, std::string &emessage) {
int count = 0;
emessage = message;
std::transform(emessage.begin(), emessage.end(), emessage.begin(), ::toupper);
for (std::string::size_type i = 0; i < message.size(); i++) {
for (int j = 0; j < 26; j++) {
if (emessage[i] == std_alphabet[j]) {
std::replace(emessage.begin(), emessage.end(), message[i], c_alphabet[j]);
}
}
count++;
}
return count;
}
конструктор:
Caesar::Caesar (int shift) {
// loop to populate vector with 26 letters of English alphabet
// using ASCII uppcase letter codes
for (int i = 0; i < 26; i++) {
std_alphabet.push_back(i + 65);
}
// fills Caesar alphabet with standard generated alphabet
c_alphabet = std_alphabet;
// shifts Caesar alphabet based off the constructor parameter
std::rotate(c_alphabet.begin(), c_alphabet.begin() + shift, c_alphabet.end());
}
тестовый файл:
void testCaesar() {
Caesar test(4);
std::string original = "HELLO";
std::string encrypted = "";
test.encrypt(original,encrypted);
std::cout << encrypted << std::endl;
std::cout << original << std::endl;
}
int main() {
testCaesar();
return 0;
}
Очевидно, что есть заголовок и включает в себя и прочее, но это основной код
заголовочный файл включает в себя два частных вектора
Решение
Конкретная проблема, которую вы видите, заключается в том, что вы заменяете не ту вещь здесь:
std::replace(emessage.begin(), emessage.end(), message[i], c_alphabet[j]);
Если message был в нижнем регистре, то emessage будут все заглавные буквы — ни одна из которых не будет message[i], так что замена ничего не сделает. Вы имели в виду:
std::replace(emessage.begin(), emessage.end(), emessage[i], c_alphabet[j]);
^^^^^^^^^^^
Тем не менее, ваш алгоритм совершенно не так, как HELLO шифрует как BCBBA со сдвигом 4. Существует 1-1 отображение на буквы, так H а также L оба не могут пойти в B, То, что вы хотите сделать, это сдвигать каждую букву по ходу, просто заменяя ее на то, какой должна быть следующая буква. То есть:
for (std::string::size_type i = 0; i < emessage.size(); ++i) {
emessage[i] = c_alphabet[emessage[i] - 'A'];
}
С которой вам на самом деле не нужен начальный шаг преобразования:
emessage = message;
for (std::string::size_type i = 0; i < emessage.size(); ++i) {
emessage[i] = c_alphabet[::toupper(emessage[i]) - 'A'];
}
Все это можно немного сократить, просто бросив count (который в любом случае является просто размером, поэтому является избыточным) и принимает сообщение по значению:
std::string encrypt(std::string from) { // intentionally copying
for (char& c : from) {
c = c_alphabet[::toupper(c) - 'A'];
}
return from;
}
Другие решения