代替暗号化文字が返されない

Question

現在、代替暗号化/復号化プログラムを作成しようとしています。このプログラムはファイルから読み込み、与えられたキーに基づいて文字ごとに暗号化します。たとえば、ファイルが "ATTACK AT DAWN"であり、与えられたキーが "LEMON"である場合、結果として得られる暗号文は "LXFOPV EF RNHR"となります。

私は暗号化の機能を作っていますが、値を英数字以外の文字に変更しています。どんな助けもありがとう。私は以下の機能を貼り付けました。

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

あなたはoperator precedenceに問題があります。モジュロ演算子%は、加算よりも優先度が高くなります。それでは、あなたの合計を打破してみましょう：

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

これは次のように処理されます：あなたが実際に剰余が全部渡って行動するためだった何を望むか

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

：

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

あなたは実際にその外側のかっこのセットを必要としませんが、それは少し読みやすくなります。

Answer 2

問題は、私の配列の入力がどのようになっていたかです。 私はこのような何かしようとしていた：私が見

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