c - Vigenere CS50 - 需要帮助循环浏览字母

Question

我正在尝试执行CS50 Vigenere exercise .

#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++ 只会迭代字母字符，但它似乎没有帮助。

这里有什么我错过的 super 简单的东西吗？

Answer 1

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

我发现它存在三个问题:

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 链可以正确处理这种情况。

2. 当前明文字符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]);
   }
   

   (我厌倦了重复输入相同的表达式，因此我将公共(public)位提取到变量( key_char 、 key_shift )。这里唯一棘手的部分是，只有在实际使用 j 时才应该递增 key_shift ，但是你的代码已经处理好了。)

3. 这是一个微妙的点，但是如果参数为负数，则所有 <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')
       ...