Discussion :

[James_]

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?

[DaZumba]

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 :

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

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 :

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

FILE *OpenFile(char *c);

[HRS]

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

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
 
#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;
}

[Franck.H]

Perso je préfère cette forme:

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

[Trap D]

Tant qu'à faire

Code :

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

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

[HRS]

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 :

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
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;
}

[Emmanuel Delahaye]

Code :

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

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

Tordu !

Code :

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

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

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

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

 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 :

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

FILE *MaFonction(char *filename){}

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

Code :

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

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é?

[HRS]

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

[Médinoc]

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

[Franck.H]

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

Hummm ce n'est qu'à titre d'exemple c'est tout

[James_]

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!

[fearyourself]

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 :

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

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

[HRS]

la solution de fearyourself fonctionne

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
 
#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;
}

[Trap D]

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

Code :

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

return *f != NULL;

pourrait être intéressant.
Le fichier ensuite n'est pas fermé.

[fearyourself]

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"

[Trap D]

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