C 编程中的 RPG 库存

Question

我是 C 的新手，对于我的班级，我们有一个项目，我们必须在其中编写一个函数，该函数读取文件中的所有项目并以文件顺序在动态创建的列表中返回它们。 文件的每一行都是一项，格式如下：

'<description>' <damage> <cost> <weight>

它应该按文件顺序返回指向文件中项目的基指针，如果文件不存在，则返回 NULL。

到目前为止，这是我的代码：

item_t *ReadItemsFromFile(char *file)
{
typedef struct item item_t;
struct item
{
   char name[32];
   float cost, weight;
   int dam;
};FILE *fpin = fopen(file, "r");

if(fpin != NULL)
{
  item_t i[20];
  int n = 0;
  char line[sizeof(file)];
  while(fgets(line, sizeof(line), fpin) != NULL){
      (fscanf(fpin, " '%[^']' %d %f %f", i[n].name, &i[n].dam, &i[n].cost, &i[n].weight));
  fputs(i[n].name, stdout);
  n++;
}
} else {
  return NULL;
} 
  return(0);
}

我一直在使用 fputs 来尝试测试代码，但它总是让我在第一个和最后一个项目上出现乱码。 另外，当我尝试放

fputs(&i[n].dam, stdout);

所以我可以测试结构的其他变量，我不断收到错误消息“从不兼容的指针类型传递 'fputs' 的参数 1。

我不确定我是否使用 fscanf 准确地将结构变量传递给了结构变量，或者它是否是其他东西。

Answer 1

您需要将struct 定义移到您的函数之外。

您对线应该多长的规范是错误的。您需要指定足够长的字节数以容纳一整行。

当您使用fgets() 获取一行时，您不应该使用fscanf() 从文件中读取。您已经在使用 fgets() 从文件中读取。您使用sscanf() 从包含在line 中的字符串中读取数据。

以上内容足以让你的程序做一些名义上有用的事情。

要返回项目列表，您需要实际调用malloc()（或其他类似函数）来为您指定为要求的“动态创建的列表”创建内存。由于您不知道自己有多少项目，您要么需要一种方法来发现有多少项目，要么使用一种机制来让您的列表动态增长。

Answer 2

一个问题是您使用fgets 将文件中的一行读入line，但随后您忽略该行并使用fscanf 从下一行读取。您应该使用sscanf 来解析您刚刚阅读的行。或者，根本不要使用fgets，而直接使用fscanf作为while条件：

while(4 == fscanf(fpin, " '%[^']'%d%f%f", i[n].name, &i[n].dam, &i[n].cost, &i[n].weight)) {
    fputs(i[n].name, stdout);
    n++; }

另一个问题是您声明 line 刚好足以容纳 sizeof(char *) 字符 - 可能只有 4 或 8 个，对于整行来说不够大，所以您只能阅读线的一部分。您需要声明它足够大以容纳文件中最长的行。

第三个问题是您将 struct item 和 item_t 声明为函数的本地函数，这意味着当您尝试在函数外部使用 item_t 作为其返回类型的一部分时，您的代码甚至无法编译.您需要在函数声明之前将声明移动到全局范围。

第四个问题是您将正在读取的项目数组 (i) 声明为局部变量，因此您将无法从函数中返回它——如果这样做，指针你返回会指向垃圾。但由于你总是返回 NULL，你不会看到这个问题。

Answer 3

fputs() 的问题是该函数专门用于打印 char * 字符串。要打印出其他内容，您可能需要使用fprintf()，或者如果您要打印到标准输出，则最好使用printf()。

要消除乱码，请确保您的文件以正确的编码保存。当我将我的测试库存文件保存为没有 BOM 的 ANSI 或 UTF-8（ANSI 为 UTF-8）时，一切正常，但每当我保存为 UTF-8 时，它会在我的文件前面插入 3 个字节，程序不能处理，并为我的第一个项目吐出胡言乱语。

我还修改了ReadItemsFromFile()，如您所见，接受指向int 的指针，这将是int，读取的元素数量将存储在其中，因此我可以测试我的代码，您可以如果它不符合您班级的规范，请将其删除。此外，它非常粗糙，但我在代码中添加了使用malloc() 和memcpy() 重新调整存储项目数组的内存块的大小（分别需要stdlib.h 和string.h ) 因为realloc() 不能在我运行可执行文件的环境中工作，但如果realloc() 适合你，我建议改用它。

还请注意，我放置了 128 个字符的任意缓冲区长度，包括每行的 \0 空终止符。这可以调整为适合您的任何内容。

这是我一直在谈论的代码：

struct item
{
    char name[32];
    float cost, weight;
    int dam;
};
typedef struct item item_t;

item_t *ReadItemsFromFile(char *file, int *count)
{
    FILE *fpin = fopen(file, "r");
    int n = 0;
    item_t *items = NULL;

    if(fpin != NULL)
    {
        char line[128];
        while(fgets(line, 128, fpin) != NULL){
            item_t *newptr = (item_t *)malloc((n + 1) * sizeof(item_t));
            memcpy(newptr, items, n * sizeof(item_t));
            free(items);
            items = newptr;

            if (4 == sscanf(line, "'%[^']' %d %f %f\n", items[n].name, &items[n].dam, &items[n].cost, &items[n].weight)) {
                n++;
            }
        }

        fclose(fpin)
    } else {
        *count = 0;
        return NULL;
    }

    *count = n;

    return items;
}

Answer 4

好的，我停止使用 fgets() 并将结构移到函数之外，但我仍然遇到返回指针的问题。我在 malloc 后一直试图返回指针，但它告诉我“从不兼容的指针类型返回”。到目前为止，这是我的代码：

struct items_t
{
   char name[32];
   float cost, weight;
   int dam;
};
typedef struct items_t items_t;

item_t *ReadItemsFromFile(char *file)
{

  FILE *fpin = fopen(file, "r");

if(fpin)
{   
  items_t i[50];
  int n = 0;

   while(4 == (fscanf(fpin, " '%[^']' %d %f %f", i[n].name, &i[n].dam, &i[n].cost, &i[n].weight)))
{
  n++;
  items_t *a = (items_t*)malloc(sizeof(items_t));
  return(&a);

}
} else {
  return NULL; 
}
return(0);

}