结构C的2d数组的未定义行为

时间:2019-03-14 00:54:03

标签: c csv multidimensional-array struct undefined-behavior

我有一个二维结构数组,我将字符串分配给它,这是我的结构。

struct node {
  char* value;
};

这是我的分配(我是C的新手,所以不确定是否正确),但总会有35列,但可能有数百万行。(我现在只有3列用于测试)

const int rows=3;
    struct node ** arrayofnodes[rows][35];
for(int i=0; i<rows; i++) {
    array[i] = malloc(test * sizeof array[0]);
    for(int j=0; j<35; j++) array[i][j] = malloc(sizeof array[0][0]);
}

然后我从一个csv文件中逐个字符地读取并拥有一个临时字符串,然后通过以下操作将temp的值分配给我想要的位置。

//int row and count are defined in my while loop I have for counting commas(or what col I am on) then new lines for the rows 
arrayofnodes[row][count]->value=strdup(temp);
   printf("%s  \n", arrayofnodes[row][count]->value);
   printf("%d %d \n",row, count );

当我按照上述方式分配时,它似乎起作用了。我添加了这些打印语句,以确保它分配了正确的值。

例如上面的示例将打印出

Red
0 0

这对那个位置是正确的。

但是在我完成所有分配之后。我放置了一条打印语句printf("%s \n", arrayofnodes[0][0]->value);,以测试是否可以检索如上所示的第一个值,该值为“红色”。

在我的终端中,它输出“ @`??”或“ @Pz?”或任何随机输出。除了0,0以外,我还尝试了很多其他方法,但是它们都得到相同的结果。我想我只是感到困惑,为什么打印语句在分配它们后立即起作用,而在我稍后调用它们时不在代码末尾起作用。

2 个答案:

答案 0 :(得分:0)

您可以通过在结构内部使用恒定大小来避免过于复杂的分配:

struct OneRow
{
    char Value[35];
}

const int Rows=3;

OneRow *MyArray=NULL;

MyArray = (OneRow*) malloc (Rows*sizeof(OneRow));

您现在可以按以下方式访问每个元素(字符)或整个字符串

MyArray[rownumber].Value[colnumber] = …
strcpy (MyArray[rownumber].Value, "I'm_shorter_than_35"); //34 chars max + null-term

答案 1 :(得分:0)

这就是您要尝试执行的操作。您将需要扫描csv文件并计算所需的行数,然后根据需要填充值。

#include <stdio.h>
#include <stdlib.h>

struct node {
  char* value;
};

int main() {
  const int rows = 3; // you will need to compute this beforehand
  const int columns = 35;

  struct node** arrayofnodes = malloc(rows * sizeof(struct node*));

  for (int i = 0; i < rows; ++i) {
    arrayofnodes[i] = malloc(columns * sizeof(struct node));
  }

  for (int i = 0; i < rows; ++i) {
    for (int j = 0; j < columns; ++j) {
      arrayofnodes[i][j].value = malloc(...);
      strcpy(arrayofnodes[i][j].value, ...); // etc..
    }
  }

  for (int i = 0; i < rows; ++i) {
    for (int j = 0; j < columns; ++j) {
      free(arrayofnodes[i][j].value);
    }
  }

  for (int i = 0; i < rows; ++i) {
    free(arrayofnodes[i]);
  }

  free(arrayofnodes);
}