C编程中的分段错误错误

Question

当我刚刚运行 ./caesar 命令时，我在此程序中收到 Segmentation fault 错误。我发现当我在代码的开头写 int length=strlen(argv[1]); 时会打印错误，当我将这一行放在（for循环）之前时它工作正常，但我不明白为什么它在那里工作得很好，而不是在程序的顶部？？

非常感谢您的帮助！提前致谢。

int main(int argc,string argv[])
{
    int s=0;
    int length=strlen(argv[1]); ##GETS ERROR

    if ( argc !=2)
    {
        printf("Usage:./caesar key\n");
        return 1;
    }
    else
    {

        int length=strlen(argv[1]); ##WORKS FINE

        for (int i = 0; i < length ; i += 1)
        {
            int c= argv[1][i];
            if (isdigit(c))
            {
                s += 1;
            }
            else
            {
                s+=0;
            }
        }
        printf("%i\n",s);
        if (length==s)
        {
            int key = atoi(argv[1]);
            printf("int:%i\n",key);
        }
        else
        {
            printf("Usage:./caesar key\n");
        }
    }
}

Answer 1

你的第一次尝试：

int length=strlen(argv[1]); ##GETS ERROR

这会报错，因为当你调用 main 时，有一个参数：没有索引为 1 的参数。这种方式是不安全的，所以你应该先处理参数的数量，然后再解析/读取/使用论据。

第二次尝试时（处理 argc）：

    if (argc !=2)
    {
        printf("Usage:./caesar key\n");
        return 1;
    }
    else
    {

    int length=strlen(argv[1]); ##WORKS FINE
    .....
    }

你已经确定了，你有两个参数。因为，如果没有，您的代码就会退出。问题是，您没有使用相同的测试用例测试您的代码版本：您的既和新代码（带有 if 和不带）将使用两个参数；但是，如果只有一个参数，旧的会失败，而第二个不会（它只会从程序返回）。

Answer 2

很可能，您需要在参数上使用凯撒。不要只输入“./caesar”，而是输入“caesar blah”。更重要的是，您应该编写一些代码来检查参数的数量。例如，您可能会编写类似

的内容

#include <stdlib.h>
#include <stdio.h>

int main(int argc, char *argv[]) {
    if (argc < 2) {
        fprintf(stderr, "Usage: %s arg1 arg2 ...\n", argv[0]);
        return EXIT_FAILURE;
    }

    /*
    ...
    Fill in code
    ...
    */
}

编辑：我觉得这里发生了一些误解。如果您放置“int s = strlen(argv[1]);”在 sn-p "if (argc != 2) ..." 之前，caesar 程序将尝试在不存在的字符串上调用 strlen，从而导致 seg 错误。本质上，总是先写错误检查代码（“if (argc != 2) ...”），然后再做其他事情。此外，“if (argc != 2) ...”之后不需要 else。我觉得摆脱 else 关键字并将正文保留在 else 子句中会更好。