重新分配后的动态内存存储问题 - C

Question

对于学校的作业，我们必须使用结构体来创建矩阵，该矩阵可以为无限量的矩阵存储无限量的点。 （理论上无限）

对于分配，我决定使用 calloc 和 realloc。矩阵的大小是这样的：每当其点数达到极限时，它的大小就会加倍（因此它从 1 开始，然后变为 2，然后为 4，依此类推）。每次添加矩阵时，它的大小也会加倍。

这就是我的问题所在。添加初始矩阵后，它会添加第二个矩阵名称和点，它给出以下内容：

B????????????????????? ??????????????????????????????????????????????????? ??????????????????????????????????????????????????? ??????????????????????????????????????????????????? ??????????????????????????????????????????????????? ??????????????????????????????????????????????????? ??????????????????????????????????????????????????? ??????????????????????????????????????????????????? ??????????????????????????????????????????????????? ??????????????????????????????????????????????????? ??????????????????????????????????????????????????? ??????????????????????????????????????????????????? ??????????????????????????????????????????????????? ??????????????????????????????????????????????????? ??????????????????????????????????????????????????? ??????????????????????????????????????????????????? ??????????????????????????????????????????????????? ??????????????????????????????????????????????????? ??????????????????????????????????????????????????? ??????????????????????????????????????????????????? ??????????????????????????????????????????????????? ??????????????????????????????????????????????????? ??????????????????????????????????????????????????? ??????????????????????????????????????????????????? ??????????????????????????????????????????????????? ??????????????????????????????????????????????????? ????????????????????????

B 是我想要的部分（因为我稍后使用 strcmp），但是 ?标记不应该在那里。 （显然）

我不确定它为什么要这么做。由于代码是模块化的，因此很难获取其中的某些部分来准确显示它是如何进行的。

注意：我可以通过以下方法访问矩阵的点：MyMatrix[1].points[0].x_cord;（这只是一个示例）

产生问题的示例代码：

STRUCTS:

 struct matrice {
    char M_name[256];
    int num_points[128];
    int set_points[128];
    int hasValues[1];
    struct matrice_points * points;
} * MyMatrix;
struct matrice_points {
    int set[1];
    double cord_x;
    double cord_y;
};

Setup矩阵函数：

void setupMatrix(){

    MyMatrix = calloc(1, sizeof(*MyMatrix));
    numMatrix = 1;

}

增长矩阵函数：

void growMatrix(){

    MyMatrix = realloc(MyMatrix, numMatrix * 2 * sizeof(*MyMatrix));
    numMatrix = numMatrix * 2;

}

添加矩阵函数，在增长一次矩阵后输出此问题。

void addMatrix(char Name, int Location){

    int exists = 0;
    int existsLocation = 0;
    for (int i = 0; i < numMatrix; i++){
        if (strcmp(MyMatrix[i].M_name, &Name) == 0){
            exists = 1;
            existsLocation = i;
        }
    }

    *MyMatrix[Location].M_name = Name;
    printf("Stored Name: %s\n", MyMatrix[Location].M_name);
    *MyMatrix[Location].num_points = 1;
    *MyMatrix[Location].set_points = 0;
    *MyMatrix[Location].hasValues = 1;
    MyMatrix[Location].points = calloc(1, sizeof(*MyMatrix[Location].points));

}

Answer 1

尝试将“\0”添加到数据末尾。

Answer 2

*MyMatrix[Location].M_name = Name;

您在此处复制单个字符，而不是字符串。如果您想要一个字符串，Name 应定义为 char *，并且您应使用 strcpy。

Answer 3

void addMatrix(char Name, int Location)

char Name代表单个char，即整数类型的数量。 char 只是一个数字，它根本不是一个字符串。

当您执行此操作时：

strcmp(..., &Name)

您假设存储该字符的位置代表有效的 C 字符串。这是错误的，没有理由会这样。如果要将 C 字符串传递给此函数，则需要像这样声明它：

void addMatrix(char *Name, int Location)

然后需要将该 C 字符串复制到矩阵结构中的适当位置。它应该看起来像：

strncpy(... .M_name, Name, max_number_of_chars_you_can_store_in_M_Name);

另外，这些字段定义在您的结构中很奇怪：

int num_points[128];
int set_points[128];
int hasValues[1];

这意味着您的结构将包含一个名为 num_points 的 128 个整数数组，另一个名为 set_points 的 128 个整数数组，以及一个名为 hasValues 的一个整数（奇怪）数组。如果您只需要存储总点数和设定点的计数，以及指示是否存储值的标志，则定义应该是：

int num_points;
int set_points;
int hasValues;

并更正 addMatrix 函数中的分配。

如果您确实需要这些数组，那么您的分配也是错误的。

请打开编译器中的所有警告。