pointeur de fichier dans appel de fonction

Bonjour, j'aimerais savoir une chose,
dans mon main(), j'ouvre un fichier, et je passe le pointeur à une fonction pour qu'elle écrive dans le fichier. Si je fais ca c'est parce que la fonction qui ouvre le fichier et très souvent appelé, donc j'économise du temps machine.
le problème c'est que quand j'essai d'écrire dans le fichier il me dit "access violation"

voilà le code :

ca c'est dans le main. et c'est la fonction show packet à laquelle je passe le pointeur.

Code:

---

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

FILE *fp=fopen("./fichier/resultatfonction.txt","w");   desc = pcap_open_offline("./fichier/fichiertest.pcap", errbuf); if (desc == NULL) printf("pcap_open_offline: %s\n", errbuf), exit(2);       while ((packet = pcap_next(desc, &hdr))) { showpacket(hdr, packet,fp); }

---

la fonction showpacket :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

void showpacket(struct pcap_pkthdr hdr, const u_char *packet, FILE *fp) { char chaineretour[10000]; struct tm *mytime; int i, j, k;   mytime = localtime(&hdr.ts.tv_sec);   sprintf(chaineretour, "%.2d:%.2d:%.2d.%.6i", mytime->tm_hour, mytime->tm_min, mytime->tm_sec, (int)hdr.ts.tv_usec);   fwrite(chaineretour,strlen(chaineretour),sizeof(chaineretour),fp);   printf(", Len: %i (0x%x)\n",hdr.caplen,hdr.caplen);

---

Je pense que l'erreur vient des pointeur mais j'ai un peu de mal à débrouiller tout ca.
merci pour le coup de main

Tu ne vérifies pas si l'ouverture du fichier a échoué (i.e. fp reçoit de fopen() une valeur différente de NULL)?

Thierry

Non le fichier s('ouvre bien ya pas de soucis la dessus, j'ai testé la ligne avant de faire un jolie copier,coller. Le problème doit venir du pointuer que je passe en argument. je sais plus si dans l'appel de la fonction je dois mettre &fp ou juste fp, et dans la fonction si je dois mettre FILE *fp, ou FILE **fp.

Citation:

---

Envoyé par danathane

Non le fichier s('ouvre bien ya pas de soucis la dessus, j'ai testé la ligne avant de faire un jolie copier,coller. Le problème doit venir du pointuer que je passe en argument. je sais plus si dans l'appel de la fonction je dois mettre &fp ou juste fp, et dans la fonction si je dois mettre FILE *fp, ou FILE **fp.

---

Si fp est de type pointeur sur FILE et que ta fonction attend un argument de type FILE *, il n'y a pas de raison de passer autre chose que fp en paramètre. A part si la fonction a besoin de modifier la valeur du pointeur lui même, je ne vois aucune raison d'utiliser un paramètre de type FILE ** dans l'interface de ta fonction.

Thierry

Bon après avoir tester l'ensemble des possibilté, voilà comment il faut faire pour passer le pointeur d'un fichier à une fonction pour qu'elle puisse écrire dedans :

dans le main :

Code:

---

showpacket(machin,&fp);

---

et l'en-tete de la fonction :

Code:

---

1 2 3 4 5 6 7

void showpacket(int machin, FILE ** fp){ //pour l'utiliser :   fputs("truc",*fp);   }

---

grâce à cela je peux modifier mon fichier sans avoir besoin de l'ouvrir à chaque appel de la fonction.

Merci pour les conseils

Citation:

---

Envoyé par danathane

Bon après avoir tester l'ensemble des possibilté, voilà comment il faut faire pour passer le pointeur d'un fichier à une fonction pour qu'elle puisse écrire dedans :

dans le main :

Code:

---

showpacket(machin,&fp);

---

et l'en-tete de la fonction :

Code:

---

1 2 3 4 5 6 7

void showpacket(int machin, FILE ** fp){ //pour l'utiliser :   fputs("truc",*fp);   }

---

grâce à cela je peux modifier mon fichier sans avoir besoin de l'ouvrir à chaque appel de la fonction.

Merci pour les conseils

---

Si tu veux juste que ta fonction écrive dans le fichier sans besoin de modifier la valeur du pointeur lui-même, je ne vois pas l'utilité de passer par un argument de type FILE **.

Thierry

ben le truc c'est que la fonction sera rappler, donc pour ne pas perdre la position de la dernière écriture dans le fichier j'ai pas trop le choix. sinon ca risque d'écrire n'importe ou

La fonction fwrite modifie la chose pointée par fp et non le pointeur lui même.

fp ne représente pas un pointeur vers ta position dans le fichier, juste un pointeur vers une chose représentant ton fichier.

Citation:

---

Envoyé par danathane

ben le truc c'est que la fonction sera rappler, donc pour ne pas perdre la position de la dernière écriture dans le fichier j'ai pas trop le choix. sinon ca risque d'écrire n'importe ou

---

Cela n'a strictement rien à voir. La position dans le fichier est enregistrée dans la structure de donnée de type FILE dont l'adresse est retournée par fopen(). Lorsque tu passes cette adresse à une fonction, cette dernière a ensuite tout loisir de modifier l'objet situé à l'adresse qui lui a été donné.

Thierry

Citation:

---

Envoyé par danathane

Bon après avoir tester l'ensemble des possibilté, voilà comment il faut faire pour passer le pointeur d'un fichier à une fonction pour qu'elle puisse écrire dedans :

dans le main :

Code:

---

showpacket(machin,&fp);

---

et l'en-tete de la fonction :

Code:

---

1 2 3 4 5 6 7

void showpacket(int machin, FILE ** fp){ //pour l'utiliser :   fputs("truc",*fp);   }

---

grâce à cela je peux modifier mon fichier sans avoir besoin de l'ouvrir à chaque appel de la fonction.

---

Si la seule façon pour une fonction de modifier un fichier est de recevoir un FILE **, alors pourquoi fputs (qui a quand-même une action de modification incontournable) reçoit seulement un FILE * ???
Ton problème doit se situer ailleurs...

Il n'y a plus de problème. Ca marche très bien avec ce que j'ai décrit plus haut.
Mais merci pour les précisions

Citation:

---

Envoyé par danathane

Il n'y a plus de problème. Ca marche très bien avec ce que j'ai décrit plus haut.
Mais merci pour les précisions

---

Si il n'y a plus de problème, n'oublie pas d'appuyer sur :resolu:

Thierry

Citation:

---

Envoyé par danathane

Bon après avoir tester l'ensemble des possibilté, voilà comment il faut faire pour passer le pointeur d'un fichier à une fonction pour qu'elle puisse écrire dedans :

dans le main :

Code:

---

showpacket(machin,&fp);

---

et l'en-tete de la fonction :

Code:

---

1 2 3 4 5 6 7

void showpacket(int machin, FILE ** fp){ //pour l'utiliser :   fputs("truc",*fp);   }

---

---

C'est une complication inutile. Ceci

Code:

---

showpacket(machin, fp);

---

Code:

---

1 2 3 4 5 6

void showpacket(int machin, FILE * fp){ //pour l'utiliser : fputs("truc", fp);   }

---

doit fonctionner. Ou alors tu ne montres pas tout.

Le fclose(), tu le fais quand ?