Récupérer le contenu d'un fichier texte et le copier dans une chaine

**nawal.02** · 28/05/2010, 03h28

Bonjour,
Donc voila Fgets ne marche plus a chaque fois que j'exécute tout s'arrête pourquoi??

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
void Copy_NewMsg_In_File (Message *msg)
{
	FILE *f = NULL;
	FILE *fc = NULL;
	f = fopen("C:\\glomosim\\glomosim\\network\\DataPacket.txt", "wb");
	if (f == NULL)
    {
		printf("Impossible d'ouvrir le fichier DataPacket.txt");
    }
    else
    {
		fprintf(f, "%s", msg->packet);
		printf("\n avant packet = %s\n", msg->packet); 
		main_LZW();
		fc = fopen("C:\\glomosim\\glomosim\\network\\test.out", "rb");
		if (fc == NULL)
		{
			printf("Impossible d'ouvrir le fichier test.out");
		}
		else
		{
			fgets(msg->packet, 100, fc);
//packet est de type char*
                    //peut être parce que fc  contient moins que 100 caractères!! 
			fclose(fc);
		}	
		fclose(f);
    }//else
   printf("\n packet = %s\n", msg->packet);// ici il affiche packet= 
                                                          //et hop tout s'arrête
}

S.V.P,
si seulement vous me dite ou se trouve l'erreur
merci d'avance

**Flow_75** · 28/05/2010, 07h17

Il faudrait en poster plus. Par exemple, la declaration de la structure Message...

Invité(e) · 28/05/2010, 07h55

Bonjour,

Il y a des contradictions dans ton code : les fichiers sont ouverts en mode binaire ("rb" et "wb") mais tu y accèdes avec des fonctions texte : fprintf et fgets.

Il faut soit ouvrir en texte pour utiliser ces fonctions, soit rester en binaire, mais utiliser fread, fwrite pour travailler avec les fichiers.

**nawal.02** · 28/05/2010, 17h06

Il faut soit ouvrir en texte pour utiliser ces fonctions, soit rester en binaire, mais utiliser fread, fwrite pour travailler avec les fichiers.

Voila aprés avoir fait les modifications necessaire j'ai toujours le même probleme

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
44
45
46
47
48
49
void Copy_NewMsg_In_File (Message *msg)
{
	FILE *f = NULL;
	FILE *fc = NULL;
	long lSize;
	size_t result;
	char *MyPacket;
	f = fopen("C:\\glomosim\\glomosim\\network\\DataPacket.txt", "wb");
	if (f == NULL)
    {
		printf("Impossible d'ouvrir le fichier DataPacket.txt");
    }
    else
    {
 
		//fprintf(f, "%s", msg->packet);
		fwrite (msg->packet , 1 , sizeof(msg->packet) , f );
		printf("\n avant packet = %s\n", msg->packet); 
		main_LZW();
		fc = fopen("C:\\glomosim\\glomosim\\network\\test.out", "rb");
		if (fc == NULL)
		{
			printf("Impossible d'ouvrir le fichier test.out");
		}
		else
		{
 
 
			fseek (fc , 0 , SEEK_END);
			lSize = ftell (fc);
			rewind (fc);
 
 
			MyPacket = (char*) malloc (sizeof(char)*lSize);
			if (MyPacket == NULL) {fputs ("Memory error",stderr); exit (2);}
 
 
			result = fread (MyPacket,1,lSize,fc);
			if (result != lSize) {fputs ("Reading error",stderr); exit (3);}
 
			//fgets(msg->packet, 10, fc);
			fclose(fc);
 
		}	
		fclose(f);
 
    }//else
   printf("\n MyPacket = %s\n", MyPacket); // On affiche la chaîne
}

**nawal.02** · 28/05/2010, 17h13

Envoyé par Flob91

Il faudrait en poster plus. Par exemple, la declaration de la structure Message...

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
struct message_str {
    short layerType;    
    short protocolType;
    short instanceId;    
    short eventType; 
    BOOL error;
    char* info;
    int infoSize;
    double smallInfoSpace[SMALL_INFO_SPACE_SIZE / sizeof(double)];
    int packetSize;
    char* packet;
    char* payLoad;
    int payLoadSize;
    clocktype packetCreationTime;  
};

Code :

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

message_str Message

Invité(e) · 28/05/2010, 17h14

EDIT
As tu vérifié la valeur de "sizeof(msg->packet)" ?
Tu peux remplacer sizeof(msg->packet) par strlen(msg->packet) : sizeof(msg->packet) va te renvoyer le taille du pointeur packet, pas la taille de la chaine de caractère.
/EDIT

Sinon, que te renvoie ton programme comme message ? As tu essayé d'utiliser un débogueur pour faire un exécution pas à pas.

**nawal.02** · 28/05/2010, 17h45

Envoyé par mabu

EDIT
As tu vérifié la valeur de "sizeof(msg->packet)" ?
Tu peux remplacer sizeof(msg->packet) par strlen(msg->packet) : sizeof(msg->packet) va te renvoyer le taille du pointeur packet, pas la taille de la chaine de caractère.
/EDIT

sizeof (msg->packe)=4

strlen(msg->packet) =4

La taille du file fc =1

sizeof (MyPacket)=4

strlen(MyPacket) =1 // j'ai ouvert le fichier fc et il contient un seul caractère ÿ mais il ne l'affiche pas

Envoyé par mabu

EDIT
[COLOR="Gray"]Sinon, que te renvoie ton programme comme message ? As tu essayé d'utiliser un débogueur pour faire un exécution pas à pas.
/EDIT

le message est le programme a cesser de fonctionner et il m'invite a déboguer je le fait, mais la comprend rien avec le langage machine

Invité(e) · 28/05/2010, 18h14

Voici comment j'ai testé ton code chez moi :

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
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
typedef struct message_str {
    char *packet;
} Message;
 
void test(Message * msg)
{
    FILE *f = NULL;
    FILE *fc = NULL;
    long lSize;
    size_t result;
    char *MyPacket;
    f = fopen("DataPacket.txt", "wb");
    if (f == NULL) {
        printf("Impossible d'ouvrir le fichier DataPacket.txt");
    } else {
        /* remplacement sizeof par strlen */
        fwrite(msg->packet, 1, strlen(msg->packet), f);
        printf("\n avant packet = %s\n", msg->packet);
        fc = fopen("test.out", "rb");
        if (fc == NULL) {
            printf("Impossible d'ouvrir le fichier test.out");
        } else {
 
            fseek(fc, 0, SEEK_END);
            lSize = ftell(fc);
            rewind(fc);
 
            /* +1 pour laisser la place pour le \0 final*/
            MyPacket = (char *) malloc(sizeof(char) * lSize + 1);
            if (MyPacket == NULL) {
                fputs("Memory error", stderr);
                exit(2);
            }
 
            result = fread(MyPacket, 1, lSize, fc);
            if (result != lSize) {
                fputs("Reading error", stderr);
                exit(3);
            }
            /* ajout du \0 final*/
            MyPacket[lSize] = '\0';
            fclose(fc);
 
        }
        fclose(f);
 
    }                           
    printf("\n MyPacket = %s\n", MyPacket);     // On affiche la chaîne
}
 
int main(void)
{
    Message MSG;
    MSG.packet = strdup("Bonjour le monde");
    test(&MSG);
    free(MSG.packet);
 
    return 0;
}

Voici ce que j'obtiens :

Code :

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

1
2
3
 avant packet = Bonjour le monde
Impossible d'ouvrir le fichier test.out
 MyPacket = Bonjour le monde

et une fois que le fichier test.out est en place :

Code :

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

1
2
3
 avant packet = Bonjour le monde
 
 MyPacket = Bonjour le monde

**nawal.02** · 28/05/2010, 18h15

Pour le problème du programme qui s'arrête c'est bon c'est résolu, effectivement le débogueur m'a aider a trouver ou était l'erreur, j'ai bien ouvert mes yeux cette fois ci

Merci mabu.
Mais pour l'affichage du contenu de MYPACKET c'est toujours rien
voila ce que j'ai à chaque fois
MYpacket=
pourquoi il n'affiche rien il ne reconnait pas le caractère??
et pourtant sa taille est bien 1 Octet

Invité(e) · 28/05/2010, 18h21

Envoyé par nawal.02

pourquoi il n'affiche rien il ne reconnait pas le caractère??
et pourtant sa taille est bien 1 Octet

MyPacket ne se termine pas par un '\0' donc, l'affichage peut faire n'importe quoi.

Regarde ce que j'ai commenté, cela pourra peut-être t'aider :

Code :

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

1
2
3
4
5
/* +1 pour laisser la place pour le \0 final*/
            MyPacket = (char *) malloc(sizeof(char) * lSize + 1);
           [...]
            /* ajout du \0 final*/
            MyPacket[lSize] = '\0';

**nawal.02** · 28/05/2010, 18h58

Envoyé par mabu

MyPacket ne se termine pas par un '\0' donc, l'affichage peut faire n'importe quoi.

Regarde ce que j'ai commenté, cela pourra peut-être t'aider :

Code :

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

1
2
3
4
5
/* +1 pour laisser la place pour le \0 final*/
            MyPacket = (char *) malloc(sizeof(char) * lSize + 1);
           [...]
            /* ajout du \0 final*/
            MyPacket[lSize] = '\0';

j'ai fait exectement comme vous, et toujours rien alors vous m'avez donner une idée avec

Code :

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

MyPacket[lSize] = '\0';

donc j'ai fait

Code :

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

MyPacket[lSize] = 't';

et le resultat est MyPacket= t
un espace aulieu de ÿ !!!!!

Invité(e) · 28/05/2010, 21h53

Ah mais oui, le caractère 'ÿ' ne doit pas être affichable en ASCII standard.
Essaye avec un autre fichier.

voir la fonction isgraph (ctype.h)

**nawal.02** · 29/05/2010, 01h08

Envoyé par mabu

Ah mais oui, le caractère 'ÿ' ne doit pas être affichable en ASCII standard.
Essaye avec un autre fichier.

voir la fonction isgraph (ctype.h)

J'ai essayé toutes les fonctions de ctype.h et le résultat est toujours nul .
Cela veut dire que l'affichage n'est pas évident mais le caractère est belle et bien existant??