Discussion :

[DesLife]

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

[Obsidian]

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

[DesLife]

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

[Obsidian]

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.

[DesLife]

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 )

[brachior]

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 :

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

f = fopen(chemin,"r");

sans jamais le refermer.

dans la fonction supprimerAbsence

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

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