c - 将 csv 内容插入结构数组的函数存在问题

Question

好吧，我有一个像这样的结构

typedef struct{
    int id;
    char nome;
    char cognome;
    int eta;
    char ruolo;
    squadra team;
    char college;
    int td;
} giocatore;

我有一个函数可以将 .csv 文件的内容插入到数组中。问题是我的 fscanf 不返回任何内容，并且我的数组始终为空。

我的 .csv 的结构是这样的

1   Kyler   Murray  22  QB  Arizona Cardinals   Oklahoma
2   Nick    Bosa    22  DE  San Francisco   Ohio State
3   Quinnen Williams    22  DE  New York Jets   Alabama
4   Clelin  Ferrell 22  DE  Oakland Raiders Clemson

-数组声明

FILE* file_giocatori;
giocatore* lista_giocatori[numero_giocatori];

-函数声明:

void giocatori_in_array(FILE* f, giocatore array_giocatori[]);

-我如何调用该函数:

giocatori_in_array(file_giocatori,*lista_giocatori);

-功能:

void giocatori_in_array(FILE* f, giocatore array[numero_squadre]){
    size_t count = 0;
    while(fscanf(f, "%d,%s,%s,%d,%s,%s,%s\n", &array[count].id, &array[count].nome, &array[count].cognome, &array[count].eta, &array[count].ruolo, &array[count].team.nome, &array[count].college) == 7)
    {
        printf ("%s %s", &array[count].nome, &array[count].cognome);
        count++;
    }
}

printf 没有显示我发誓的内容，似乎我的 while 被阻塞在第一个循环

Answer 1

The printf don't show me what i swear

您的代码中存在多个错误，导致出现意外行为。

<小时/>

在

while(fscanf(f, "%d,%s,%s,%d,%s,%s,%s\n", &array[count].id,
               &array[count].nome, &array[count].cognome,
               &array[count].eta, &array[count].ruolo,
               &array[count].team.nome, &array[count].college) == 7)

和

   printf ("%s %s", &array[count].nome, &array[count].cognome);

你假设nome和cognome和ruolo和college是char的数组> 但它们只是字符:

char nome;
char cognome;
...
char ruolo;
...
char college;

所以你读/写了这些字段，并且你很可能在团队中的nome上遇到同样的问题。

简单的方法是使用数组，例如(我对字符串任意使用 16 的大小，除了 rualo ，它似乎只使用结束空字符中的 2 个字符):

typedef struct{
    int id;
    char nome[16];
    char cognome[16];
    int eta;
    char ruolo[3];
    squadra team;
    char college[16];
    int td;
} giocatore;

和

while(fscanf(f, "%d,%15s,%15s,%d,%2s,%s,%15s\n", &array[count].id,
             array[count].nome, array[count].cognome,
             array[count].eta, array[count].ruolo,
             array[count].team.nome, array[count].college) == 7)

和

printf ("%s %s", array[count].nome, array[count].cognome);

或更可能

printf ("%s %s\n", array[count].nome, array[count].cognome); /* add \n */

并对 team 中的 nome 执行相同的操作(定义已隐藏)。

您还可以对 nome 和 cognome 和 ruolo 使用指向 char 的指针(对于 <团队中的em>nome):

typedef struct{
    int id;
    char * nome;
    char * cognome;
    int eta;
    char * ruolo;
    squadra team;
    char * college;
    int td;
} giocatore;

和

char nome[16];
char cognome[16];
char ruolo[3];
char college[16];

while(fscanf(f, "%d,%15s,%15s,%d,%2s,%s,%15s\n", &array[count].id, nome,
             cognome, array[count].eta, array[count].ruolo, 
             array[count].team.nome, college) == 7) {
  array[count].nome = strdup(nome);
  array[count].cognome = strdup(cognome);
  array[count].ruolo = strdup(ruolo);
  array[count].college = strdup(college);

在这种情况下，当您删除一个或多个条目或数组时，不要忘记释放动态分配的char数组，以免创建内存泄漏。

<小时/>

还有

giocatore* lista_giocatori[numero_giocatori];

通话

giocatori_in_array(file_giocatori,*lista_giocatori);

无效，因为*lista_giocatori是lista_giocatori[0]，它是一个giocatore，但giocatori_in_array等待giocatore数组。

可以

giocatore lista_giocatori[numero_giocatori];
...
giocatori_in_array(file_giocatori, lista_giocatori);

<小时/>

giocatori_in_array 不会收到数组的大小，因此您也有写出数组的风险，因为数组中有更多条目文件的值大于 giocatori_in_array 的值。

您还需要知道数组中设置了多少条目，一种方法是返回该数字，而不是使用 void 函数。

<小时/>

在这些无效的内存访问中，您无法使用fscanf通过单个%s读取多个单词，因此您需要读取<之后的行的其余部分em>ruolo 提取州和学院，如果 nome 和 cognore 也可以出现在多个单词上，您根本无法使用 fscanf 并且您必须读取所有行，然后提取字段，例如使用 strtok 和 strtol 来获取数值。

请注意，您的 csv 文件示例中缺少用于分隔字段的逗号/分号。您的 fscanf 格式假设您使用逗号，而不是您在问题备注中指出的分号。

<小时/>

提案可以是:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

typedef struct {
  char * nome;
} squadra;

typedef struct{
    int id;
    char * nome;
    char * cognome;
    int eta;
    char * ruolo;
    squadra team;
    char * college;
    int td; /* unused */
} giocatore;

size_t giocatori_in_array(FILE* f, size_t sz, giocatore array_giocatori[]);

int main(int argc, char ** argv)
{
  if (argc != 2)
    printf("Usage : %s <csv file>\n", *argv);
  else {
    FILE* file_giocatori = fopen(argv[1], "r");

    if (file_giocatori == NULL)
      fprintf(stderr, "cannot open '%s'\n", argv[1]);
    else {
      const size_t numero_giocatori = 16;
      giocatore lista_giocatori[numero_giocatori];
      size_t n = giocatori_in_array(file_giocatori, numero_giocatori, lista_giocatori);

      fclose(file_giocatori);

      /* debug */
      for (size_t i = 0; i != n; ++i) {
        printf("id:%d nome:'%s' cognome:'%s' eta:%d ruolo:'%s' team:'%s' college:'%s'\n",
               lista_giocatori[i].id,
               lista_giocatori[i].nome,
               lista_giocatori[i].cognome,
               lista_giocatori[i].eta,
               lista_giocatori[i].ruolo,
               lista_giocatori[i].team.nome,
               lista_giocatori[i].college);
      }

      /* free resources */
      for (size_t i = 0; i != n; ++i) {
        free(lista_giocatori[i].nome);
        free(lista_giocatori[i].cognome);
        free(lista_giocatori[i].ruolo);
        free(lista_giocatori[i].team.nome);
        free(lista_giocatori[i].college);
      }
    }
  }

  return 0;
}

size_t giocatori_in_array(FILE* f, size_t sz, giocatore array[])
{
  size_t count = 0;
  char line[256];

  while ((count < sz) && fgets(line, sizeof(line), f)) {
    char * s = strtok(line, ",;");

    if ((s == NULL) || (sscanf(s, "%d", &array[count].id) != 1)) {
      fprintf(stderr, "invalid id line %zu\n", count);
      break;
    }

    if ((s = strtok(NULL, ",;")) == NULL) {
      fprintf(stderr, "invalid nome line %zu\n", count);
      break;
    }
    array[count].nome = strdup(s);

    if ((s = strtok(NULL, ",;")) == NULL) {
      fprintf(stderr, "invalid cognome line %zu\n", count);
      break;
    }
    array[count].cognome = strdup(s);

    if (((s = strtok(NULL, ",;")) == NULL)  || (sscanf(s, "%d", &array[count].eta) != 1)) {
      fprintf(stderr, "invalid eta line %zu\n", count);
      break;
    }

    if ((s = strtok(NULL, ",;")) == NULL) {
      fprintf(stderr, "invalid ruolo line %zu\n", count);
      break;
    }
    array[count].ruolo = strdup(s);

    if ((s = strtok(NULL, ",;")) == NULL) {
      fprintf(stderr, "invalid team nome line %zu\n", count);
      break;
    }
    array[count].team.nome = strdup(s);

    if ((s = strtok(NULL, ",;\n")) == NULL) {
      fprintf(stderr, "invalid college line %zu\n", count);
      break;
    }
    array[count].college = strdup(s);

    count += 1;
  }

  return count;
}

编译与执行:

pi@raspberrypi:/tmp $ gcc -pedantic -Wall -Wextra c.c
pi@raspberrypi:/tmp $ cat f.csv 
1;Kyler;Murray;22;QB;Arizona Cardinals;Oklahoma
2;Nick;Bosa;22;DE;San Francisco;Ohio State
3;Quinnen;Williams;22;DE;New York Jets;Alabama
4;Clelin;Ferrell;22;DE;Oakland Raiders;Clemson
pi@raspberrypi:/tmp $ ./a.out f.csv 
id:1 nome:'Kyler' cognome:'Murray' eta:22 ruolo:'QB' team:'Arizona Cardinals' college:'Oklahoma'
id:2 nome:'Nick' cognome:'Bosa' eta:22 ruolo:'DE' team:'San Francisco' college:'Ohio State'
id:3 nome:'Quinnen' cognome:'Williams' eta:22 ruolo:'DE' team:'New York Jets' college:'Alabama'
id:4 nome:'Clelin' cognome:'Ferrell' eta:22 ruolo:'DE' team:'Oakland Raiders' college:'Clemson'
pi@raspberrypi:/tmp $ 

在valgrind下执行:

pi@raspberrypi:/tmp $ valgrind ./a.out f.csv 
==4207== Memcheck, a memory error detector
==4207== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==4207== Using Valgrind-3.13.0 and LibVEX; rerun with -h for copyright info
==4207== Command: ./a.out f.csv
==4207== 
id:1 nome:'Kyler' cognome:'Murray' eta:22 ruolo:'QB' team:'Arizona Cardinals' college:'Oklahoma'
id:2 nome:'Nick' cognome:'Bosa' eta:22 ruolo:'DE' team:'San Francisco' college:'Ohio State'
id:3 nome:'Quinnen' cognome:'Williams' eta:22 ruolo:'DE' team:'New York Jets' college:'Alabama'
id:4 nome:'Clelin' cognome:'Ferrell' eta:22 ruolo:'DE' team:'Oakland Raiders' college:'Clemson'
==4207== 
==4207== HEAP SUMMARY:
==4207==     in use at exit: 0 bytes in 0 blocks
==4207==   total heap usage: 23 allocs, 23 frees, 5,637 bytes allocated
==4207== 
==4207== All heap blocks were freed -- no leaks are possible
==4207== 
==4207== For counts of detected and suppressed errors, rerun with: -v
==4207== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 6 from 3)
pi@raspberrypi:/tmp $ 

该提案接受逗号和分号来分隔 csv 文件中的列，但它假设名称周围没有空格(如果名称周围有空格，则它们不会被删除，您必须这样做如果需要的话)。

如果您没有strdup:

char * strdup(char * s)
{
    char * r = malloc(strlen(s) + 1);

    strcpy(r, s);
    return r;
}