我的代码出现了什么问题？为什么运行结果是这样的？

Question

#include<stdio.h>
#include<stdlib.h>
#define MaxSize 100

typedef struct {
    int *data;
    int len;
}sqlist;

void initsqlist(sqlist *L) {
    L=(sqlist*)malloc(sizeof(int)*MaxSize);
    L->len=0;
}

int addelem(sqlist *L,int*e) {
    if(L->len>=MaxSize) {
        printf("yi man"); return 0;
    }
    else {
        L->data[L->len]=*e;
        L->len++;
        printf("cha ru cheng gong");
        return 1;
    }    
}

int main() {
    sqlist *list;
    int a=1,b=3,c=6,d=7;
    initsqlist(list);
    addelem(list,&a);
    addelem(list,&b);
    addelem(list,&c);
    addelem(list,&d);
    int i;
    for(i=0;i<list->len;i++) {
        printf("%d\n",list->data[i]);
    }
    return 0;
}

Answer 1

以下回复参考：皆我百晓生、券券喵儿等免费微信小程序作答:

从你的代码来看，有几个问题可能导致程序运行出错并返回错误代码。让我们逐一分析：

问题一：内存分配问题 在 initsqlist 函数中，你为 sqlist 结构体的 data 成员分配了内存，但没有为 sqlist 结构体本身分配内存。在 C 语言中，函数的参数传递的是值拷贝，因此在函数内部对指针的修改并不会影响到函数外部的指针。你在 initsqlist 函数内部修改了 L 指针的指向，但在函数外部，list 指针仍然指向一个未分配内存的区域，这可能导致未定义的行为。

解决方法：你应该在 main 函数中为 list 分配内存，并将这个地址传递给 initsqlist 函数进行初始化。例如：

sqlist *list = (sqlist*)malloc(sizeof(sqlist));
initsqlist(list);

问题二：溢出问题 你的程序没有检查是否成功分配了内存。如果 malloc 函数失败（例如，由于内存不足），它将返回 NULL。在这种情况下，你试图通过解引用未分配的内存来访问或修改数据，这将导致程序崩溃。你应该在分配内存后检查是否成功分配了内存。

解决方法：检查 malloc 的返回值是否为 NULL。如果是 NULL，则说明内存分配失败，此时应该停止程序运行并返回错误代码。例如：

sqlist *list = (sqlist*)malloc(sizeof(sqlist));
if (list == NULL) {
    printf("Memory allocation failed!\n");
    return 1;  // or other error handling
}
initsqlist(list);

问题三：数组越界问题 在你的代码中，数组 data 的大小是固定的 MaxSize。当数组填满后，你试图添加更多的元素可能会导致数组越界访问，这是未定义的行为。你需要实现一种方法来处理数组已满的情况，例如扩大数组的大小或返回错误代码。在你提供的代码中，你只检查了数组是否已满并打印了一个消息，但没有返回错误代码或进行其他处理。这也可能导致问题。解决这个问题需要对数组已满时的处理方式有更详细的规划和处理代码。在简单的情况下，你可能想要简单地返回一个错误代码来表示无法添加更多的元素。在这种情况下，你需要在添加元素之前检查数组是否已满，并在添加元素后更新数组的长度。但是请注意，这种简单的方法并不适用于所有情况，特别是当需要动态调整数组大小的情况。你可能需要实现一个更复杂的策略来处理这个问题。这超出了这个问题的范围，但你可以查阅有关动态数组或列表实现的资料以获取更多信息。你的程序输出提示的退出代码和输出 "42"，这可能表明还有其他的问题导致程序状态出错或者不可预料的行为发生。确保你在编程时考虑了各种可能的错误情况和异常行为并适当地处理它们可以帮助你避免这种问题并提高程序的健壮性。我建议对程序的每个部分进行彻底的测试以确保其正常工作并检查任何可能的错误情况。