Discussion :

[Terordar]

Bonsoir à tous et à toutes !
Cela fait maintenant plusieurs heures que je cherche une solution à mon problème.. Et je commence à en avoir un tout petit peu marre Je viens donc vous demander un peu de votre temps pour m'aider à résoudre mon problème ! Ce serait vraiment super sympa

Bon, alors, j'essaye d'implémenter une fonction qui me permettra d'indexer mon fichier dans la RAM. La voici :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
void Indexation(Index *pIx){
    FILE *fp;
    Joueurs *Jr;
    short i = 0;
 
 
    fp = fopen(FILE_NAME_JOUEURS, "rb");
 
 
    Jr = (Joueurs *) malloc (sizeof(Joueurs));
 
 
    if(fp == (FILE *) NULL)
    {
        printf("Erreur d'ouverture du fichier !\n");
        printf ("Code de l'erreur : %d\n", errno);
        printf ("La signification de l'erreur est donc : %s\n", strerror(errno));
    }
    else
    {
        fread(&Jr, sizeof(Joueurs), 1, fp);
 
 
        while(!feof(fp) && i < MAXIND)
        {
            pIx->IdJoueur = Jr.IdJoueur;
            strcpy(pIx->Nom, Jr.Nom);
            pIx->indice = ftell(fp) - sizeof(Joueurs);
 
 
            fread(&Jr, sizeof(Joueurs), 1, fp);
            pIx++;
            i++;
        }
    }
 
 
    fclose(fp);
 
 
    free(Jr);
}

Mon problème se déroule à partir de la ligne 24, avec le while(!feof..). Je ne comprends pas pourquoi, peu importe le nombre d'enregistrements que j'ai dans mon fichier, après le fread il considère qu'il est à la fin du fichier et ne passe donc pas par le while ! J'ai bien vérifié ma fonction d'ajout, si je ne me trompais pas de fichier ni rien. Tout se passe correctement.. Mais ce foutu fread... Je ne comprends vraiment pas ce qui se passe :'(

Quelqu'un pour éclairer ma lanterne ?

Merci ! Bonne soirée !

[Sve@r]

Bonjour

Plusieurs erreurs dans ce code

1. tu alloues Jr avant d'avoir détecté si le fichier a bien été ouvert (donc s'il ne l'a pas été, tu alloues pour rien)
2. tu alloues Jr de 1 => quand tu as besoin d'un int, est-ce que tu fais int *i puis i=malloc(sizeof int) ???
3. Tu utilises Jr comme si c'était une variable (Jr.Nom) alors que c'est un pointeur => Jr->Nom (t'as pas eu de warning ???). Ceci dit, si tu supprimes le pointeur ça sera bon.
4. tu passes l'adresse de Jr à fread() alors que Jr est déjà une adresse !!!
5. tu fais un fclose() même si le fichier n'a pas été ouvert

Donc l'ensemble de ces erreurs fait que rien ne peut fonctionner comme tu le veux. Et ta gestion de lecture est bricolée. Une première lecture puis un while !feof() avec une seconde lecture dans la boucle. Autant supprimer les deux lectures mettre directement while (fread(...) > 0 && i < MAXIND). Pareil avec ton indice. Tu as fp placé au bon endroit. Tu fais un fread() donc fp est décalé de sizeof joueurs puis tu calcules ton indice sur la position précédente de fp. Tu ne pouvais donc pas mémoriser directement la position de fp avant qu'il ne soit déplacé !!!???

[foetus]

@ Sve@r
6. La condition else peut être supprimée (et donc on gagne une indentation et en lisibilité) si on met un return dans le if
On ne craint rien, si on déplace ou supprime le malloc

[Sve@r]

6. La condition else peut être supprimée (et donc on gagne une indentation et en lisibilité) si on met un return dans le if

Oui, c'est aussi ce que je fais mais bon, tel qu'il l'a écrit c'est correct

De plus, il faut faire attention quand on prend cette habitude surtout si on commence à gérer plusieurs ressources car à chaque ressource non récupérée, il faut penser à libérer à chaque fois toutes les ressources précédentes avant de quitter
Exemple

Code c :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
void fonction()
{
    if ((ressource1=fopen(...)) == NULL)
        return;
 
    if ((ressource2=malloc(...)) == NULL)
    {
        fclose(ressource1);
        return;
    }
 
    if ((ressource3=shmat(...)) == NULL)
    {
        free(ressource2);
        fclose(ressource1);
        return;
    }
 
    // etc etc jusqu'au final
    shmdt(ressource3);
    free(ressource2);
    fclose(ressource1)
}

Ou alors prendre le risque d'introduire du goto

Code c :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
void fonction()
{
    ressource1=NULL;
    ressource2=NULL;
    ressource3=NULL;
    if ((ressource1=fopen(...)) == NULL)
        goto fin;
 
    if ((ressource2=malloc(...)) == NULL)
        goto fin;
 
    if ((ressource3=shmat(...)) == NULL)
        goto fin;
 
    // etc etc jusqu'au final
    fin:
    if (ressource3) shmdt(ressource3);
    if (ressource2) free(ressource2);
    if (ressource1) fclose(ressource1)
}

Alors qu'avec son écriture, il n'y a ni répétition de code ni goto. C'est pour ça que certains puristes hurlent devant cette façon de faire...

On ne craint rien, si on déplace ou supprime le malloc

Oui évidemment !!! Tout ce que j'ai cité ne sont pas que des erreurs fatales je le sais bien !!! Le malloc et son appel parfois inutile ne gênent pas mais bon, autant lui donner aussi les bonnes habitudes quoi...

[Terordar]

Merci beaucoup pour vos conseils ! Je viens donc de faire quelques modifications et tout marche maintenant.

@Sve@r, non, je n'ai pas eu de warning lorsque j'ai utilisé Jr.Nom !

Bon, je vais continuer à me casser la tête avec mon programme Je vous souhaite une agréable soirée et vous remercie à nouveau pour votre aide !