Discussion :

[letoil]

'soir
jai un petit probleme dans la fonction de suppression appartir d'un fichier en utilisant des structurs
pour la fonction d'ajout ca va bien passer mais la supresssion ca va pas et voici tous le programme dans la pièce jointe .

Merci de votre aide !!!!!!

[Pragmateek]

Des remarques:
-main retourne int
-penser à faire de la compilation séparée, ne pas tout mettre dans le fichier principal.

Sinon quels sont les problèmes exactement?

[FidoDido®]

Bon j'ai pas tout lu, pcq c'est vraiment long...
Je n'ai regardé que ta fonction supprimer_lecteur, et donc forcément aussi modifier_lecteur

Code :

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

1
2
3
4
5
6
7
8
9
10
 
void supprimer_lecteur(unsigned id,char * nomfichier)
{
     lecteur *supl;
     supl.nom[]="";
     supl.prenom[]="";
     supl.id=0;
 
     modifier_lecteur(supl,id,nomfichier);
}

Bon alors déjà, tu dois faire un malloc qd tu déclares ton pointeur lecteur

Code :

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

lecteur * supl=malloc(sizeof(lecteur));

*

Ensuite, tu essaies d'accéder aux informations de supl comme s'il était de type lecteur, or c'est un pointeur, il faut donc procéder comme ceci :

Code :

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

(*supl).nom=strdup("");

ou encore

Code :

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

supl->nom=strdup("");

Ce qui amène à une deuxième remarque, tu ne peux pas définir la valeur de nom comme tu le fais. Il faut utiliser les fonctions dédiées aux strings.
http://nicolasj.developpez.com/articles/libc/string/

Enfin, tu appelles la fonction modifier_lecteur avec le pointeur supl qui n'est pas de type lecteur, mais de type lecteur * ce qui n'est pas le type de cette fonction.

Ceci n'est pas un relevé exhaustif des erreurs de ton code.
Reste plein de trucs à voir je crois

[crocodilex]

Au lieu d'utiliser "strdup" que je trouve assez lourd à utiliser, il est peut être plus simple de faire comme ceci :

Code :

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

1
2
3
4
5
6
7
8
9
void supprimer_lecteur(unsigned id, char *nomfichier)
{
    lecteur supl;
    supl.nom[0] = '\0';
    supl.prenom[0] = '\0';
    supl.id = 0;

    modifier_lecteur(&supl, id, nomfichier);
}

Et bien entendu modifier la fonction "modifier_lecteur" avec la signature suivante :

Code :

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

void modifier_lecteur(lecteur *l,unsigned id,char *nomfichier);

Et sinon, il y a pas mal de choses à revoir...