为何数字与字母验证结果相同？如何修复命令行参数校验问题

原代码存在几个关键问题，导致验证逻辑失效：

1. 错误访问了字符串的第二个字符（verify[1]），但我们需要验证的是第一个字符（索引为0）
2. 未检查命令行参数数量，用户未传入第二个参数时，argv[1]为NULL会直接导致程序崩溃
3. 未验证参数长度，无法确保用户输入的是单个数字（比如输入"12"或"a1"都会被错误通过）
4. 使用isdigit时未处理字符符号问题，可能引发未定义行为

修改后的完整代码 #

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

bool is_valid(string verify);
int main(int argc, string argv[])
{
    // 先检查是否传入了恰好一个命令行参数
    if (argc != 2 || !is_valid(argv[1]))
    {
        printf("try again:\n");
        return 1;
    }

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

bool is_valid(string verify)
{
    // 验证参数长度必须为1，且该字符是数字
    if (strlen(verify) != 1)
    {
        return false;
    }
    
    // 转换为unsigned char避免isdigit的未定义行为
    unsigned char ch = (unsigned char)verify[0];
    return isdigit(ch);
}

修改说明 #

• 参数数量检查：调用is_valid前先判断argc != 2，确保用户传入且仅传入第二个参数
• 长度验证：通过strlen(verify) != 1确保输入是单个字符
• 字符索引修正：改为访问verify[0]，即用户输入的唯一字符
• isdigit安全处理：将字符转换为unsigned char，避免因char为有符号类型导致的未定义行为
• 返回值规范：使用false和true替代0和1，提升代码可读性

内容的提问来源于stack exchange，提问作者Sizwe