C++ :: toupper не позволяет сравнивать равенство?

Question

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

Например, если я передаю «привет» через шифрование, моя зашифрованная строка становится «HELLO», но когда я передаю «HELLO» через (естественно заглавную), она правильно сдвигается.

Есть ли другой способ форсировать прописные буквы, которые мне нужно использовать, или я делаю что-то неправильно?

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; 
} 

Очевидно, есть заголовок и включает в себя и прочее, но это является основной код

заголовок файл включает в себя две приватные съедали векторы

Answer 1

Конкретная проблема, которую Вы видите, что вы заменяете неправильную вещь здесь:

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; 
}