cs50 pset2 caesar--验证密钥

Question

我正在经历cs50，我被困在pset2凯撒。我按照指示一步一步地进行，我已经通过了关于验证密钥的部分——确保命令行参数只是一个数字（./caesar 20），如果不是（./caesar 20x或。/caesar xyz），则打印出用法：/caesar密钥\n.但一旦我添加下一行提示用户输入明文，然后打印回密文，它似乎不会注册验证密钥的前几行。我知道哈佛大学的学术诚信政策，所以我更多地是在寻求这方面的提示/帮助/解释。我已经在下面发布了我的代码，非常感谢您的帮助。

int main(int argc, string argv[])       
{

        int i = 0;
        if(isalpha(argv[1][i]))
        {
            printf("Usage: ./caesar key\n");
            return 1;
        }
        else
        {
            int key = atoi(argv[1]);
            string text = get_string("plaintext: ");
            printf("ciphertext: ");
            int n = strlen(text);
            for(i = 0; i < n; i++)
            {
                if(isupper(text[i]))
                {
                    printf("%c", (((text[i] - 65) + key) % 26) + 65);
                }
                else if(islower(text[i]))
                {
                    printf("%c", (((text[i] - 97) + key) % 26) + 97);
                }
                else
                {
                    printf("%c", text[i]);
                }
            }
    }
    printf("\n");
    return 0;
} 

Answer 1

你必须在命令行参数中检查2件事。1.检查你是否有2个参数。（这在你的程序中没有检查）2.检查它是否不是字母表...由于第二个参数自动是一个字符串（由于字符串artv[]），你必须首先将其转换为整数. ---- 这滋生了你必须通过以下方式来处理的子问题： 2. a.逐个检查字符串（每个字符），所有必须是一个整数。2. B。如果至少有一个是字符，那么返回false...

2.A.和2。B.应该嵌套在一个单独的函数中，该函数将返回一个布尔值（T/F），然后从该布尔值将这两个布尔表达式括在while循环中。

我的回答晚了2个月，但是因为我刚刚完成了这个问题，我想也许有人会遇到和我一样的困难，所以希望有人能读到这篇文章并帮助他们。：）

Answer 2

您需要重新进行输入验证。

1. 它不能确保在命令行中提供一个参数。
2. 它只是验证artv[1]的第一个字符。程序应该测试参数的每个字符都是数字（请参见man isdigit）。isalpha（）不是一个好的测试。如果用户输入$怎么办？

不清楚程序何时不工作，但基于这些缺陷，这将无法通过校验50。