Substitution Verschlüsselung nicht zurück Zeichen

Question

Ich versuche derzeit, eine Substitution Verschlüsselung/Entschlüsselungsprogramm zu machen. Dieses Programm liest aus einer Datei und verschlüsselt Zeichen für Zeichen basierend auf einem vom Benutzer angegebenen Schlüssel. Wenn die Datei beispielsweise "ATTACK AT DAWN" lautet und der Schlüssel "LEMON" lautet, lautet der resultierende Chiffretext "LXFOPV EF RNHR".

Ich habe eine Funktion für die Verschlüsselung gemacht, ändert jedoch die Werte in andere Zeichen als alphanumerische Buchstaben. Jede Hilfe wäre willkommen. Ich habe die Funktion unten eingefügt:

void encrypt(ifstream& in_s, ofstream& out_s, char next, char& cypher, int& i, char key[]) 
{ 

    in_s.get(next); // get character from file 
    while (!in_s.eof()) 
    { 
      check_i(i); // check i value is within the constraint for key[4] 

      if(isupper(next)) // if uppercase run cypher 
      { 
        cypher = ((next - 65) + (key[i] - 65) % 26) + 65; 
      } 
      else if (islower(next)) 
      { 
        next = toupper(next); // change to upper 
        cypher = ((next - 65) + (key[i] - 65) % 26) + 65; 
        cypher = tolower(cypher); // change to lower 
      } 
      else // if blank do nothing 
      { 
        cypher = next; 
      } 
      cout << cypher; 


      in_s.get(next); // get next char 
      i++; // add 1 to i for key[i] 
    } 
} 

Answer 1

Sie haben ein Problem mit operator precedence. Der Modulo-Operator % hat eine höhere Priorität als die Addition. Also lassen Sie uns brechen Ihre Summe:

cypher = ((next - 65) + (key[i] - 65) % 26) + 65; 

Dies wird wie folgt verarbeitet werden:

cypher = (next - 65) 
     + ((key[i] - 65) % 26) 
     + 65; 

Was Sie wollten eigentlich war für die Modulo über die ganze Sache zu handeln:

cypher = ((next - 65 + key[i] - 65) % 26) + 65; 

Sie brauchen diese äußere Klammer nicht wirklich, aber sie ist ein wenig einfacher zu lesen.

Answer 2

Das Problem war, wie ich die Eingabe für mein Array bekam. ich versuchte, so etwas zu tun:

cout << "Please enter a 5 character key: "; 
cin >> key[5]; 

Wenn ich so das Array gespeichert haben sollte:

cout << "Please enter a 5 character key: "; 
cin >> key;