Vigenere CS50 - 需要帮助循环字母

Question

#include <stdio.h>
#include <cs50.h>
#include <ctype.h>
#include <stdlib.h>
#include <string.h>

int main(int argc, string argv[])
{
//Check for 2 command line arguments
if (argc != 2)
{
    printf("Nah bro, you gotta have 2 arguments.\n");
    return 1;
}
//Check is alpha
else {
    for (int i = 0; i < strlen(argv[1]); i++)
    {
        if (isalpha(argv[1][i]) == 0)
        {
            printf("Nah bro, u gots to use letters.\n");
            return 1;
        }
    }
}


//Prompt user to input text
    printf("plaintext: ");
    string p = get_string();

//Cipher
    printf("ciphertext: ");
    string k = argv[1];
    int cipherlen = strlen(k);



//Cycle through key letters
for (int i = 0, j = 0, n = strlen(p); i < n; i++)
{


    if (isalpha(p[i]))
    {

        if (isupper(p[i]))
            {
            printf("%c", ((p[i] - 65) + (k[(j % cipherlen)]) - 65) % 26 + 65);
            j++;
            }


        else if (islower(p[i]))
            {
            printf("%c", ((p[i] - 97) + (k[(j % cipherlen)]) - 97) % 26 + 97);
            j++;
            }


        else
            printf ("%c", p[i]);
    }

    }
        printf("\n");
        return 0;
}

根据检查，这是我的错误代码：

https://cs50.me/checks/a56bc9325327035cb0e8d831693c9805c4b6468b

我知道我的问题与循环遍历每个字母有关，但没有将其应用于空格或符号。我试过使用 if (isalpha) 语句和 else printf(" ") 但它对数字或符号效果不佳。我认为添加 j++ 只会遍历字母字符，但它似乎没有帮助。

这里有什么我遗漏的超级简单的东西吗？

Answer 1

您的代码的基本结构看起来不错。

我发现它存在三个问题：

你所有的 printf 都受到 if (isalpha(p[i])) 检查的保护，所以如果明文字符不是字母，你的程序永远不会输出任何东西（它应该输出字符不变反而）。解决这个问题很简单；只需删除循环中的外部 if (...):

for (int i = 0, j = 0, n = strlen(p); i < n; i++)
{
    if (isupper(p[i]))
        {
        printf("%c", ((p[i] - 65) + (k[(j % cipherlen)]) - 65) % 26 + 65);
        j++;
        }
    else if (islower(p[i]))
        {
        printf("%c", ((p[i] - 97) + (k[(j % cipherlen)]) - 97) % 26 + 97);
        j++;
        }
    else
        printf ("%c", p[i]);
}

内部 if/else if 链正确处理这种情况。

当前明文字符p[i]和当前关键字字符k[j % cipherlen]可以独立大写/小写。您的代码目前根本不处理这个问题；相反，它假定如果 p[i] 是大写，则 k[j % cipherlen] 也必须是大写，对于小写也类似。

顺便说一下，我建议不要在代码中写 65 和 97。我会分别使用 'A' 和 'a'，这使恕我直言更具可读性。

要解决此问题，您必须分别测试 k[j % cipherlen] 的大写/小写。例如：
```
for (int i = 0, j = 0, n = strlen(p); i < n; i++)
{
    char key_char = k[j % cipherlen];
    int key_shift;
    if (isupper(key_char)) {
        key_shift = key_char - 'A';
    } else {
        key_shift = key_char - 'a';
    }
    if (isupper(p[i]))
        {
        printf("%c", ((p[i] - 'A') + key_shift) % 26 + 'A');
        j++;
        }
    else if (islower(p[i]))
        {
        printf("%c", ((p[i] - 'a') + key_shift) % 26 + 'a');
        j++;
        }
    else
        printf ("%c", p[i]);
}
```
（我厌倦了重复输入相同的表达式，所以我将公共位提取到变量（key_char、key_shift）。这里唯一棘手的部分是 j 应该只在实际使用 key_shift 时递增，但您的代码已经处理了。）
这是一个微妙的问题，但是所有 <ctype.h> 函数（例如 isupper、isalpha、...）如果参数为是负的。 char 在许多实现中是有符号类型，因此随机字符 str[i] 很可能是负数。为了完全便携和正确，您应该在每次这样的调用中将字符转换为 (unsigned char)：
```
if (isupper((unsigned char)key_char))
    ...
if (isupper((unsigned char)p[i]))
    ...
else if (islower((unsigned char)p[i]))
    ...
```
或者，完全接受 ASCII（您的代码的其余部分已经采用它）并执行：
```
if (key_char >= 'A' && key_char <= 'Z')
    ...
if (p[i] >= 'A' && p[i] <= 'Z')
    ...
else if (p[i] >= 'a' && p[i] <= 'z')
    ...
```

Vigenere CS50 - 需要帮助循环字母

Vigenere CS50 - Need help cycling through alpha letters

c

vigenere

cs50