Paramètre FILE* dans une fonction...

Version imprimable

18/01/2006, 12h15
James_

Paramètre FILE* dans une fonction...

Bonjour,

J'utilise une fonction main où j'ai déclaré un type
FILE *fp = NULL;
dans ma fonction main j'appelle une fonction
// Protoype
OpenFile(FILE *f, char *c);
cette fonction OpenFile doit ouvrir un fichier en utilisant la fonction f=fopen(...);
Ceci se fait correctement.
Mon soucis vient qu'ensuite je veux écrire dans mon fichier depuis la fonction main et là je n'arrive pas à écrire car mon "fp" déclaré dans mon main n'est pas le même que celui ouvert dans ma fonction OpenFile.
Comment créer l'ouverture de fp de ma fonction main() en utilisant une autre fonction?
18/01/2006, 12h57
DaZumba

Citation:

Envoyé par James_

Comment créer l'ouverture de fp de ma fonction main() en utilisant une autre fonction?

Tu as deux choix:
- envoyer un pointeur contenant l'adresse du pointeur de type FILE *, afin que la fonction modifie l'argument envoye et non sa copie:

Code:

void OpenFile(FILE **f, char *c);

- faire comme fopen, et s'arranger pour que la fonction renvoit un FILE * (NULL en cas d'erreur)

Code:

FILE *OpenFile(char *c);

ce code fonctionne mais il y a surement une solution plus élégante

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
#include <stdio.h>
#include <stdlib.h>
typedef struct _MONFILE {
        FILE * f;
    } MONFILE;
void OpenFile (MONFILE *mf, char *c) {
     mf->f=fopen(c,"r");
     }
int main (void) {
    MONFILE mfp = {NULL};
    char enr[100]; 
    OpenFile (&mfp,"c:\\temp\\brex.txt");
    fgets (enr,100,mfp.f);
    printf(enr);
    getchar();
    return 0;
}

Perso je préfère cette forme:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
 
#include <stdio.h>
#include <stdlib.h>
 
 
FILE * OpenFile (char * s_filename)
{
   FILE * p_file = fopen (s_filename, "r");
   return p_file;
}
 
 
int main (void)
{
   FILE * p_file = NULL;
   char enr[100]; 
 
   p_file = OpenFile ("c:\\temp\\brex.txt");
 
   if (!p_file)
   {
      /* Erreur ... */
      exit (EXIT_FAILURE);
   }
 
   ...
 
   return EXIT_SUCCESS;
}

Mais franchement, je ne vois pas l'utilité de créer une telle fonction, surtout si elle ne se charge que de l'ouverture du fichier et rien d'autre !

Tant qu'à faire

Code:

1
2
3
4
FILE * OpenFile (char * s_filename)
{
   return fopen (s_filename, "r");
}

:lol:

c'est vrai que les solutions de Csoldier/trap D sont meilleures. donc un FILE* doit être un retour de fonction plutôt qu'un paramètre de fonction

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
#include <stdio.h>
#include <stdlib.h>
FILE * OpenFile (char * s_filename)
{
   FILE * p_file = fopen (s_filename, "r");
   return p_file;
}
int main (void)
{
   FILE * p_file = NULL;
   char enr[100];
   p_file = OpenFile ("c:\\temp\\brex.txt");
   if (!p_file)
   {
      /* Erreur ... */
      exit (EXIT_FAILURE);
   }
    fgets (enr,100,p_file);
    printf(enr);
    getchar();
   return EXIT_SUCCESS;
}

19/01/2006, 10h23
Emmanuel Delahaye

Citation:

Envoyé par HRS

Code:

p_file = OpenFile ("c:\\temp\\brex.txt");

Tordu !

Code:

p_file = OpenFile ("c:/temp/brex.txt");
19/01/2006, 11h02
James_
C'est pour un système embarqué du coup j'ai besoin d'optimiser au maximum la place mémoire.
J'utilise plusieurs fonctions qui ouvrent des fichiers, je me suis dit autant réaliser une seule fonction utilisant la fonction

Code:

fopen(filename, "")

créant les répertoires s'il le faut et traitant les résultats de fopen...
Du coup je ne peux pas écrire ma fonction comme telle

Code:

FILE *MaFonction(char *filename){}

car je retourne un int représentant un code retour.
mon proto serait de la forme:
Code:

1 2 int OpenFile(FILE *fp, char *filename);
suivant cette config, comment faire pour que dans ma fonction main() le paramètre FILE soit passé en argument et modifié?
19/01/2006, 11h25
HRS

si FILE* doit ABSOLUMENT être passé en paramètre, on en revient à ma
proposition initiale
19/01/2006, 11h28
Médinoc

Y'a des const qui se perdent, ici...
19/01/2006, 11h51
Franck.H

Citation:

Envoyé par Médinoc

Y'a des const qui se perdent, ici...

Hummm ce n'est qu'à titre d'exemple c'est tout :wink:
19/01/2006, 12h32
James_

Citation:

si FILE* doit ABSOLUMENT être passé en paramètre, on en revient à ma
proposition initiale

Sauf que dans ta proposition, le FILE* est mis dans une structure, ce qui en fait une déclaration de type globale.
Mais je m'en contenterai!

Merci à tous!

Citation:

Envoyé par James_

Citation:

si FILE* doit ABSOLUMENT être passé en paramètre, on en revient à ma
proposition initiale

Sauf que dans ta proposition, le FILE* est mis dans une structure, ce qui en fait une déclaration de type globale.
Mais je m'en contenterai!

Merci à tous!

En quoi, est-ce que la déclaration d'une structure fait que c'est une déclaration de type globale? Je en vois pas pourquoi mettre une structure en fait...

Par contre, rien ne t'empêche de faire une fonction comme ceci:
Code:

1 2 3 4 5 6 7 8 9 int MaFonction( FILE **file, char *filename ) { int res=0; //0 -> tout s'est bien passé //Traitement... *file = fopen(filename,"r"); //Traitement... return res; //Retour de res }
Jc

la solution de fearyourself fonctionne

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
#include <stdio.h>
#include <stdlib.h>
int OpenFile (FILE **f, char *c) {
     int res=0;
     *f =fopen(c,"r");
 
     return res;
}
int main (void) {
    FILE * fp = NULL;
    char enr[100]; 
    OpenFile (&fp,"c:/temp/brex.txt");
    fgets (enr,100,fp);
    printf(enr);
    getchar();
    return 0;
}

19/01/2006, 16h07
Trap D

Et on ne teste pas le retour de fopen ????
A quoi sert le res ?

Code:

return *f != NULL;

pourrait être intéressant.
Le fichier ensuite n'est pas fermé.
19/01/2006, 16h18
fearyourself

Citation:

Et on ne teste pas le retour de fopen ????
A quoi sert le res ?

C'est vrai, je l'avais supposé à la place des commentaires "//Traitement", j'aurais sûrement dû mettre "//Test et traitement"

:wink:
19/01/2006, 16h21
Trap D

Comme quoi il faut toujours donner un code complet sinon il peut être mal interprété :wink: