C++::toupper не разрешает сравнение на равенство?

Question

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

Например, если я передаю "привет" через шифрование, моя зашифрованная строка становится "ПРИВЕТ", но когда я передаю "ПРИВЕТ" через (естественно заглавные буквы), она корректно сдвигается.

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

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