为什么我的 Caesar 解决方案 (CS50) 重复输入大写字符？

Question

我的程序可以正确加密小写字母，但对于大写字母，output 将双重输入字符。 例如，如果我将输入输入为“BARFOO”，我的程序将为 output “EBDAURIFRRORO”。 代码如下所示。

printf("ciphertext: ");
for (int i = 0, n = strlen(plain); i < n; i++) 
{
    if (isalpha(plain[i]) && isupper(plain[i])) //problem must be here
    {
        int c = 65;
        printf("%c", (plain[i] - c + key) % 26 + c); 
    }
    
    if (isalpha(plain[i]) && islower(plain[i])) //this works
    {
        int c = 97;
        printf("%c", (plain[i] - c + key) % 26 + c);
    }
    
    else //this preserves punctuation, space, etc.
    {
        printf("%c", plain[i]);
    }
}
printf("\n");

我在第一个 if 循环中尝试做的是检查大写字母字符。 然后我遍历每个字符，将其转换为字母索引，其中“A”在 [0] 处索引，B 在 [1] 处索引，依此类推。 最后你可以看到我将它转换回 Ascii 索引。 为什么它只使用大写字母？ 我的数学错了吗？ 对于小写字母，数学运算基本相同，但它的行为应如此。 我假设这是我的 for 循环的问题，但我不确定我必须做些什么来改变它。

Answer 1

你的第一个“如果”是独立的。 所以当输入为大写时，它会在第一个“if”中编码并显示，然后绕过并在“else”中显示，因为它不是大写的。

你可以让“else”绕过每个非字母字符而不是绕过非小写字符来解决这个问题，像这样：

printf("ciphertext: ");
for (int i = 0, n = strlen(plain); i < n; i++)
{
    if (isalpha(plain[i]))
    {
        if(isupper(plain[i])) //problem must be here
        {
            int c = 65;
            printf("%c", (plain[i] - c + key) % 26 + c);
        }else{
            int c = 97;
            printf("%c", (plain[i] - c + key) % 26 + c);
        }

    }else //this preserves punctuation, space, etc.
    {
        printf("%c", plain[i]);
    }
}
printf("\n");