Discussion :

[Trademark]

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.

[gl]

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 :

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 ===|

qui indiquent que différentes fonctions attendent un tEmploye* mais tu leur passe un tEmploye**.

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).

[Trademark]

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

Bye bye.