使用凯撒密码 C# 加密文本

Encrypt text using Caesar cipher C#

我的objective是在一个方框内插入文字,选择一个值作为字母表的移位值,得到加密后的输出。

我刚开始使用 C#,我发现了这个练习。这就是我想出的,这肯定是错误的,因为当我 运行 它时,我收到此错误消息 "An unhandled exception of type 'System.IndexOutOfRangeException' occurred"。我知道错误在哪里,但我不知道如何解决。我想知道我做错了什么,或者我这样做的方式是否正确。

提前致谢!

private void cifrar_Click(object sender, EventArgs e)
    {
        string input = message.Text;

        int shift = int.Parse(shifting.Text);

        char[] alphabet = new char[26] { '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' };

        string output_text = "";

        char[] input_array = input.ToCharArray();

        for (int i = 0; i < alphabet.Length; i++)
        {
            for (int j = 0; j < input_array.Length; j++)
            {
                if (input_array[j] == alphabet[i])
                    input_array[j] = alphabet[i + shift];
            }                    
        }

        output_text = new string(input_array);

        output.Text = output_text;
    }

    }

}

shift添加到i时,您可以获得一个大于数组中元素数量的新值。尝试使用此索引访问数组将导致 IndexOutOfRangeException.

使用modulo operator.

限制索引在有效范围内
input_array[j] = alphabet[(i + shift) % alphabet.Length];

你还需要稍微改变一下逻辑:

for (int j = 0; j < input_array.Length; j++)
{
    for (int i = 0; i < alphabet.Length; i++)
    {
        if (input_array[j] == alphabet[i])
        {
            input_array[j] = alphabet[(i + shift) % alphabet.Length];
            break;
        }
    }
}                    

我交换了内循环和外循环,还添加了 break 语句。这是必要的,这样密码就不会多次应用于输入字符串中的同一位置。