printf не показывает мне, что я клянусь
есть несколько ошибок в вашем коде, приводящих к этому неожиданному поведению.
In
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);
вы предполагаете ном и когном и руоло и колледж являются массивом char , но они всего лишь char :
char nome;
char cognome;
...
char ruolo;
...
char college;
, поэтому вычтение / запись этих полей, и у вас, вероятно, возникнет та же проблема для nome в team .
Самый простой способ - использовать массивы, например (Я использую произвольно размер 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 */
и сделать то же самое для ном в команде (определение скрыто).
Вы также можете использовать указатель на char для nome и cognome и ruolo (и то же самое для 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);
Не забудьте в этом случае free динамически распределенные массивы 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 функция.
Из этих недействительных мемДля доступа к ory вы не можете использовать fscanf для чтения нескольких слов через один %s, поэтому вам нужно прочитать остаток строки после 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;
}