Question

我一直在完成一项已完成的任务，我需要做的就是释放双指针分配的内存，但是当我通过循环时它会触发一个断点并且似乎没有清楚这一切我不确定是什么触发它。上下文：Create Data函数基本上是一个返回malloc的函数。

int main(void){



int i;
    static STUDENT** records; 
    static float averageTotal[SIZE];
    createData(); 
    records = calloc(3, sizeof(STUDENT*));
    for (i = 0; i < 3; i++){
        records[i] = createData();
    }

    menu(records, userinput, &averageTotal[0]); 
    int c;
    for (c = 0; c < 3; c++){


            free(records[c]);


    }
    free(records);


    return 0;

}
STUDENT* createData()
{
    STUDENT* result;
    result = malloc(3*sizeof(STUDENT*));
    return result;
}

编辑：添加了上下文菜单

void menu(STUDENT* records, int user_size, float* averageTotal[]){
    int menuV = 0;
    do{

        int userinput;  
        int i;
        static int counter = 0;
        printf("       **********************************\n");
        printf("       *        MENU                    *\n");
        printf("       *  1. Add Student                *\n");
        printf("       *  2. Display all student records*\n");
        printf("       *  3. Quit                       *\n");
        printf("       **********************************\n");
        scanf_s("%d%*[^\n]", &userinput); '\n' == getchar();
        switch (userinput){
            // switch statement with loops.
        case 1:do
        {
            addStudent(records, &counter, user_size);
            printf("Add another record? 1(y) 2(n)\n");
            scanf_s("%d%*[^\n]", &userinput); '\n' == getchar();
        } while (userinput == 1);
        break;
        case 2:
        do{
            displayStudent(records, user_size);
            printf("Display again? 1(y) 2(n)\n");
            scanf_s("%d%*[^\n]", &userinput); '\n' == getchar();
        } while (userinput == 1);
        break;

        case 3:
            menuV++;
                break;





        }
    } while (menuV == 0);
}

编辑：为上下文添加了addStudent函数

void addStudent(STUDENT* records[], int* counter, int user_size)

{ // A simple function that lets the user add values to the struct. The counter keeps track and changes the element.
    printf("Student: %d", (*counter+1));
    printf(" of 3\n");
    printf("Enter name:\n");
    fgets(records[*counter]->name, 40, stdin);
    printf("Enter Exam 1 Score: \n");
    scanf_s("%f", &records[*counter]->exam1); 
    printf("Enter Exam 2 Score: \n");
    scanf_s("%f", &records[*counter]->exam2);
    printf("Name:%s",records[*counter]->name);
    printf("Exam 1:%0.2f\n", records[*counter]->exam1);
    printf("Exam 2:%0.2f\n", records[*counter]->exam2);
    (*counter)++;


}

Answer 1

这里有很多错误。你似乎对指针和解除引用的把握很弱。

这一行：

static STUDENT** records;

创建一个变量，该变量是指向数据类型指针的指针。

理想情况下，您应该将声明更改为惯用且更容易理解的内容。类似的东西：

records = calloc(3, sizeof(*records));

在函数CreateData（）中，您将records的成员分配给指向3个指针的指针。

这一行：

result = malloc(3 * sizeof(*STUDENT));

应阅读：

result = malloc(3 * sizeof(STUDENT));

。秒将内存分配为数据类型STUDENT长度的三倍。（你想要什么）。

继续前进 -

以下是函数menu的声明/定义。

void menu(STUDENT* records, int user_size, float* averageTotal[])

以下是您在代码中使用它的方式：

menu(records, userinput, &averageTotal[0]);

第一个参数records被定义为指向STUDENT的指针。但是menu的定义只需要一个指向学生的指针。

查看代码的其余部分，它的定义是错误的，而不是函数调用。

相同的功能：

void menu(STUDENT* records, int user_size, float* averageTotal[])

menu期望指向第三个参数的指向浮点指针。但是你在调用中提供了一个指向索引的指针：

menu(records, userinput, &averageTotal[0]);

如果不进一步阅读，很明显你需要更好地理解指针（或者停止输入这么快并理解你正在写的东西！）。

最后，一个实际的挑选。应该删除函数CreateData（），并且应该直接用函数调用calloc / malloc（）替换它的使用。

free（）循环，触发一个断点，似乎没有释放元素

1 个答案: