Discussion :

[poche]

Mon programme remplis mal le fichier, lorsque j'essaye de le lire et le mettre dans un maillon, il me sort n'importe quoi!!!

Voici mon code :

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
64
65
66
67
68
69
70
71
72
#include <stdio.h>
#include <stdlib.h>
#include <conio.h>
#include <string.h>
 
 
typedef struct Voiture        //Définition de la structure Voiture
{
        char marque[20];
        char modele[20];
        char puissance[3];
        struct Voiture *suivant;
}Voiture;
 
 
void Creer (Voiture **p);
void Ecrire (Voiture *p);
void Lecture (void);
 
int main(void)                 
{      
 
      Voiture *tete=NULL;
	  Creer (&tete);
	  Ecrire(tete);
	  Lecture();
	  //system("cls");
	 // printf("%s\n",tete->marque);
	  //printf("%s\n",tete->suivant->marque);
      free(tete); tete=NULL; /* il faut liberer la memoire. C'est tres imporant */
	  return EXIT_SUCCESS; 
 
 
 }
 
void Creer (Voiture **p)        //Définition de la fonction Creer qui est de type Voiture
{		
		Voiture *voit=malloc(sizeof *voit);
		if(voit != NULL)
		{
			printf("Entrez la marque :\n");
			scanf("%s",&(voit->marque));
			printf("Entrez le modele :\n");
			scanf("%s",&(voit->modele));
			printf("Entrez la puissance :\n");
			scanf("%s",&(voit->puissance));
			voit->suivant=*p;
			*p=voit;
		}        
}
void Ecrire(Voiture *p)
{	
 
	FILE *fic = NULL;
	fic = fopen("c:\\test.txt", "wb");
	fwrite(p,sizeof(p),1, fic);
	fclose(fic);
	fic=NULL;
}
void Lecture (void)
{	
	Voiture *voit=malloc(sizeof voit);
	FILE *fic = NULL;
	fic = fopen("c:\\test.txt", "rb");
	while(fread(voit,sizeof(voit),1,fic))
	{
		printf("%s,: %s : %s;",voit->marque,voit->modele,voit->puissance);
	}
 
	fclose(fic);
	fic=NULL;
}

[stephl]

Le second paramètre de fread() et fwrite() n'est pas bon, ce n'est pas la taille de la structure.

[poche]

J'ai essayé ça:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
void Lecture (void)
{	
	Voiture *voit=malloc(sizeof voit);
	FILE *fic = NULL;
	fic = fopen("c:\\test.txt", "rb");
	while(fread(voit,sizeof(Voiture),1,fic))
	{
		printf("%s,: %s : %s;",voit->marque,voit->modele,voit->puissance);
	}
 
	fclose(fic);
	fic=NULL;
}

et ça:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
void Lecture (void)
{	
	Voiture *voit=malloc(sizeof voit);
	FILE *fic = NULL;
	fic = fopen("c:\\test.txt", "rb");
	while(fread(voit,sizeof(*voit),1,fic))
	{
		printf("%s,: %s : %s;",voit->marque,voit->modele,voit->puissance);
	}
 
	fclose(fic);
	fic=NULL;
}

Et sa marche tjrs pas!!!

[tinico]

change aussi le fwrite !

[poche]

sa marche bien, la lecture s'affiche bien mais juste après ça bug, sa fait bugger windows!!!!!
J'ai fais ça :

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
64
65
66
67
68
69
70
71
72
#include <stdio.h>
#include <stdlib.h>
#include <conio.h>
#include <string.h>
 
 
typedef struct Voiture        //Définition de la structure Voiture
{
        char marque[20];
        char modele[20];
        char puissance[3];
        struct Voiture *suivant;
}Voiture;
 
 
void Creer (Voiture **p);
void Ecrire (Voiture *p);
void Lecture (void);
 
int main(void)                 
{      
 
      Voiture *tete=NULL;
	  Creer (&tete);
	  Ecrire(tete);
	  Lecture();
	  //system("cls");
	 // printf("%s\n",tete->marque);
	  //printf("%s\n",tete->suivant->marque);
      free(tete); tete=NULL; /* il faut liberer la memoire. C'est tres imporant */
	  return EXIT_SUCCESS; 
 
 
 }
 
void Creer (Voiture **p)        //Définition de la fonction Creer qui est de type Voiture
{		
		Voiture *voit=malloc(sizeof *voit);
		if(voit != NULL)
		{
			printf("Entrez la marque :\n");
			scanf("%s",&(voit->marque));
			printf("Entrez le modele :\n");
			scanf("%s",&(voit->modele));
			printf("Entrez la puissance :\n");
			scanf("%s",&(voit->puissance));
			voit->suivant=*p;
			*p=voit;
		}        
}
void Ecrire(Voiture *p)
{	
 
	FILE *fic = NULL;
	fic = fopen("c:\\test.txt", "wb");
	fwrite(p,sizeof(*p),1, fic);
	fclose(fic);
	fic=NULL;
}
void Lecture (void)
{	
	Voiture *voit=malloc(sizeof voit);
	FILE *fic = NULL;
	fic = fopen("c:\\test.txt", "rb");
	while(fread(voit,sizeof(*voit),1,fic))
	{
		printf("%s,: %s : %s;",voit->marque,voit->modele,voit->puissance);
	}
 
	fclose(fic);
	fic=NULL;
}

[stephl]

sa marche bien, la lecture s'affiche bien mais juste après ça bug, sa fait bugger windows!!!!!

Le malloc() dans la fonction de lecture n'est pas bon (mauvaise taille).

[tinico]

Ha oui, c'est vrai... Oula, c'est loin, mais je m'en souvient oui !!!

[tinico]

Normal,

Essaye ça dans la lecture :

Code :

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

1
2
 
Voiture *voit=malloc(sizeof *voit);

[tinico]

Salut,

Dans la fonction

Code :

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

void Ecrire(Voiture *p)

p est un pointeur... Il faut donc écrire comme ceci

Code :

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

fwrite(p,sizeof(*p),1, fic);

et non

Code :

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

fwrite(p,sizeof(p),1, fic);

et dans

Code :

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

void Lecture (void)

Essaye

Code :

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

while(fread(voit,sizeof(*voit),1,fic))

au lieu de

Code :

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

while(fread(voit,sizeof(voit),1,fic))

Sinon, juste un truc, la bugati Veyron fait 1001chevaux, la puissance devrais plutot etre un char(4) ! lol

Désolé si je dis n'importe quoi, ça fait longtemps que j'ai pas fait de C... mais de souvenir, c'est comme ça !

[tinico]

mince, jme suis fait grillé !! lol

[fearyourself]

Sinon, juste un truc, la bugati Veyron fait 1001chevaux, la puissance devrais plutot etre un char(4) ! lol

En effet, car pour avoir "1001", il faut 5 cases dans un tableau de caractères...
(un pour le '\0')

Jc