检查字符串是否为 c 中的数字时出现问题

Problem while checking if a string is a number in c

提问人:Melsunshine 提问时间:10/24/2019 最后编辑:Peter DunihoMelsunshine 更新时间:10/24/2019 访问量:253

问:

对于 CS50 类中的问题,我必须检查密钥是否有效,即用户给出的输入是否为数字。我尝试了不同的代码,但每次,如果输入中至少有一个数字,它就会返回密钥正确。虽然只有当字符串的所有字符都是数字时,它才应该是正确的......

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

int main(int argc, string argv[])
{
    if (argc !=2)
    {
        printf("Usage: %s key\n", argv[0]);
        return 1;    
    }
    else 
    {   
        for (int i = 0; i <= (strlen(argv[1])); i++)
        {
            if (isalpha(argv[1][i]))
            {
                printf("Usage: ./caesar key \n");
                return 1;
            }

            else
            {
                printf("%s\n", argv[1]);
                return 0;
            }
        }
    }
}

当输入超过 2 个字符串时,它应该返回 1(即工作);

当字符串 [1] 不是数字时,它应该返回 1(这也有效);

当字符串 [1] 是数字时,它应该打印 2 个字符串(不起作用);

例如:我希望 ./caesar 222f 返回: 用法:./caesar key,但实际输出是 222f

C CS50

评论

1赞 Shawn 10/24/2019
您的代码在返回之前只查看 的第一个字符...argv[1]

答:

0赞 Michael Horn 10/24/2019 #1

您的代码仅检查字符串的第一个字符 - 它始终根据第一个字符是否为数字返回结果。

不要在循环中打印成功的结果,而是等待循环遍历所有字符,并且仅在找到非数字字符时才提前返回:

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

int main(int argc, string argv[])
{
    if (argc !=2)
    {
        printf("Usage: %s key\n", argv[0]);
        return 1;    
    }
    else 
    {   
        for (int i = 0; i <= (strlen(argv[1])); i++)
        {
            if (isalpha(argv[1][i]))
            {
                printf("Usage: ./caesar key \n");
                // Return early for badly formed argument
                return 1;
            }
        }

        // Now that all chars have been validated, print the argument
        printf("%s\n", argv[1]);
        return 0;
    }
}

评论

0赞 chux - Reinstate Monica 10/24/2019
尝试。这会导致“所有字符都已验证”argv[1] == "!"