分配或处理指向结构的指针数组时出现问题

Question

我有一个问题（或者可能有几个）
我正在努力解决指向 struct 的指针的问题，但我不明白是内存分配问题，还是将正确的东西传递给我的函数的问题。
这是（部分）代码

typedef struct Corsa {
  char codice[15];
  char partenza[15];
  char destinazione[15];
  int giorno, mese, anno, oraP, minP, oraA, minA, ritardo;
} corsa;

corsa** allocaStructArray(int numStruct);
void leggiEStampaStruct(FILE *fp, corsa **pntToCorsa, int numR);

// ..main and other functs..

corsa** allocaStructArray(int numStruct) //numStruct is the number of 'rows'
{
  corsa **pointerToStructCorsa;
  pointerToStructCorsa = malloc(numStruct * sizeof(corsa));

  if (pointerToStructCorsa == NULL)
    printf("\nErrore di allocazione.\n");

  return pointerToStructCorsa;
}

void leggiEStampaStruct(FILE *fp, corsa **pntToCorsa, int numR) {
  int i;
  fclose(fp);
  fp = fopen("file.txt", "r");
  if (fp == NULL)
    exit(232);

  for (i = 0; i < numR; i++) {
    fscanf(fp, "%s %s %s %d/%d/%d %d.%d %d.%d %d", pntToCorsa[i]->codice,
        pntToCorsa[i]->partenza, pntToCorsa[i]->destinazione,
        &pntToCorsa[i]->giorno, &pntToCorsa[i]->mese, &pntToCorsa[i]->anno,
        &pntToCorsa[i]->oraP, &pntToCorsa[i]->minP, &pntToCorsa[i]->oraA,
        &pntToCorsa[i]->minA, &pntToCorsa[i]->ritardo);

  }

  for (i = 0; i < numR; i++) {
    printf("\n%s %s %s %d/%d/%d %d.%d %d.%d %d\n", pntToCorsa[i]->codice,
        pntToCorsa[i]->partenza, pntToCorsa[i]->destinazione,
        pntToCorsa[i]->giorno, pntToCorsa[i]->mese, pntToCorsa[i]->anno,
        pntToCorsa[i]->oraP, pntToCorsa[i]->minP, pntToCorsa[i]->oraA,
        pntToCorsa[i]->minA, pntToCorsa[i]->ritardo);
  }

  return;
}

我不喜欢的另一件事是我无法手动检查是否所有东西都在原位。

就像它是一个普通矩阵一样，我可以检查值是否放置在正确的位置，但使用指针我确实不能，这无助于发现问题。

你们对这个其他问题有什么建议吗？

Answer 1

至少这些问题

fscanf() 中没有宽度

不要在fscanf() 中使用没有宽度限制的"%s"。使用 "%14s %14s %14s ... 作为每个指向 15 的数组。

检查输入结果

检查返回值。添加一个测试，看看这 11 个说明符是否都被解析了。

 if (fscanf(....) != 11) Handle_Error();

分配错误

sizeof 输入错误。

通过调整被引用对象的大小而不是而不是类型来避免这个问题。更容易正确编码、审查和维护。另外，不清楚numStruct < 0时OP想做什么，所以加了个测试。

// Wrong size                                v-----------v 
// pointerToStructCorsa = malloc(numStruct * sizeof(corsa));
if (numStruct >= 0) {
  //                            v--------------------------v Right size for referenced object. 
  pointerToStructCorsa = malloc(sizeof *pointerToStructCorsa * numStruct);
  ...