将CSV内容插入到struct

时间:2019-06-26 17:49:51

标签: c arrays function csv struct

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

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并没有告诉我我发誓,好像我的那一刻被挡在了第一柱上

1 个答案:

答案 0 :(得分:2)

  

printf没有告诉我我发誓

您的代码中有几个错误会导致这种意外行为。


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

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

所以您读/写了这些字段,并且 team 中的 nome 也很可能遇到相同的问题。

例如,简单的方法是使用数组(我对 rualo 的字符串任意使用大小为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);

在这种情况下,请不要忘记删除条目/条目或 array 而不创建内存时动态释放 char char 数组泄漏。


也有

giocatore* lista_giocatori[numero_giocatori];

通话

giocatori_in_array(file_giocatori,*lista_giocatori);

无效,因为*lista_giocatorilista_giocatori[0],它是 giocatore ,但是giocatori_in_array等待数组 giocatore

可以

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

giocatori_in_array未获得 array 的大小,因此您也有可能写出 array 的风险,因为文件中的条目数比值giocatori_in_array

您还需要知道在数组中设置了多少个条目,一种方法是返回该数字而不是具有 void 函数。


在这些无效的内存访问之外,您不能使用 fscanf 通过单个%s读取多个单词,因此您需要在 ruolo <之后读取其余的行。 / em>提取州和大学,并且如果 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;
}