Question

当我试图在命令行上将一个数字传递到我的应用程序时，在下面的代码中出现了一个奇怪的分段错误。

int offset = 3;

int main(int argc, char *argv[]) {
    // Check for arguments to see whether there is a custom offset
    if (argc == 2) {
        // If argc == 2 then we have a offset?
        if (isdigit((unsigned char)*argv[1])) {
            offset = atoi(*argv[1]);
            printf("Offset changed to: %d
", offset);
        } else {
            printf("Offset not changed due to %s not being a number.
", *argv[1]);
        }
    } else if(argc >= 2) {
        // If argc >= 2 then we have too many arguments
        printf("Too many arguments.");
        return 0;
    }
}

Answer 1

argv[1]已经是一个字符串（类型为char*），因此在将第一个字节传递给atoi（）和printf（）时，写入*argv[1]会导致segfault。

将其修复为：

offset = atoi(argv[1]);

和

printf("Offset not changed due to %s not being a number. ", argv[1]);

Answer 2

代码中真正的问题是您试图调用未声明的函数（您必须使用C89/90编译器）。您调用isdigit。您可以调用printf。您调用atoi。你对后两者的称呼不正确。编译器无法通知您这些函数被错误调用的唯一原因是您忘记了声明它们。

包括<；ctype.h>、<；标准库.h>和<；stdio.h>，以便编译器知道atoi和其他函数的正确参数类型。一旦这样做，您就应该能够找出atoi的问题，因为编译器会发出一条诊断消息来解释问题。然后您可以相应地更改呼叫。一些编译器也能够检测到printf调用的问题。

请注意，即使您按照其他答案中的建议更改atoi和printf调用（即对阿托伊（argv[1]）等），您的代码仍将保持无效，因为在C89/90中，在未首先声明print f的情况下调用它会导致未定义的行为（在C99中，在不首先声明的情况下，调用任何函数完全是非法的）。

Answer 3

问题是打电话给atoi。它需要一个字符串。将其更改为

   offset = atoi(argv[1]);

Answer 4

#include <ctype.h>
int isdigit(int c);

isdigit() expects single char to check, argv[] is a pointer to string (array of chars). Long story short, it won t check strings like "1234", but it will check 1 .

友情链接