分段故障（核心倾倒）CS50错误

Question

我一直在编写针对CS50（Caesar）的程序，该程序应该使用命令行中用户给出的密钥来加密消息。该程序应该测试针对许多条件的密钥。我希望我的程序使用返回值在给定的命令行参数无效时停止运行。

下面的代码：（到目前为止，我尚未完成它，仅检查命令行输入）

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

int char_repeat(string key);
int key_check(string key);

int main(int argc, string argv[])
{
    string key = argv[1];
    for (int i = 0; i < strlen(key); i++)
    {
        char c = toupper(key[i]);
        // conditions for key (string) written in argv[1]
        if (argc < 2 || strlen(key) != 26 || c < 65 || c > 90)
        {
            printf("Usage: ./substitution key\n");
            return 1;
        }
    }
    char_repeat(key);
    printf("this should not print if invalid...");
    return 0;
}



int char_repeat(string key)
{
    // check for repeated characters in key
    for (int i = 0; i < strlen(key); i++)
    {
        for (int j = i + 1; j < strlen(key); j++)
        {
            if (toupper(key[i]) == toupper(key[j]))
            {
                printf("Usage: ./substitution key\n");
                return 1;
            }
        }
    }
    printf("this should not print if invalid...");
    return 0;
}

当我给出“无效”输入（例如

$ ./sub AB*DEFGHIJKLMNOPQRSTUVWXYZ
$ ./sub ABC
$ ./sub ABCABCABCABCABCABCABCABCAB 
$ ./sub ABCDEFGHIJKLMNOPQRSTUVWXYZ- 

代码）时，通过打印“用法：./substitution key \ n”，请跳过我的测试打印。消息，“如果无效，这不应打印...”。

但是，如果我在不给它的情况下进行第二个论点，这就是问题所在。也就是说，

$ ./sub 

收到一条错误消息

segmentation fault (core dumped) 

我在代码本身中 ，我在其中一个if语句中使用条件（argc＆lt; 2），因此它不应该作为所有其他条件并打印“用法：./substitution key \ n “？我知道细分故障是一个错误，试图访问不应访问的内存或其他类型的内存损坏，我只是看不到这与我的程序有何关系。我想这是我使用返回方式的问题。

先感谢您！

Answer 1

分段故障是试图访问不属于您的内存引起的错误。

当您在不提供其他参数的情况下运行命令./ sub程序中的代码试图访问不存在的参数2时，您无法访问该内存时。

string key = argv[1];

相反，您应该在尝试访问已传递的参数的内存之前检查。

if(argc < 2)
{
    printf("Usage: ./substitution key\n");
    return 1;
}

string key = argv[1];