Problème de programme

**poche** · 21/03/2007, 08h50

Pourquoi mon programme reste bloqué dans la boucle de la fonction affichée, je pense que la fonction érire et lecture sont mal codées :

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
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
#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 Creer_vide (Voiture **p);
void Ecrire (Voiture *p);
void Lecture (Voiture **p);
void Afficher (Voiture *p);
 
int main(void)                 
{      
 
      Voiture *tete=NULL;
      Lecture(&tete);
	  Afficher(tete);
	  Creer_vide (&tete);
	  Creer (&tete);
	  //Afficher(tete);
	  Ecrire(tete);
      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");
	while(p!=NULL)
	{
		fwrite(p,sizeof(*p),1,fic);
		puts(p->marque);
		p=p->suivant;
	}
	fclose(fic);
	fic=NULL;
}
void Lecture (Voiture **p)
{	
	Voiture *voit=malloc(sizeof *voit);
	FILE *fic = NULL;
	fic = fopen("c:\\test.txt", "rb");
	while (fread(voit,sizeof(*voit),1,fic)!=NULL)
	{
		puts(voit->marque);
		voit->suivant=*p;
		*p=voit;
	}
	fclose(fic);
 
}
 
void Afficher (Voiture *p)
{
	while (p!= NULL)
	{
	printf("%s %s %s\n",p->marque,p->modele,p->puissance);
	p=p->suivant;
	}
}
void Creer_vide (Voiture **p)
{
	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=NULL;
		*p=voit;
	}        
}

Je sais, c'est encore le même programme, mais je dois le rendre impérativement ce soir!!!
Merci d'avance pour votre aide...

**stephl** · 21/03/2007, 09h04

Au moins un problème. Dans votre fonction de lecture, vous n'allouez de la mémoire qu'une seule fois (en dehors du while) et donc vous écrasez vos données en mémoire au fur et à mesure que vous les lisez.

**josse95** · 21/03/2007, 09h14

Oui, et du coup voit->suivant = voit, d'où le bouclage

**poche** · 21/03/2007, 09h29

Ok, j'ai compris mais ça me mais une erreur

error C2275: 'Voiture'*: utilisation non conforme de ce type comme expression

et j'ai rajouté ce code dans la fonction lecture :

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
void Lecture (Voiture **p)
{	
	Voiture *voit=malloc(sizeof *voit);
	FILE *fic = NULL;
	fic = fopen("c:\\test.txt", "rb");
	while (fread(voit,sizeof(*voit),1,fic)!=NULL)
	{
		puts(voit->marque);
		voit->suivant=*p;
		*p=voit;
		voit=NULL;
		Voiture *voit=malloc(sizeof *voit);
	}
	fclose(fic);
 
}

**stephl** · 21/03/2007, 09h33

Ne déclarez pas une nouvelle variable "voit"; il faut utiliser la même et faire l'appel à malloc() dans la boucle.

**Thierry Chappuis** · 21/03/2007, 09h34

Moi, je vois au moins un problème avec la saisie:

Code :

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

scanf("%s",&(voit->marque));

La syntaxe n'est ici pas correcte, et il faudrait écrire:

Code :

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

scanf("%s",voit->marque);

Au-delà de ça, ce n'est pas une façon sécurisée d'utiliser scanf(). Si tu tiens absolument à utiliser scanf(), il faut investir un peu temps pour apprendre à l'utiliser correctement, utiliser fgets() à la place, ou se construire ses propres entrées utilisteurs.

En l'état, les tes fonctions Creer() et Creer_vide() sont bugées. En plus de cela, enregistrer une structure telle quelle dans un fichier binaire est délicat du point de vue de la portabilité. J'espère que ton prof, qui va tester le programme, possède exactement la même architecture matérielle que toi, sinon tu risque d'avoir des surprises. Pour faire simple, j'aurais plutôt enregistré les données dans un fichier texte au format CSV (Comma Separated Values).

Il est également possible d'enregistrer une structure dans un fichier binaire de manière portable. Pour en savoir plus à ce sujet: http://emmanuel-delahaye.developpez....m#enreg_struct

Thierry

**poche** · 21/03/2007, 09h48

Envoyé par mujigka

Moi, je vois au moins un problème avec la saisie:

Code :

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

scanf("%s",&(voit->marque));

La syntaxe n'est ici pas correcte, et il faudrait écrire:

Code :

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

scanf("%s",voit->marque);

Au-delà de ça, ce n'est pas une façon sécurisée d'utiliser scanf(). Si tu tiens absolument à utiliser scanf(), il faut investir un peu temps pour apprendre à l'utiliser correctement, utiliser fgets() à la place, ou se construire ses propres entrées utilisteurs.

En l'état, les tes fonctions Creer() et Creer_vide() sont bugées. En plus de cela, enregistrer une structure telle quelle dans un fichier binaire est délicat du point de vue de la portabilité. J'espère que ton prof, qui va tester le programme, possède exactement la même architecture matérielle que toi, sinon tu risque d'avoir des surprises. Pour faire simple, j'aurais plutôt enregistré les données dans un fichier texte au format CSV (Comma Separated Values).

Il est également possible d'enregistrer une structure dans un fichier binaire de manière portable. Pour en savoir plus à ce sujet: http://emmanuel-delahaye.developpez....m#enreg_struct

Thierry

Je comprends pas trop prk mon prof doit doit avoir la même architecture matérielle que moi???
Sinon toi comment ferai tu pour enregistrer des structures dans un fichier binaire, et pour que n'importe qui puisse compiler mon programme sans problème???

**stephl** · 21/03/2007, 09h55

Envoyé par mujigka

En l'état, les tes fonctions Creer() et Creer_vide() sont bugées.

Je ne vois pas de bug dans la fonction Creer() (si ce n'est ce '&' mais ce n'est pas vraiment un bug). A moins que par "bug", vous n'entendiez le fait que ce n'est pas sécurisé (débordements de chaîne éventuels avec les scanf())...

**stephl** · 21/03/2007, 09h56

Envoyé par poche

Je comprends pas trop prk mon prof doit doit avoir la même architecture matérielle que moi???

cf Little Endian & Big Endian

**poche** · 21/03/2007, 10h32

Envoyé par mujigka

Moi, je vois au moins un problème avec la saisie:

Code :

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

scanf("%s",&(voit->marque));

La syntaxe n'est ici pas correcte, et il faudrait écrire:

Code :

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

scanf("%s",voit->marque);

Au-delà de ça, ce n'est pas une façon sécurisée d'utiliser scanf(). Si tu tiens absolument à utiliser scanf(), il faut investir un peu temps pour apprendre à l'utiliser correctement, utiliser fgets() à la place, ou se construire ses propres entrées utilisteurs.

En l'état, les tes fonctions Creer() et Creer_vide() sont bugées. En plus de cela, enregistrer une structure telle quelle dans un fichier binaire est délicat du point de vue de la portabilité. J'espère que ton prof, qui va tester le programme, possède exactement la même architecture matérielle que toi, sinon tu risque d'avoir des surprises. Pour faire simple, j'aurais plutôt enregistré les données dans un fichier texte au format CSV (Comma Separated Values).

Il est également possible d'enregistrer une structure dans un fichier binaire de manière portable. Pour en savoir plus à ce sujet: http://emmanuel-delahaye.developpez....m#enreg_struct

Thierry

J'ai été voir tes liens, mais il ya juste l'exemple pour un seul maillon et non pour plusieurs!!Pourtant, je pense avoir fais pareil que l'exemple!!!

**poche** · 21/03/2007, 10h46

Envoyé par stephl

Au moins un problème. Dans votre fonction de lecture, vous n'allouez de la mémoire qu'une seule fois (en dehors du while) et donc vous écrasez vos données en mémoire au fur et à mesure que vous les lisez.

En effet, j'écrase ce que j'ai mis en mémoire mais avant d'écraser, je copie le maillon dans la liste chaînée "tete"

Code :

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

1
2
3
4
5
6
while (fread(voit,sizeof(*voit),1,fic)!=NULL)
	{
		voit->suivant=*p;
		*p=voit;
 
	}

**stephl** · 21/03/2007, 10h52

Envoyé par poche

En effet, j'écrase ce que j'ai mis en mémoire mais avant d'écraser, je copie le maillon dans la liste chaînée "tete"

Code :

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

1
2
3
4
5
6
while (fread(voit,sizeof(*voit),1,fic)!=NULL)
	{
		voit->suivant=*p;
		*p=voit;
 
	}

Avec votre code, vous ne copiez que l'adresse. Au final, il n'y a qu'une seule structure allouée et donc les infos sont écrasées à chaque itération, et comme l'a précisé josse95, cela crée en outre une liste bouclée sur elle-même.

**Thierry Chappuis** · 21/03/2007, 10h54

Envoyé par stephl

Je ne vois pas de bug dans la fonction Creer() (si ce n'est ce '&' mais ce n'est pas vraiment un bug). A moins que par "bug", vous n'entendiez le fait que ce n'est pas sécurisé (débordements de chaîne éventuels avec les scanf())...

C'est ça! un espace de 20 caractère est alloué pour chacun des champs marque et modele de la structure Voiture, et 3 caractères pour le champs puissance. Il est très facile de faire déborder tout cela. scanf("%s", tampon) comme gets() représentent un trou de sécurité, et il est très facile pour un utilisateur moyen comme moi de faire planter le programme (volontairement ou pas!).

Thierry

**Thierry Chappuis** · 21/03/2007, 10h57

Envoyé par stephl

cf Little Endian & Big Endian

+ contraintes d'alignement.

Si tu enregistres ta liste de Voiture sous format binaire avec ton pc processeur x86 et que ton prof essaie de le lire avec son Mac, processeur PowerPC (big endian)... tu risques d'avoir des mauvaises surprises.

Thierry

**poche** · 21/03/2007, 10h57

Envoyé par stephl

Avec votre code, vous ne copiez que l'adresse. Au final, il n'y a qu'une seule structure allouée et donc les infos sont écrasées à chaque itération, et comme l'a précisé josse95, cela crée en outre une liste bouclée sur elle-même.

Si je copie l'adresse, normalement, j'arriverrai à lire se qui se trouve à cet adresse!!!Sinon comment faire si je veux copier ce qui est lis dans une liste chaînée??

**stephl** · 21/03/2007, 11h01

Envoyé par mujigka

+ contraintes d'alignement.

Je ne prétendais pas être exhaustif; le lien que vous avez fourni vers la page d'Emmanuel était en effet plus précis sur ces problèmes de portabilité.

**stephl** · 21/03/2007, 11h06

Envoyé par poche

Si je copie l'adresse, normalement, j'arriverrai à lire se qui se trouve à cet adresse!!!

Oui, mais comme vous écrasez ces données à chaque itération, vous ne serez capable de lire que les dernières. La solution consiste à déplacer le malloc() dans la boucle.

**poche** · 21/03/2007, 11h10

Envoyé par stephl

Oui, mais comme vous écrasez ces données à chaque itération, vous ne serez capable de lire que les dernières. La solution consiste à déplacer le malloc() dans la boucle.

ok, je l'ai fais mais une erreur c'est produite (voir le message N°4 qui se trouve sur la page 1/2).