Petit soucis en C, 5min chrono la résolution

[fmr130]

Salut tous le monde,

je suis pas un grand spécialiste en C et donc j'ai un petit soucis qui je pense il vous faudra pas plus de 5 min pour me dire ou est mon erreur.


Habituellement pour ouvrir un fichier je fais ça :

char ancien[100], nouveau[100]; /* noms des fichiers                  */
   FILE *infile, *outfile;         /* poiteur sur les fichiers           */  
   char c;                         /* c pour déplacer le curseur dans le */
                                   /* fichier ouvert                     */

   /* Ouvrir le fichier à modifier                                       */
   /* ------------------------------------------------------------------ */ 
   do                           
   {
      printf("Entrez le nom du fichier a modifier : ");
      scanf("%s", ancien);  
      infile = fopen(ancien, "rb"); /* ouvrir en binaire pour lire CR   */
      if (!infile)                  /* Verifie que le fichier existe    */
      printf(" Impossible d'ouvrir le fichier: %s.\n", ancien);
   }
   while (!infile);

Mais aujourd'hui je souhaiterai modifier ce programme pour éviter à quelqu'un de taper l'adresse du fichier. En effet mon fichier sera toujours au même endroit et je souhaite juste récuperer des valeurs
dans ce fichier.
donc ça donne :

char fichier_init[1000];
   FILE *infile;

   fichier_init[] = {"C:/PROJET/xxxx/xxx.txt"};
   infile = fopen(fichier_init, "rb"); /* ouvrir en binaire pour lire CR   */

voila j'ai donc un soucis pour mettre mon adresse dans le char...

Merci d'avance.

[fmr130]

ça y est j'ai trouvé!!!

File *infile;
infile = fopen("C:\PROJET\xxxx\xx.txt", "rb");

et voili

[Philou67]

Si tu tiens à utiliser une variable pour le nom de ton fichier (ce qui est recommandé si tu l'utilises à plusieurs endroits dans ton programme), ton dispose de différentes moyens pour cela (par ordre de préférence) :

FILE *infile;
const char *fichier_init = "C:\\PROJET\\xxxx\\xx.txt";
infile = fopen(fichier_init, "rb");

FILE *infile;
const char fichier_init[] = "C:\\PROJET\\xxxx\\xx.txt";
infile = fopen(fichier_init, "rb");

FILE *infile;
char fichier_init[1000];
strcpy(fichier_init, "C:\\PROJET\\xxxx\\xx.txt");
infile = fopen(fichier_init, "rb");

Attention également à antislasher les antislash.

[SuperTux]

Petite remarque : mieux vaut prendre l'habitude d'utiliser strncpy, même lorsque les données ne viennent pas de l'exterieur. L'utilisation de strcpy est la première chose à regarder si l'on veut trouver des buffers overflows dans un source en C...

[SuperTux]

PS: D'ailleur pareil pour le char ancien[100] / scanf("%s", ancien); remplacer par scanf("%99s", ancien); si l'on tient vraiment à utiliser scanf.

[Philou67]

On est d'accord supertux. A utiliser ainsi :

strncpy(fichier_init, "C:\\PROJET\\xxxx\\xx.txt", sizeof(fichier_init));