Discussion :

[bonjour69]

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 :

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

[ShootDX]

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 :

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

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 :

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

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.

[Trap D]

Salut
Correction du programme

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

[gege2061]

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

Code :

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

return 0

mais un

Code :

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

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 :

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

$ ./configure && make && make install

par exemple.

[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é.

[bonjour69]

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

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

[Emmanuel Delahaye]

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.

[Emmanuel Delahaye]

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 :

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

scanf("%s",myname);

[bonjour69]

bonjour


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

Code :

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

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

[artatum]

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

Code :

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

fopen(myname,"r");

sans les guillemets.

[Trap D]

Plus c'est gros, moins on voit !!!

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

[Emmanuel Delahaye]

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

Code :

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

fopen(myname,"r");

sans les guillemets.

ARF!On va tous se laver les yeux!

[bonjour69]

Rebonjour,

Même avec le

Code :

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

       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

[Emmanuel Delahaye]

Même avec le

Code :

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

       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 ?

[bonjour69]

bonjour,


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


++ Bruno

[Trap D]

As-tu essayé en donnant le chemin absolu du fichier ?
As tu essayé de lancer l'exe en cliquant dessus sous Windows ?

[artatum]

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

[artatum]

bonsoir,

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