Discussion :

[walrus_55]

Bonjour,

supposons un programme "file_name.cpp":

Code :

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

1
2
3
4
5
6
7
int main()
{
  FILE * p_file;
  p_file = fopen( "output.txt" , "w" );
  fprintf( p_file, "Hello World !\n" );
  fclose (p_file );
}

Maintenant, je désire ajouter:
int a = 2;
et que le fichier de sortie s'appelle output_2.txt
cad en fonction de la valeur de a

un peu comme le printf: ("output_%i.txt", a)

mais ça ne marche pas avec le fopen()...

l'un d'entre vous aurait-il un conseil ?

Merci d'avance

[Franck.H]

Salut et bienvenue sur les forums de developper.com


En fait, c'est facile, il faut d'abord construire une chaîne contenant le nom du fichier avec le numéro, ce que tu peux faire avec la fonction sprintf ... Voilà !

Pour info, test toujours le retour de fopen pour le cas où la création ou l'ouverture du fichier échouerais et aussi, les fichiers source en C on l'extension .c et non pas .cpp qui est pour le C++ !

[walrus_55]

merci pour la réponse !
mais je n'rrive pas à utiliser sprintf (malgré la page du manuel)

Voilà ce que j'écris:

Code :

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

1
2
3
4
5
6
7
  int a = 5;
  FILE * p_file;
  char * file_name;
  sprintf( file_name, "output_%i.txt", a );
  p_file = fopen( file_name , "w" );
  fprintf( p_file, "Hello World !\n" );
  fclose ( p_file );

ça compile bien avec: gcc -o file_name file_name.c
mais après ./file_name
ça me renvoie Segmentation Fault

et sinon, o`u pourrai-je trouver des infos pour le test de fopen ?

[Emmanuel Delahaye]

Code :

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

1
2
3
  int a = 5;
  char * file_name;
  sprintf( file_name, "output_%i.txt", a );

ça compile bien avec: gcc -o file_name file_name.c

Euh, non...

http://emmanuel-delahaye.developpez....tm#cfg_compilo

mais après ./file_name
ça me renvoie Segmentation Fault

Ce n'est pas étonnant. Tu définis un pointeur non initialisé et tu passes sa valeur à une fonction. Le comportement est donc indéterminé.

Il faut passer une valeur valide. Le plus simple, ici, est de passer l'adresse du premier élément d'un tableau de char.

[walrus_55]

alors j'ai un peu arrangé le code selon vos conseils et ça donne ça:

Code :

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

1
2
3
4
5
6
7
8
  int a = 10;
  char s[] = "output_.txt";
  char *file_name = &s;
  sprintf( file_name, "output_%i.txt", a );
  FILE * p_file;
  p_file = fopen( file_name , "w" );
  fprintf( p_file, "Hello World !\n" );
  fclose ( p_file );

mais quand je compile avec: "gcc -o file_name file_name.c"
ça me donne: "warning: initialization from incompatible pointer type"
pour la ligne "char *file_name = &s;"
mais ça compile quand même
et quand j'exécute avec "./file_name"
ça crée bien le fichier "output_10.txt"

donc je ne comprends pas le warning à propos du "incompatible pointer type"...

[walrus_55]

ça y est, j'ai compris
il faut préciser

Code :

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

char *file_name = &s[0];

pour initialiser le pointeur

donc, c'est bon, c'est résolu.
Merci

[Bayard]

Code :

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

1
2
char s[] = "output_.txt";
char *file_name = &s;

1) -> char *file_name = s me parait plus logique
2) à quoi sert de faire une chaine et de déclarer un pointeur dessus ?

Je propose:

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
 
#include <stdio.h>
 
#define C_MAX 128
 
void main(void)
{
  int a = 10;
  char file_name[C_MAX];
 
  FILE * p_file;
  sprintf( file_name, "output_%i.txt", a );
  p_file = fopen( file_name , "w" );
  if (p_file != NULL)
  {
	fprintf( p_file, "Hello World !\n" );
	fclose ( p_file );
  }
}

La valeur de fopen est testé. Il y a un limitation dans le nombre de digits du à la taille de file_name, mais je suis optimiste

Ceci n'est pas testé...