[c] Creare una classifica con dati letti da file

[Giupoz]

Ehi ragazzi, ho questo codice che, premetto, è funzionante.
In pratica in un gioco devo leggere dal file record.txt i nomi dei giocatori che hanno vinto e il numero di mosse impiegato per vincere.
Nel file i dati appaiono così
CARLO 20
GIONN 13 ecc...
Nel prendere i dati dal file funziona, e anche nello stamparli.
Però se in contemporanea volessi stilare una classifica in ordine decrescente dei giocatori che hanno impiegato il minor numero di mosse per vincere, il codice si blocca. Come devo risolvere????


#include <stdio.h>
#include <stdlib.h>
int main ()
{
printf ("\n\t\t *** RECORD ***\n\n\n");
printf ("%21s%21s","PLAYER", "TURNI\n\n");
char name [7]; int i = 0; int k = 0;
int mosse;
int primo, secondo, terzo;
char first [7], second [7], third [7];
primo = 50; secondo = 50; terzo = 50;
FILE *cfPtr;
if ((cfPtr = fopen ("record.txt", "r")) == NULL)
{
puts ("File could not be opened");
}
while (!feof (cfPtr))
{
fscanf (cfPtr, "%6s%d", name, &mosse);
printf ("%d - %17s %19d\n", i++, name, mosse);
if (mosse < primo)
{ primo = mosse;
k = 0;
while (k < 7)
{
third [i] = second [i];
second [i] = first [i];
first [i] = name [i];
i++;
}
}
else if ( (mosse > primo) && (mosse < secondo) )
{ secondo = mosse;
k = 0;
while (k < 7)
{
third [i] = second [i];
second [i] = name [i];
i++;
}
}
else if ( (mosse > secondo) && (mosse < terzo) )
{ terzo = mosse;
k = 0;
while (k < 7)
{
third [i] = name [i];
i++;
}
}
}

system ("pause");
}

[gianmpu]

while (k < 7)
{
third [i] = second [i];
second [i] = first [i];
first [i] = name [i];
i++;
}
Questi sono tutti cicli infiniti perchè non aggiorni il valore di k che perciò resta sempre a 0 e non si esce mai dal ciclo while
Devi impostare le istruzioni come qualcosa del tipo
while (k++ .....)
Inoltre all'interno dei cicli while usi la variabile i senza inizializzarla quindi al secondo ciclo while in cui entri i ha un valore non corretto

[bancodeipugni]

poi a me il feof() non piace, preferisco sempre in questi casi il comtrollo sulla read()

è anche una questione di portabilità

[Giupoz]

Si scusate, sul file che ho io avevo già sostituito il i++ con k++, ma non va lo stesso....

[gianmpu]

Si scusate, sul file che ho io avevo già sostituito il i++ con k++, ma non va lo stesso....

Ok, ma se non pubblichi il codice modificato che hai scritto, per capire se ci sono altri errori oltre a quelli indicati, più che un programmatore ci vuole un indovino con la sua palla di cristallo. :D

[sottovento]

if ((cfPtr = fopen ("record.txt", "r")) == NULL)
{
puts ("File could not be opened");
}
while (!feof (cfPtr))

Controlli che il file venga correttamente aperto. Ottimo!
Nel caso non venga aperto, lo segnali all'utente, pero'.... continui l'esecuzione e otterrai un crash. Non ti conviene, per esempio,terminare l'applicazione?

ciao

[Giupoz]

avete ragione. Però parlando con i miei amici che hanno consegnato il progetto mi hanno consigliato, invece di ordinare i numeri nel file durante la stampa, di salvarli nel file già ordinati. Ciò significa che la funzione di salvataggio deve, oltre a scrivere, anche aprire il file e leggerlo, e andrà a scrivere il punteggio in maniera ordinata.
Quindi, se sul file c'è
Andrea 9
Giorgio 17
e il punteggio appena conseguito è 11, dovrà inserirlo tra Andrea e Giorgio. Come devo fare per farglielo capire? :confused:
Questo che sto usando adesso scrive soltanto, non ho idea di come farglielo l'ordinamento.. :muro:

void salva_record (int turno)
{
int mosse = 0;
mosse = turno;
char name [7];
FILE *cfPtr;
if ((cfPtr = fopen ("record.txt", "a")) == NULL)
{
puts ("\nFile could not be opened");
}
else {
puts("enter player name:\t");
puts ("Enter EOF to end input"); //CTRL+Z
printf ("%6s", "?");
scanf ("%6s", &name);
}
while (!feof (stdin))
{
fprintf (cfPtr, "%s %d\n", name, mosse);
printf ("%s", "?");
scanf ("%6s%d", name, &mosse);
}
fclose (cfPtr);
}

[Giupoz]

Ragazzi ho trovato questo codice di ordinamento su file, come posso applicarlo al mio????? MI aiutate gentilmente? mi mancano 12 ore prima di consegnare il progetto :doh:
#include <stdio.h>
#include <stdlib.h>

const int MAX = 1024;

void insert_sorted (long *sorted, int count, long value)

{

int i = 0;

sorted[count] = value;

if (count == 0) return;

for (i = count;i >= 0; i--)

{

if (value < sorted[i-1])

sorted[i] = sorted[i-1];

else break;

}

sorted[i] = value;

}

int main (int argc, char *argv[])

{

FILE *infile = NULL;

long sorted[1024];

long value;

int count = 0;

int i = 0;

if (argc < 2) {

fprintf (stderr, "Usage : %s \n", argv[0]);

return 1;

}

infile = fopen (argv[1], "r");

if (NULL == infile) {

perror ("fopen");

return -1;

}

/* while file not ends */

while (!feof (infile)) {

fscanf (infile, "%ld\n", &value); /* fetch value */

insert_sorted (sorted, count, value); /* sort */

++count; /* increase number of sorted values */

}

/* display values */

printf ("Sorted values : ");

for (i = 0; i < count; i++ ) {

printf ("%ld ", sorted[i]);

}

/* cleanup */

if (infile) {

fclose (infile);

infile = NULL;

}

return 0;

}