Ouverture d'un fichier avec son nom

Version imprimable

Voir 40 message(s) de cette discussion en une page

bonjour,
je charge un fichier dans lequel je sotck les valeurs dans un tableau.
ce fichier s'apelle : "level1.txt"
mais lorsque que je tape son nom le fichier ne s'ouvre le probleme doit resier dans la manipulation de la chaine de caractere

j'ai un debug assertion invalid
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 void main (void) { FILE *fp; char *myname; myname = (char*)malloc(256 * sizeof (char)); scanf("%s",&myname); fp = fopen("myname","r"); for ( int i =0 ; i < 5 ; i++ ) for ( int j=0 ; j< 6 ; j++) fscanf(fp,"%d",&Tab[i][j]); fclose(fp); free(myname); }
Cordialement Bruno

Plusieurs choses:

1) sizeof(char) == 1, donc la multiplication est inutile dans le malloc().

2) le scanf est absolument à proscrire. Utilise plutôt fgets() et vire le '\n' terminal. Exemple:
Code:

1 2 3 4 5 6 7 8 int getstring(char *s, size_t n) { if (fgets(s, n, stdin) != NULL) { char *p = strchr(s, '\n'); if (p != NULL) *p = '\0'; /* on vire le '\n' terminal */ return 1; } return 0; /* erreur de lecture: fin de fichier ou autre */ }
3) Tu ne vérifie pas que le fichier fp est bien ouvert, et en plus tu lui donne en paramètre un nom de fichier erroné. Correction:
Code:

1 2 3 4 if ((fp = fopen(myname, "r")) == NULL) { perror(myname); /* affichage de l'erreur */ /* On saute ensuite la lecture du fichier ainsi que sa fermeture */ }
4) Ceci est un peut du pinaillage, mais c'est se compliquer la vie pour rien de faire une allocation dynamique d'un tableau dont la taille est fixe et de 256 octets seulement.

Salut
Correction du programme

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
30
31
32
33
34
35
36
37
38
39
40
41
42
43
// un des prototypes de main est int main(void)
int main (void)
{
 
        FILE *fp;       
        char *myname;
 
        // myname = (char*)malloc(256 * sizeof (char));
        // par définition sizeof(char) = 1
        // le cast en C est inutile et il peut cacher l'oubli de l'inclusion du fichier .h nécessaire
        myname = malloc(256);
 
       // il faut tester le retour de malloc
      if (myname == NULL)
      {
         fprintf(stderr, "Pb malloc\n");
         return 0;
      }
 
       // pour scanf il donner l'adresse ou ranger, donc myname 
      // et non l'adresse du pointeur
        scanf("%s",myname);
 
     // note scanf est déconseillé car très difficile d'utilisation
     // il vaut mieux utiliser fgets(myname, 256, stdin);
 
        fp = fopen("myname","r");
       // il faut tester le retour de fopen
      if (fp == NULL)
      {
          fprintf(stderr, "Pb ouverture %s\n", myname);
         free(myname);
         return 0;
      }
 
             for ( int i =0 ; i < 5 ; i++ )
                 for ( int j=0 ; j< 6 ; j++)
                     fscanf(fp,"%d",&Tab[i][j]);
 
        fclose(fp);
        free(myname);
  return 0;
}

13/10/2005, 18h11
gege2061

Bonjour,
juste une petite correction à ton code Trap D, lorsqu'une erreur se produit, il ne faut pas faire un

Code:

return 0

mais un

Code:

return EXIT_FAILURE

ou tout autre chiffre différent de 0 pour que l'OS sache que ça n'a pas bien fonctionné. C'est utilile sous Linux lorsque l'on lance plusieurs commandes qui dépendent les unes des autres :

Code:

$ ./configure && make && make install

par exemple.
13/10/2005, 18h16
Trap D

Très juste gege2061, mais comme je n'utilise jamais *nix et que je ne travaille pas sous DOS je n'y avais pas pensé. :oops:
13/10/2005, 18h41
bonjour69

yhg

rebonjour,

le code se compile parfaitement par contre impossible d'ouvrir mon fichier level1.txt qui pourtant est bien existant
je tape bien : level1.txt

mais il me dis pb d'ouverture level1.txt

Voila merci
13/10/2005, 18h44
Trap D

Se trouve-t-il dans le répertoire d'exécution ?
Donnes-tu le bon nom, il est difficile pour nous de savoir.
13/10/2005, 19h29
Emmanuel Delahaye

Citation:

Envoyé par Trap D

Se trouve-t-il dans le répertoire d'exécution ?
Donnes-tu le bon nom, il est difficile pour nous de savoir.

et aussi avec la bonne casse si tu es sous unixoide ou XP.

level1.txt != LEVEL1.TXT != Level1.txt etc.
13/10/2005, 19h34
Emmanuel Delahaye

Re: Ouverture d'un fichier avec son nom

Citation:

Envoyé par bonjour69

je charge un fichier dans lequel je sotck les valeurs dans un tableau.
ce fichier s'apelle : "level1.txt"
mais lorsque que je tape son nom le fichier ne s'ouvre le probleme doit resier dans la manipulation de la chaine de caractere

j'ai un debug assertion invalid

Il est inutile d'allouer dynamiquement une taille fixe à moins qu'on cherche à la réallouer...

scanf() avec "%s" attend l'adresse d'une chaine, pas l'adresse d'un pointeur sur chaine. Normal que ça ne fonctionne pas (il suffit de faite puts (myname); pour s'en rendre compte...

Code:

scanf("%s",myname);
13/10/2005, 19h52
bonjour69

gf

bonjour

Mon fichier est dans le bon repertoire, je tape effectivemment le bon nom mais egalement quand je fais
un

Code:

puts(myname);

j'obtient tient bien mon : level1.txt
donc le fichier doit être normalement bien chargé

mais bon bizarre j'ai le meme plantage ..

Merci
13/10/2005, 22h19
artatum

bonsoir,
si je peux me permettre une suggestion, tu devrais essayer:

Code:

fopen(myname,"r");

sans les guillemets.
13/10/2005, 22h21
Trap D

Plus c'est gros, moins on voit !!!
13/10/2005, 22h24
artatum

c'est l'avantage d'être petit comme moi, on fait tellement d'erreurs de ce type qu'on en devient parano à chaque relecture.
13/10/2005, 23h41
Emmanuel Delahaye

Citation:

Envoyé par artatum

bonsoir,
si je peux me permettre une suggestion, tu devrais essayer:

Code:

fopen(myname,"r");

sans les guillemets.

ARF!On va tous se laver les yeux!
http://www.cdc.gov/global/images/eyewash.jpg
14/10/2005, 13h20
bonjour69

hjh

Rebonjour,

Même avec le

Code:

fp = fopen(myname,"r");

j'ai toujours" pb d'ouverture fichier" qui s'affiche ://
le fichier est dans le bon repertoire courant
je l'appelle par level.txt avec les mon caracteres j'ai essayé sans l'extension mais impossible.

Merci
14/10/2005, 13h51
Emmanuel Delahaye

Re: hjh

Citation:

Envoyé par bonjour69

Même avec le

Code:

fp = fopen(myname,"r");

j'ai toujours" pb d'ouverture fichier" qui s'affiche ://
le fichier est dans le bon repertoire courant
je l'appelle par level.txt avec les mon caracteres j'ai essayé sans l'extension mais impossible.

Tu peux l'ouvrir avec autre chose ? Un éditeur...

Sinon, tu utilises quel environnement de développement ?
14/10/2005, 18h02
bonjour69

b

bonjour,

j'utilise visual C++ 6.0 c'est extremement bizarre , j'ai encore tripatouillé de partout mais rien a faire.. :/

++ Bruno
14/10/2005, 18h08
Trap D

As-tu essayé en donnant le chemin absolu du fichier ?
As tu essayé de lancer l'exe en cliquant dessus sous Windows ?
14/10/2005, 20h04
artatum

salut, est-ce que tu pourrais mettre le code complet ? parce que la version postée est incompilable, je ne peux rien tester.

bonsoir,
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 #include <stdio.h> #include <stdlib.h> FILE *fp; int main (void) { char *myname; myname = (char*)malloc(256 * sizeof (char)); scanf("%s",myname); fp = fopen(myname,"a"); fprintf(fp,"%s","hello"); fclose(fp); free(myname); return 0; }
j'ai essayé ce code (compilé sous gcc), et il marche impeccable, en m'ouvrant le fichier myname.

est-ce que cela te donne une piste ?

Voir 40 message(s) de cette discussion en une page