Discussion :

[Kallidus]

Bonjour à tous,

Je dois faire un exercice avec des structures et fichier. J'ai fais l'ajout, la recherche et l'affichage. Mais la modification je coince.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
void Modif(struct Donnees *pNew,FILE*fp)
{
    int nb;
    nb=Rech(pNew); // l'emplacement du nom recherché
    if(nb!=0)
        fp= fopen (chemin, "w+b"); //ouverture du fichier en écriture
    fseek(fp, nb*sizeof(pNew),SEEK_SET); 
    LireDonnee(pNew,fp); // Ajout des données de ma structure
    fclose(fp);
}

Mon programme si besoin 8.2.c

Merci d'avance

Kallidus

[Médinoc]

Tu mélanges trop de tâches dans ton code: Tu devrais séparer la saisie des données sur l'entrée standard et l'écriture du fichier.
De plus:

• fflush(stdin) est une erreur. fflush() n'est défini que pour les flux sortants. Ce n'est pas comme ça qu'on nettoie le buffer clavier.
• Pourquoi faire les saisies à coup de getchar() plutôt que fgets()?
• Ne passe pas en paramètre de fonction des variables dont la valeur n'est pas utilisée par la fonction (ta fonction Modif() n'a pas besoin de fp en paramètre, mais en variable locale)
• Tes boucles de saisie de chaîne et de retour si erreur, tu peux en faire une fonction: Passe-lui juste les messages d'erreur et la limite de taille en paramètres.

[Matt_Houston]

J'ajoute que tu ne vérifies la valeur de retour d'aucun de tes appels système. Comment espères-tu de cette manière découvrir l'origine des erreurs à l'exécution ?

[Kallidus]

Merci de vos réponses @Médinoc et @Matt_Houston.

Tu mélanges trop de tâches dans ton code: Tu devrais séparer la saisie des données sur l'entrée standard et l'écriture du fichier.
De plus:

Tu dis que je dois faire une fonction de mon menu dans le main?

Pourquoi faire les saisies à coup de getchar() plutôt que fgets()?

C'est le prof qui l'a demandé.

Ne passe pas en paramètre de fonction des variables dont la valeur n'est pas utilisée par la fonction (ta fonction Modif() n'a pas besoin de fp en paramètre, mais en variable locale)
Tes boucles de saisie de chaîne et de retour si erreur, tu peux en faire une fonction: Passe-lui juste les messages d'erreur et la limite de taille en paramètres.

Ok merci du conseil.

J'ajoute que tu ne vérifies la valeur de retour d'aucun de tes appels système. Comment espères-tu de cette manière découvrir l'origine des erreurs à l'exécution ?

Vérifier la valeur de retour??

Bien à vous

Kallidus

[Matt_Houston]

Vérifier la valeur de retour??

Okay...

Les fonctions qui sollicitent des ressources du systèmes peuvent échouer : que se passera-t-il si tu tentes d'accéder à un fichier qui n'existe pas ? Afin de déterminer la manière dont a été effectuée l'opération demandée ou si une erreur s'est produite, ces fonctions disposent d'un mécanisme pour informer l'appelant du résultat. Le plus souvent ce mécanisme prend la forme d'un code transmis par la valeur de retour de la fonction. Je t'invite à consulter les pages de manuel des appels que tu utilises.

[Kallidus]

Bonjour,

Sur le site, y aurait-il un tutoriel ou des explications sur les fichiers en C?

Car là, je patauge.

Kallidus

[LittleWhite]

Bien sûr, il y a des tutoriels, et la FAQ aussi.

Et bien sûr, un tutoriel sur les fichiers