Question

#include <cs50.h>
#include <stdio.h>
#include <stdlib.h> //covert str into int atoi()
#include <ctype.h> // isalpha(), isdigit() etc.
#include <string.h> //strlen() etc.


int main(int argc, string argv[])
{
    // convert argv[1] into int
    int key = atoi(argv[1]);
    // if input is 2 command lines and argv > 0 and there is no alphabet n shit in argv[1] we good to go
    if (argc == 2 && key > 0 && isdigit(argv[1])) 
    {
        //ask for input to chiper
        string plain = get_string("Plaintext: ");
        int len_plain = strlen(plain);
        // check each char in string plain
        for (int i = 0; i < len_plain; i++)
        {
            // if its from 'a' to 'z' add number of key
            if (plain[i] >= 'a' && plain[i] <= 'z')
            {
                char cipher = (plain[i] + key) % 26;
                printf("%c", cipher);
                return 0;
            }
        }

    }
    else
    {
        printf("Usage: ./caesar kay");
        return 1;
    }
}

运行此代码时，我遇到了分段错误。我做错了什么？我已经在这段代码上工作了几天，但我无法使其正常工作。

Answer 1

此isdigit(argv[1])出现段错误。 isdigit的功能签名（来自手册页）：

int isdigit(int c);

但是argv[1]（如果存在！）是一个字符串。建议您通过

遵循规范

验证正确数量的命令行参数是输入
检查所述参数中的每个字符是否为十进制数字

只有这样，您才能放心atoi将得到期望的结果。

CS50 PSET-2凯撒分割故障

1 个答案: