Question

我目前正在研究C编程课程的项目，并遇到了一个问题。我仍然是C语言的新手，但有编程Java的经验。

当我尝试将值分配给 struct 的 char * 变量成员时，我的程序遇到了问题。有问题的代码如下：

void ReadFile(ComponentType * Data, int * numEl, int * numNodes)
{
  size_t index;
  FILE * dataFile;
  char * data;
  float * value;

  numEl = malloc(sizeof(int *));
  numNodes = malloc(sizeof(int*));
  Data = malloc(6 * sizeof(ComponentType));

  * numEl = 0;
  * numNodes = 0;
  index  = 0;   

  if((dataFile = fopen("mydata.dat", "r")) == NULL)
      puts("Error - file \"mydata.dat\" could not be opened!");
  else
  { 

      while(!feof(dataFile)) 
      { 

          fscanf(dataFile, "%s", data);
          Data[index].name = (char *)malloc(strlen(data)+1);
          strcpy(Data[index].name, data);

          fscanf(dataFile, "%s", data);
          Data[index].node1 = (char *)malloc(strlen(data)+1);
          strcpy(Data[index].node1, data);

          fscanf(dataFile, "%s", data);
          Data[index].node2 = (char *)malloc(strlen(data)+1);
          strcpy(Data[index].node2, data);

          fscanf(dataFile, "%f", value);
          Data[index].value = * value;


          int i = char2int(CircuitData[index].node1);
          if(i > * numNodes)
              * numNodes = i;

          i = char2int(CircuitData[index].node2);
          if(i > * numNodes)
              * numNodes = i;

          (* numEl)++;
          (* numNodes)++;
          index++;
      }

      fclose(dataFile);
  }

  free(Data);
  free(numEl);
  free(numNodes);       
}

结构定义如下：

typedef struct
{
  char * name;
  char * node1;
  char * node2;
  float value;
} ComponentType;

我已对程序运行调试，基本上发生的是文件读取正确，数据中存储的值是正确的，但打印出数据[索引] .name < / strong>在格式化为 char * 时导致分段错误，或者在调试程序中打印出0x0。简而言之，从文件中读取的值不会被复制到我想要的结构变量中。

任何建议/帮助将不胜感激。谢谢。

编辑：我正在使用Linux 64位代码

编辑2：我添加了发生错误的整个函数以及下面建议的更改

Answer 1

首先：调试程序时，请检查sizeof(data)是什么你会惊讶地发现它是8（指针的大小），而不是6（你分配的大小）
这是因为sizeof测量了类型的大小，data是char*，（也就是指针）。

指针是8字节。 （在64位系统上。其他系统不同）
这会导致您为Data[index].name分配8个字节。

下一步：您无法使用=运算符将数据从一个数组复制到另一个数组，正如您尝试的那样：

Data[index].name = data;

要做得好：

fscanf(dataFile, "%s", data);
// Measure the length of data, and allocate enough space (including the null-termiantor)
Data[index].name = malloc(strlen(data) + 1);  

// Copy from data into Data[index].name.   For greater safety, look into strcpy_s
strcpy(Data[index].name, data);

将值分配给结构成员

1 个答案: