Version imprimable
Bonjour à tous,
je me décide à poster pour ce nouveau problème que je n'arrive pas à régler. Il s'agit en fait de créer une liste d'employé et d'écrire celle-ci dans un fichier. On retrouve les fonctions classiques : Ajouter, Supprimer, Modifier, Sauvegarder, Afficher. J'ai néanmoins un problème d'exécution lors de l'encodage du prénom.
Les codes sont ci-joints.
Merci à vous.
J'ai juste jeté un coup d'oeil rapide, mais il y a surtout un micmac dans les pointeurs.
Pour commencer, il y a plusieurs warnings :
qui indiquent que différentes fonctions attendent un tEmploye* mais tu leur passe un tEmploye**.Citation:
C:\download\employeMain.c||In function `main':|
C:\download\employeMain.c|50|warning: passing arg 1 of `ajouterEmploye' from incompatible pointer type|
C:\download\employeMain.c|65|warning: passing arg 1 of `synchroFileStruct' from incompatible pointer type|
C:\download\employeMain.c|77|warning: passing arg 1 of `ajouterEmploye' from incompatible pointer type|
C:\download\employeMain.c|81|warning: passing arg 1 of `supprimerEmploye' from incompatible pointer type|
C:\download\employeMain.c|85|warning: passing arg 1 of `sauvegarderListe' from incompatible pointer type|
C:\download\employeMain.c|90|warning: passing arg 1 of `modifierEmploye' from incompatible pointer type|
C:\download\employeMain.c|15|warning: unused parameter 'argc'|
C:\download\employeMain.c|15|warning: unused parameter 'argv'|
||=== Build finished: 0 errors, 8 warnings ===|
Tu as également un problème dans la fonction allocStruct(). Dans celle-ci, tu modifies le paramètre s_Employe qui est passé par valeur, le changement n'est pas répercuté à l'appelant donc tu vas perdre l'adresse de la mémoire allouée et surtout utiliser un pointeur NULL.
A côté de ça :
- Il manque beaucoup de vérification des valeurs de retours (fopen, fscanf, etc.).
- Il y a des risques de débordement de buffer (fscanf(... %s ...)).
- Pas mal de code inutile (caractere[0] = 0; avant le fgets, libération du pointeur dont l'allocation vient d'échouer, etc.).
- De construction inutilement complexe, peu maintenanble (fgets(caractere,10,fNB);) ou peu réutilisable (exit dans le corps d'une fonction).
Merci grâce à toi j'ai pu trouver mon erreur :)
En tout cas, bravo pour les explications, nettes, précises et surtout bien structurées :ccool:
Bye bye.