Ouverture d'un fichier avec son nom

Version imprimable

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 ?

Bonjour !!!

et bien je vous remercie le code suivant marche parfaitement :

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
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
 
#include <stdlib.h>
#include <stdio.h>
#include <malloc.h>
 
void charger_fichier(int Tab[][100]);
void affiche_map(int Tab[][100]);
void init_tab(int Tab[][100]);
 
 
void main (void)
 
{
   int Map[5][100];
 
	init_tab(Map);
   charger_fichier(Map);
   affiche_map(Map);
 
}
 
 
void init_tab(int Tab[][100])
{
	for ( int i =0 ; i < 5 ; i++ )
         for ( int j=0 ; j< 6 ; j++)
			 Tab[i][j]=0;
}
 
void charger_fichier(int Tab[][100])
{ 
 
	FILE *fp; 
 
	char *myname;
 
   myname = (char*)malloc(256 * sizeof (char));
 
   if (myname == NULL)
      {
         fprintf(stderr, "Pb malloc\n");
 
      }
 
         scanf("%s",myname);
 
         fp = fopen(myname,"r");
 
 
 
        if (fp == NULL)
      {
          fprintf(stderr, "Pb ouverture %s\n", myname);
 
 
      }
 
 
             for ( int i =0 ; i < 5 ; i++ )
                 for ( int j=0 ; j< 6 ; j++)
                     fscanf(fp,"%d",&Tab[i][j]);
 
   fclose(fp);
 
 free(myname);
 
 
}  
void affiche_map(int Tab[][100])
{  
 
	  for ( int i =0 ; i < 5 ; i++ )
	  {
 
         for ( int j=0 ; j< 5 ; j++)
		 {
               printf("%d",Tab[i][j]);
		 }
			printf("\n");
	  }
}

Sans pour autant comprendre pourquoi...
Merci a vous

15/10/2005, 13h36
artatum

bien que petit, je vais de nouveau me permettre une nouvelle suggestion:

Code:

void main(void)

à proscrire, les 2 versions acceptées par la norme iso sont
. int main(void)
. int maint (int argc, char *argv[ ]), où argc représente le nombre d'arguments avec lesquels la fonction main est appelée (par une commande transmise au shell, par ex), et *argv[ ] un vecteur de chaînes qui est la liste de ces arguments.

Re: f

Citation:

Envoyé par bonjour69

et bien je vous remercie le code suivant marche parfaitement :

Moyennement, je dirais...

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
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
 
#include <stdlib.h>
#include <stdio.h>
 
/* -ed-
#include <malloc.h>
 
non standard et probablement inutile
*/
 
void charger_fichier(int Tab[][100]);
void affiche_map(int Tab[][100]);
void init_tab(int Tab[][100]);
 
/* -ed-
void main (void)
 
<a href="http://emmanuel-delahaye.developpez.com/notes.htm" target="_blank">http://emmanuel-delahaye.developpez.com/notes.htm</a>
 
On en a pas deja parle de ca ?
 
*/
int main (void)
{
    int Map[5][100];
 
    init_tab(Map);
    charger_fichier(Map);
    affiche_map(Map);
 
    /* -ed- puisque main() retourne un int... */
 
    return 0;
 
}
 
 
void init_tab(int Tab[][100])
{
    /* -ed
       for ( int i =0 ; i < 5 ; i++ )
             for ( int j=0 ; j< 6 ; j++)
     
    - C99 uniquement...
    - Preferer les {}
    - Pourquoi 6 ? Le mieux est de passer la taile en parametre... 
    */
    int i;
    for ( i =0 ; i < 5 ; i++ )
    {
        int j;
        for ( j=0 ; j< 6 ; j++)
        {
            Tab[i][j]=0;
        }
    }
}
 
void charger_fichier(int Tab[][100])
{
    FILE *fp;
 
    char *myname;
 
    /* -ed-
    pourquoi une allocation dynamique ? 
    Un tableau statique en memoire auto suffit. 
    C'est du jetable.. 
     
        char myname[256];
     
    Et si tu y tiens, pourquoi le cast ?
    sizeof(char) vaut 1 par definition, donc redondant...
     
    <a href="http://emmanuel-delahaye.developpez.com/notes.htm" target="_blank">http://emmanuel-delahaye.developpez.com/notes.htm</a>
     
    */
    myname = (char*)malloc(256 * sizeof (char));
 
    if (myname == NULL)
    {
        fprintf(stderr, "Pb malloc\n");
        /* -ed-
        insuffisant. En cas d'erreur, il ne faut pas continuer 
        avec myname == NULL, car le comportement est indetermine. 
        */
 
    }
 
    /* -ed- saisie 'aveugle'. L'utilisateur ne sait pas ce qu'il faut faire... */
    scanf("%s",myname);
    /* -ed- fonction dangereuse car mal utilisee ici.
     
    <a href="http://emmanuel-delahaye.developpez.com/notes.htm" target="_blank">http://emmanuel-delahaye.developpez.com/notes.htm</a>
    <a href="http://emmanuel-delahaye.developpez.com/notes.htm" target="_blank">http://emmanuel-delahaye.developpez.com/notes.htm</a>
     
    il me semble avoir deja donne ces liens 3000 fois, 
    mais visiblement, ca a du mal a rentrer... 
    */
    fp = fopen(myname,"r");
 
 
 
    if (fp == NULL)
    {
        fprintf(stderr, "Pb ouverture %s\n", myname);
 
        /* -ed-
        insuffisant. En cas d'erreur, il ne faut pas continuer 
        avec fp == NULL, car le comportement est indetermine. 
        (chez moi, crash velu...) 
        */
    }
 
    /* -ed-
                 for ( int i =0 ; i < 5 ; i++ )
                     for ( int j=0 ; j< 6 ; j++)
    */
    int i;
    for ( i =0 ; i < 5 ; i++ )
    {
        int j;
        for ( j=0 ; j< 6 ; j++)
        {
            fscanf(fp,"%d",&Tab[i][j]);
            /* -ed-
            La detection de fin de lecture n'est pas faite. 
            Il faut savoir qu'elle peut survenir a tout moment. 
            (erreur, fin de fichier etc.)
            
            Si on insiste, le comportement est indetermine. 
            Le code retour de la fonction de lecture doit etre 
            evalue (et ce pour plusieurs raisons). 
            */
        }
    }
    fclose(fp);
 
    free(myname);
 
 
}
void affiche_map(int Tab[][100])
{
 
    /* -ed-
         for ( int i =0 ; i < 5 ; i++ )
         {
             for ( int j=0 ; j< 5 ; j++)
    */
 
    int i;
    for ( i =0 ; i < 5 ; i++ )
    {
        int j;
        for ( j=0 ; j< 6 ; j++)
        {
            printf("%d",Tab[i][j]);
        }
        printf("\n");
    }
}

15/10/2005, 19h36
bonjour69

ok

Bonjour,

Merci pour toutes ces précisions :)

Cordialement