Problème au niveau de la fonction remove

Bien le bonjour ,
Je suis étudiant dans une école d'ingénieur en informatique en 1ère année . Je suis en train de réaliser un petit logiciel sur console qui permettrait à des professeurs (avec leurs comptes) d'école d'ajouter ou de supprimer les absences de leurs élèves.

En plus des professeurs , il existe aussi des comptes administrateurs pouvant ajouter/supprimer des classes ou des utilisateurs , et aussi des comptes élèves (ceux-ci pourront lister leurs absences selon la date/matière)


Je vous expose donc mon problème . Mon logiciel utilise le système de fichiers. Les absences sont notées dans le fichier de l'élève par exemple .. Même chose pour les matières ou les comptes d'utilisateurs qui ont leurs propres fichiers à eux.

J'ai un problème au niveau de la fonction remove() , que j'ai utilisée à maintes fois dans le code, notamment dans les fonctions de suppression ( supprimerUtilisateur / supprimerClasse / supprimerMatière etc. ) Pour supprimer une matière (dans le fichier matieres.txt), je procède comme suit :
Je crée un fichier temp.txt qui contiendra tout le contenu de matieres.txt , sauf la matière que je veux supprimer.
Je supprime ensuite le fichier matieres.txt grâce à la fonction remove(), puis renomme le fichier temp.txt en matieres.txt grâce à rename(). Ni vu ni connu ;) .


Cependant , la fonction remove ne marche absolument pas. Je me retrouve donc avec un fichier matieres.txt intact , et un fichier temp.txt qui contient toutes les matières sauf celle qui est supprimée . :?

J'ai essayé à maintes fois de trouver l'origine du problème . Peut être est-ce à cause de la non-fermeture d'un fichier ... En tout cas , ça fait 2 semaines que j'essaye de régler ce problème , mais rien n'y fait ... C'est pourquoi je vous demande votre aide ;) . Je joins à ce message le fichier rar du projet , qui contient le main.c ainsi que les fichiers d'élèves/matières/comptes

Merci d'avance ;)

PS : pour le compte admin , login : admin / pass : admin et le compte prof , login : p / pass : p

Bonsoir et bienvenue,

Tu ne testes pas la valeur de retour de remove(), qui peut pourtant échouer.

Cette fonction renvoie 0 si tout s'est bien passé et -1 dans le cas contraire. Tu peux utiliser perror("") pour obtenir un message à l'écran indiquant en toutes lettres ce qu'il s'est passé.

Par ailleurs, créer un second fichier pour supprimer des enregistrements est une pratique courante, mais qui n'est pas sans poser problème : http://www.developpez.net/forums/d10...r/#post6012305

Bonsoir et merci pour l'acceuil :)
J'ai déjà auparavant affiché la valeur de retour de remove() , ça m'a affiché -1 .
Je viens d'essayer tout à l'heure avec perror , ça m'a affiché " Permission denied" .
Comme je l'ai écrit dans mon précédent post , je pense que l'erreur vient d'un fichier qui n'a pas été fermé avec fclose auparavant . J'ai cependant cherché partout et je n'ai pas trouvé d'où venait le problème :(

Comme son nom l'indique, « permission denied », ça veut dire « permission refusée ». Un fichier ouvert ne t'empêche pas de le supprimer, en principe (il disparaît effectivement du disque qu'au moment où le dernier descripteur est refermé).

La première chose qu'il faut regarder, ce sont les droits d'accès du fichier (avec « ls -l »). Il y a de fortes chances pour qu'il soit créé par défaut en read-only. L'absence du bit « w » t'empêcherait de le supprimer, même si la commande Shell la rajoute elle-même après confirmation éventuelle.

Ensuite, il faut vérifier si le nom de fichier que tu passes à remove() est bien le bon, notamment au niveau des chemins d'accès.

Je précise tout d'abord que je suis sur Windows, donc j'ai fait clic-droit->propriétés sur le fichier matieres.txt . Il n'était pas en mode lecture seule ;) .

Tout à l'heure j'ai essayé d'appeler la fonction SupprimerMatiere au tout début du programme , c'est à dire au début de la fonction main, de cette façon :
SupprimerMatiere ("Maths");

Résultat : La modification a bien eu lieu , c'est à dire que le fichier matieres.txt a été supprimé et que le fichier temporaire a bien été renommé en matieres.txt ;) . Le problème vient donc d'autre part . (je cherche encore et toujours :mrgreen: )

Bonjour,
J'ai regardé ton code, (pas testé ^^)

Juste une chose avant de commencer,
D'après la composition de l'archive,
Tu sembles utiliser un IDE style Visual ou autre ...
Donc pour plus de clarté,
Pense à la programmation modulaire (dans plusieurs fichier)

Évite les exit(0); en plein milieu du code,
Favorise la remonté des erreurs.

Et aussi, les indentations sont parfois spéciales =/
Essaye de régler ton IDE pour qu'il le fasse tout seul ^^

Bon sinon,
Mis à part mon petit dégout sur l'ouverture/fermeture sans cesse de fichiers,
( essaye si possible de garder le moins longtemps un fichier ouvert
. par exemple, dans ajouterUti
. Tu ouvres f au début pour ne t'en servir qu'une fois à la fin ... )

J'ai remarqué que la fonction appel ouvre un fichier

Code:

---

f = fopen(chemin,"r");

---

sans jamais le refermer.

dans la fonction supprimerAbsence

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

void supprimerAbsence (char matiere[], char fichierEleve[], date d1) { FILE *f = NULL; FILE *g = NULL; /* [...] */ f=fopen(fichierEleve,"r"); g=fopen("temp.txt","w"); if (f==NULL || g == NULL) { printf("Erreur"); } else { /* [...] */ remove(fichierEleve); rename("temp.txt",fichierEleve); } fclose(f); fclose(g); }

---

Tu fermes les fichiers même s'ils ne sont pas ouvert Oo
Et pourquoi tu ne les ferme pas avant de les renommer & co ?

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

void supprimerAbsence (char matiere[], char fichierEleve[], date d1) { FILE *f = NULL; FILE *g = NULL; /* [...] */ f=fopen(fichierEleve,"r"); g=fopen("temp.txt","w"); if (f==NULL || g == NULL) { printf("Erreur"); } else { /* [...] */ fclose(f); fclose(g); remove(fichierEleve); rename("temp.txt",fichierEleve); } }

---

(surtout que tu le fais dans une autre fonction ^^)

et sinon, bon c'est sous linux mais vu que c'est C89, Microsoft doit faire la même ^^
man rename
C'est dit que rename écrasera le fichier,
Donc peut être que la fonction remove n'a pas le temps de finir
Avant ton appel à rename et que ça bloque ...
Essaye avec juste rename pour voir ^^

Tu ne fermes pas k dans la fonction certifierAbsence,
Ta fonction tri ne ferme ni f ni g,
listerAbsences ne ferme pas f ...
listerAbsencesMatiere ne ferme pas g
listerAbsencesDate ne ferme pas f

Dans la fonction creationFichiers
Tu perds des pointeurs,
Quand tu fais :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

f=fopen("comptes.txt","r"); if (f==NULL) { /* manque un fclose */ f=fopen("comptes.txt","w"); fclose(f); } /* manque un fclose */ f=fopen("matieres.txt","r");   /* [...] idem ensuite */

---

(C'est pour ça que j'aime pas utiliser des fichiers trop souvent =/)

Bonjour ,
Tout d'abord , merci de ta patience à l'égard de mon problème , tu as dû en baver en voyant toutes ces erreurs :mrgreen:

Parmi les erreurs que tu as pu lister, la seule que j'ai trouvé hier était celle de creationFichiers . J'ai simplifié la fonction, tout en gardant son utilité :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

void creationFichiers() { FILE *f;   f=fopen("comptes.txt","a"); fclose(f); f=fopen("matieres.txt","a"); fclose(f); f=fopen("classes.txt","a"); fclose(f); }

---

En voyant les autres erreurs , j'ai remarqué que mon programme avait beaucoup plus d'erreurs que je ne le croyais :mrgreen: . Je les ai corrigés , et maintenant il n'y'a plus aucun problème . Le fichier matières.txt est supprimé et le fichier temporaire est renommé correctement :).

En tout cas , je viens de comprendre ton dégoût pour l'ouverture/fermeture des fichiers, je tacherai d'y faire plus attention ;) . Je tiendrai aussi compte de tes remarques pour m'améliorer (parce que là je viens de me foutre la honte :mrgreen: )

Problème résolu donc , merci à vous deux ;)

Citation:

---

Envoyé par DesLife

Tout d'abord , merci de ta patience à l'égard de mon problème , tu as dû en baver en voyant toutes ces erreurs :mrgreen:

---

J'ai vu pire ^^ (et j'ai aussi fait pire ^^')

J'ai un prof qui me remontait les bretelles à chaque projet que je lui rendait ^^
Il était très pointilleux ...
J'ai même eut le droit sur une remarque sur la fonction Rand et son utilisation avec référence à l'appuie
Parce que je demandais mal des nombres de 0 a 10 xD
Donc maintenant j'essaye de faire tout ce qu'il m'a dit.
Surtout de tester tous les retours de la lib C (fopen et compagnie) Oo

Donc n'ai pas honte de tes erreurs,
Il en faut pour s'améliorer ^^

La citation préféré d'un de mes prof est : "Dans le meilleur des cas, ça plante" ^^