为什么说正在从'c'读取无效数据和未初始化内存c

Question

#include <stdio.h>


int main()
{
    char a[] = "abdgikm", b[] = "cefhjlnop", c[81];
    int i = 0, j = 0, k = 0;
    while (a[i] != '\0' && b[j] != '\0')
    {
        if (a[i] < b[j]) { i++; j++; }
        else c[k] = b[j];
        k++;
    }
    c[k]=c[k-1];
    printf("%s\n", c);

}

Answer 1

你的代码试图合并两个字符串"abdgikm"和"cefhjlnop"，并在合并过程中，如果遇到其中一个字符串中的字符比另一个中的字符小，就先移动较小的字符。然后，你的代码将结果存储在字符数组"c"中，并打印出结果。

然而，你的代码中存在几个问题：

在C语言中，字符串是以'\0'（空字符）结尾的。在你的代码中，数组"c"并没有初始化，这意味着它可能包含任何随机数据。如果你试图访问未初始化的内存，程序可能会产生未定义的行为，这可能包括崩溃、产生垃圾数据，或者其他的异常行为。
在你的代码中，你试图将字符数组"b"中的字符逐个复制到"c"中，但你并没有为数组"c"预留足够的空间来存储完整的字符数组"b"。因此，你可能在试图向数组"c"写入数据时超出其界限，这也可能导致未定义的行为。
解决上述问题的一种方法是：

初始化数组"c"，使其内容为'\0'。
确保你有足够的空间来存储合并后的字符串。
修改后的代码如下：

#include <stdio.h>

int main()
{
    char a[] = "abdgikm", b[] = "cefhjlnop", c[81] = ""; // initialize c with empty string
    int i = 0, j = 0, k = 0;
    while (a[i] != '\0' && b[j] != '\0')
    {
        if (a[i] < b[j]) { i++; j++; }
        else { c[k] = b[j]; j++; } // update j here to ensure we copy all characters from array b
        k++;
    }
    while (a[i] != '\0') { c[k++] = a[i++]; } // copy remaining characters from array a, if any
    c[k] = '\0'; // append the null character at the end of the string
    printf("%s\n", c);
}

这段代码会先比较两个数组中的字符，然后复制较小的字符，然后再复制剩余的字符。最后，它会在字符串的末尾添加一个空字符，以确保字符串以正确的格式结束。