使用 cs.50.h 中的“字符串”在 C 代码中出现分段错误

Question

我在这里有一个程序，我正在尝试使用凯撒密码来解码一串字母；本质上，我将字符串中的每个字符“向下”移动一个字母（“a”->“b”、“f”->“g”、“z”->“a”）。

我向下移动一个字母的数量取决于我给它的键。

在这个特定的程序中，我在 main() 函数中硬编码了一个秘密编码消息，以及一个遍历每个可能键的 for 循环。

这个想法是，如果这个秘密信息简单地向下移动 x 个字母，吐出 25 个版本的密码将揭示一个可理解的答案。

不幸的是，我在一个程序中使用了一些新概念 - argc、argv 和多个函数。我对此很陌生。

有人可以帮助解释我遇到的分段错误吗？我认为我不会导致任何参数溢出。

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

string decode(int key, string message);

int main(void)
{
    string secret_message = "ueuag rKJ AGIQ GIAR FEgN";

    for (int i = 0; i < 25; i++)
    {
        decode(i, secret_message);
        printf("%s", secret_message);
    }

    return 0;
}

string decode(int key, string message)
{
    int i;

    for (i = 0; i < strlen(message); i++)
    {
        if (message[i] >= 'a' && message[i] <= 'z')
        {
            message[i] = ((message[i] - 97 + key) % 26) + 97;
        }
        else if (message[i] >= 'A' && message[i] <= 'Z')
        {
            message[i] = ((message[i] - 65 + key) % 26) + 65;
        }
    }

    return message;
}

Answer 1

为什么string 对于c 中的类型不是一个好主意？，在这里你可以看到一个例子。您正在修改字符串文字，您不应该这样做。这样做会调用未定义的行为。而不是 string 类型，您应该将字符串视为它们在 c 中的内容。

改成这样

char secret_message[] = "ueuag rKJ AGIQ GIAR FEgN";

还有解码功能

char *decode(int key, char *message)
{
    int i;

    for (i = 0; message[i] != '\0'; i++)
    {
        if (message[i] >= 'a' && message[i] <= 'z')
        {
            message[i] = ((message[i] - 97 + key) % 26) + 97;
        }
        else if (message[i] >= 'A' && message[i] <= 'Z')
        {
            message[i] = ((message[i] - 65 + key) % 26) + 65;
        }
    }

    return message;
}

如您所见，我将字符串视为一个数组，因为它就是一个字节数组，末尾带有 '\0'。如果您知道这一点，您将永远不会做类似typedef char * string 的事情，因为它非常具有误导性。

在cs50.h中string是一个typedef代表一个char指针，而指针不是字符串，字符串是字节序列和char *指针可以指向char 的数组，如果你定义它并正确初始化它，它可能是一个字符串。但是 char * 指针可以指向字符串文字，并且您无法更改它们，如果将其定义为则不清楚

string string_literal = "Do not attempt to modify me, it's undefined behavior";

在声明指向字符串文字的指针时，您应该使用const char * 以避免意外尝试修改它。

另外，在我看来，typedefing 指针根本没有任何好处，并且会引起很多混乱，指针是指针，并且在声明时必须在其标识符附近有一个 *，如果你不需要*你很容易忽略代码中的指针而感到困惑。