Question

我的目标是在框中插入文字，选择一个值作为字母表中的移位值并获取加密输出。

我刚开始使用C＃，我发现了这个练习。这就是我提出的，这是完全错误的，因为当我运行它时，我收到此错误消息“类型'System.IndexOutOfRangeException'发生了未处理的异常”。我知道错误在哪里，但我不知道如何解决它。我想知道我做错了什么，或者即使我这样做的方式是对的。

提前谢谢！

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

    }

}

Answer 1

将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语句。这是必要的，以便密码不会多次应用于输入字符串中的相同位置。

使用Caesar密码C＃

1 个答案: