fscanf (C/C++)

[Vik27]

Salve a tutti,
devo leggere un file in cui ho una riga di questo tipo

<nome> <numero> <nome1> <nome2> ... <nome numero>

quindi tanti nomi dopo il numero quanto vale il numero

pensavo di poter fare così:

while (!feof(fin))
{
fscanf(fin, "%s %d", nome, numero);

for (int i=0; i < numero; i++)
{
fscanf(fin, "%s", nome);
....
....
}
fscanf(fin, "\n"); //fine riga
}

Che ne dite, potrebbe funzionare?

Grazie

[XWolverineX]

Hai provato almeno???

[Vik27]

Non ne ho la possibilità: il programma devo scriverlo su carta!

[MItaly]

Originariamente inviato da Vik27
Non ne ho la possibilità: il programma devo scriverlo su carta!

:master:
... be', puoi sempre scriverlo sul PC, compilarlo, verificare se va e alla fine stamparlo.

[Vik27]

Beh, mi sa che 'stasera devo provare

[MItaly]

Comunque:

fscanf(fin, "%s %d", nome, &numero);

Nel ciclo immediatamente successivo stai andando a rileggere continuamente il nome nella stessa variabile; probabilmente è voluto, ma nel dubbio te lo segnalo.

Inoltre dovresti sempre verificare che il valore restituito da fscanf sia uguale al numero di campi che stai leggendo, in caso contrario si è verificato un errore di lettura e dovresti interrompere la lettura segnalando il problema all'utente.

L'fscanf finale è inutile, fscanf si "mangia" da sola i whitespace, e \n rientra in ciò che è considerato whitespace (come spiegato nella sua documentazione).

Tutte le volte che leggi una stringa vedo che non limiti l'input alle dimensioni del buffer: è una potenziale vulnerabilità di sicurezza e può facilmente mandare in crash il tuo programma se il file contiene stringhe più lunghe della dimensione del buffer (se ti va bene, altrimenti il programma non va in crash ma inizia a fare cose senza senso perché hai sovrascritto parti di stack che non c'entrano nulla).

[Vik27]

Grazie, pensavo che la scanf avesso bisogno della \n per terminare.