1

解决我必须将文件中的数据读取到结构中的问题。

该文件被组织成有一个名称、几行以# 结尾的ASCII 艺术和一个等级。这是一个例子

Sample Name
( S )
( S )
# 5

我的结构设置如下：

typedef struct
 {
   char* name;
   char* art;
   int rating;
 }CASE;

segmentation fault我的问题是，当我尝试在下面的代码中为名称字符串动态分配内存时，我一直得到一个：

/*FPin is file pointer to the txt file and all is the array of structs*/
void readFile(FILE* FPin, CASE** all)
{ 
  CASE* walker = *all;
  int count = 0;
  char buffer[160];
  char* bufferPtr = buffer;
  char nameBuffer[100];

  /*Reads in the name*/

  while(fscanf(FPin, "%[^\n]", nameBuffer))
  {
    printf("string is %s\n", nameBuffer);
    walker->name = (char*)malloc(sizeof(char)*(strlen(nameBuffer+1)));  /*ERROR*/  
    strcpy(walker->name, nameBuffer);
  } 

  return;
  }

我记下了我认为错误在上面的代码中的位置，因为一旦我添加了该行，我就开始得到它。

我基本上是将文本中的名称读入 nameBuffer （数组），然后使用 strcpy 将此名称复制到结构中。关于如何解决这个问题的任何建议？

感谢您的关注。

我将在下面包括我的其余来源：

int main (void)
{
  CASE* all;
  FILE* FPin;

  if((FPin = fopen("art.txt", "r")) == NULL)
  {
    printf("Error opening file.");
    exit(100);  
  }

  allocateStructMem(&all);
  readFile(FPin, &all);

  fclose(FPin);
  return 0;
}

void allocateStructMem (CASE** all)
{
  if((*all = (CASE*)malloc(sizeof(CASE)*1000)) == NULL)
    {
      printf("Fatal memory error!\n");
      exit(1);
   }

  return;
}
4

2 回答 2

3

strlen(nameBuffer+1)

strlen(nameBuffer)+1

当您 malloc 时，您还必须执行以下操作all：

int allocateStructMem(CASE **all)
{
    /*  +----- Note. And no need to cast as malloc returns (void *)
        |                                           */
    if((*all = malloc(sizeof(CASE*) * 1000)) == NULL)

为防止溢出，您必须限制 的长度fscanf，即：

while (fscanf(FPin, "%99[^\n]", nameBuffer) == 1) {

确保您实际上已将某些内容1读入 nameBuffer。

也不strcpy填充——但你可能知道。

于 2012-04-24T01:18:00.130 回答
2

将 +1 移到括号外有帮助吗？

walker->name = (char*)malloc(sizeof(char)*(strlen(nameBuffer)+1));  /*ERROR*/
于 2012-04-24T01:16:53.503 回答