Простая программа Cypher не сбрасывается правильно

Question

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

Все работает хорошо, но почему-то я получаю старые всплывающие окна, которые появляются при повторном запуске программы.

Например, если я введу «greg», затем расшифрую его, затем снова введу «greg», я получу два «greg» (зашифрованных) - поэтому он не сбрасывается правильно, он будет добавлять новые слова без сброса.

#include <iostream>
#include <string>

using namespace std;

int main() {
  string alphabet{"a b c d e f g h i j k l m n o p q r s t u v w x y z 1 2 3 4 "
                  "5 6 7 8 9 0"};
  string key{"m h i p g f b t x u r l w a j e q k z d y v o c n s 9 5 0 7 8 2 "
             "3 4 1 6"};
  string word{};
  string secretword{};
  string decryptedword{};
  int selection{};
  // int newpos {};
  int position{};
  //int keypos{};

  do {
    cout << "-------------------------------------" << endl;
    cout << "Select an Option: " << endl;
    cout << "1: Encrypt" << endl;
    cout << "2: Decrypt" << endl;
    cout << "3: Quit" << endl;

    cin >> selection;

    if (selection == 1) {
      cout << "Enter a Word to Encrypt:";
      cin.sync();
      getline(cin, word);

      for (auto i : word) {
        if (isupper(i)) {
          cout << "Please use lower case only" << endl;
          break;
        }

        position = alphabet.find(i);
        secretword += key.at(position);
      }
      cout << "Encrypted Word: " << secretword << endl;
      secretword = "";
    }

    if (selection == 2) {
      cout << "Enter a Word to Decrypt: ";
      cin.sync();
      getline(cin, secretword);

      for (auto i : secretword) {
        if (isupper(i)) {
          cout << "Please use lower case only" << endl;
          break;
        }
        position = key.find(i);
        decryptedword += alphabet.at(position);
      }

      cout << "Decrypted Word: " << decryptedword << endl;
      decryptedword = "";
    }
  } while (selection != 3);

  return 0;
}

Answer 1

Программа использует secretword как для вывода шифрования, так и для ввода дешифрования и никогда не очищает его. По электронной почте Ой.

Вместо очистки переменных уменьшите их область видимости, чтобы их не нужно было очищать и не конфликтовать. Например:

#include <iostream>
#include <string>

using namespace std;

int main()
{
    string alphabet { "a b c d e f g h i j k l m n o p q r s t u v w x y z 1 2 3 4 "
                      "5 6 7 8 9 0" };
    string key { "m h i p g f b t x u r l w a j e q k z d y v o c n s 9 5 0 7 8 2 "
                 "3 4 1 6" };
    //int keypos{};

    int selection;
    do
    {
        cout << "-------------------------------------" << endl;
        cout << "Select an Option: " << endl;
        cout << "1: Encrypt" << endl;
        cout << "2: Decrypt" << endl;
        cout << "3: Quit" << endl;

        cin >> selection;

        if (selection == 1)
        {
            cout << "Enter a Word to Encrypt:";
            cin.sync();
            string word;
            string secretword;
            getline(cin, word);

            for (auto i : word)
            {
                if (isupper(i))
                {
                    cout << "Please use lower case only" << endl;
                    break;
                }

                auto position = alphabet.find(i);
                secretword += key.at(position);
            }
            cout << "Encrypted Word: " << secretword << endl;
            secretword = "";
        }

        if (selection == 2)
        {
            cout << "Enter a Word to Decrypt: ";
            cin.sync();
            string secretword;
            string decryptedword;
            getline(cin, secretword);

            for (auto i : secretword)
            {
                if (isupper(i))
                {
                    cout << "Please use lower case only" << endl;
                    break;
                }
                auto position = key.find(i);
                decryptedword += alphabet.at(position);
            }

            cout << "Decrypted Word: " << decryptedword << endl;
            decryptedword = "";
        }
    } while (selection != 3);

    return 0;
}

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