0

我对 C 语言和分配内存/使用指针相当陌生。无论如何，我正在尝试读取文件，将这些值放入结构中，等等。我确切地知道我想要做什么，当然程序运行但输出不正确并且某种混乱的数字和字母。

每行都有一个包含新信息的文本文件。每条线代表一个对象。

文件中的一行可能如下所示：

肉牛腩 6.55 8 8.50 4

总的来说，我希望能够将所有PRODUCT对象存储在一个数组中（所以我有一个结构数组）。所以我尝试使用指针分配内存，使用行数，然后将指针发送到名为 read 的函数。在读取中，我通过指针将每个结构添加到数组中。该程序没有崩溃，输出不正确，我不知道为什么不。这很可能是带有指针的东西。如果有人可以帮助我，我将不胜感激。任何帮助都会很棒。

      //prototype
void read(pointerToArr);


typedef struct
{
    char supType[15];
    char prodName[15];
    double wholePrice;
    int quantWhole;
    double retPrice;
    int retProdQuantity;
}PRODUCT;

FILE *fr;
int lineCount = 0;
int main()
{
    PRODUCT *ptr;

    int i;
    char check[50];


     fr = fopen("ttt.txt", "r");

      while(fgets(check, sizeof(check), fr)!= NULL)
      {
          if(check[0] != '\n')
          {
              lineCount++;
          }
      }
    // allocate memory for array based on line count.
     PRODUCT prodContainter[lineCount];
     ptr = (PRODUCT*)malloc(sizeof(PRODUCT)*lineCount);
     ptr = prodContainter;

     read(ptr);

      //print after adding to array via pointer from other
      //function. this was a test.

      for(i = 0; i < lineCount; i++)
      {
          printf("%s ", prodContainter[i].supType);
          printf("%s ", prodContainter[i].prodName);
          printf("%f ", prodContainter[i].wholePrice);
          printf("%d ", prodContainter[i].quantWhole);
          printf("%f ", prodContainter[i].retPrice);
          printf("%d\n\n", prodContainter[i].retProdQuantity);

      }

    return 0;
}



void read(PRODUCT *pointerToArr)
{

    // objective in this method is to read in data from the file, create an object for every line and
    // then use the pointer array to add those objects to prodConstainer up above.

       char supplyName[15];
       char productName[15];
       double wholeP = 0;
       int  quantityWhole = 0;
       double retailPrice = 0;
       int retailProductQuant = 0;

    while(fscanf(fr, "%s %s %lf %d %lf %d", supplyName, productName, &wholeP, &quantityWhole, &retailPrice, &retailProductQuant) == 6)
    {
        PRODUCT record;
        int i;

        strcpy(record.supType, supplyName);
        strcpy(record.prodName, productName);
        record.wholePrice = wholeP;
        record.quantWhole = quantityWhole;
        record.retPrice = retailPrice;
        record.retProdQuantity = retailProductQuant;

        for(i = 0; i < lineCount; i++)
        {
            pointerToArr[i] = record;
        }
    }

    fclose(fr);
}
4

1 回答 1

2

您永远不会倒带文件，因此计算行数后的所有读取都会失败。

您正在打印的只是内存中的内容。

当然，有很多方法可以解决这个问题。

  1. 倒带文件，使用rewind()
  2. 关闭文件并让您的read()函数（其名称与 POSIX 标准函数冲突，顺便说一句）重新打开文件。这也将涉及删除可怕的全局变量fr。
  3. 重新构建结构，这样您就不会计算行数，只需读取并让ptr数组根据需要增长（请参阅 参考资料realloc()）。

此外，您应该真正避免强制转换in C的返回值malloc()。这：

ptr = (PRODUCT*)malloc(sizeof(PRODUCT)*lineCount);

最好写成：

ptr = malloc(lineCount * sizeof *ptr);

这消除了强制转换，并且还使用sizeof 指向的类型的值 来自动计算要分配的正确字节数。

于 2013-01-31T16:34:31.540 回答