petit problème sur les listes chaînées

Version imprimable

voici mon 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
#include <stdio.h>
#include <stdlib.h>
#include <conio.h>
#include <string.h>
 
typedef struct Voiture        //Définition de la structure Voiture
{
        char *marque;
        char *modele;
        char *puissance;
        struct Voiture *suivant;
}Voiture;
 
 
void Creer (Voiture *);
 
 
void main(void)                 
{                      
      Voiture *tete=NULL;
	  tete = malloc(sizeof *tete);
	  Creer (&tete);	  
 //     ecrire (&tete);
 }
 
void Creer (Voiture *p)        //Définition de la fonction Creer qui est de type Voiture
{
		Voiture *voit=NULL;
		voit = malloc(sizeof *voit);
        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;
 
}

il se compile bien et se génère bien mais après avoir entrer le marque dans le programme, une erreur survient et me fais arrêter l'application!!!

D'ou cela peux venir???
Merci d'avance.

19/03/2007, 08h18
Emmanuel Delahaye

Citation:

Envoyé par poche

voici mon programme :
<rien>

Tu n'as toujours pas compris comment on mettait les balises codes... Il suffit pourtant de lire les règles du forum où tout est expliqué...

Et je rappelle que, contrairement à d'autres langages, le C est sensible à la casse : creer != Creer...
19/03/2007, 08h35
poche

Citation:

Envoyé par Emmanuel Delahaye

Tu n'as toujours pas compris comment on mettait les balises codes... Il suffit pourtant de lire les règles du forum où tout est expliqué...

Et je rappelle que, contrairement à d'autres langages, le C est sensible à la casse : creer != Creer...

Merci je m'était trompé, à la place de [CODE] j'avais mis [QUOTE], désolé...
19/03/2007, 08h49
poche

je crois que j'ai trouvé l'erreur:

fallait mettre ça :

Code:

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

à la place de ça:

Code:

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

Est ce bon?Est ce la bonne syntaxe?

Et sinon quand j'appel la fonction Creer() il faut que je mette en paramettre "&tete ou tete, c'est à dire:

Code:

Creer(&tete);

ou

Code:

Creer(tete);
19/03/2007, 09h01
salseropom
Salut, il faut que tu mettes
Code:

1 2 Creer(tete);
car tete est est Voiture *

1) après un malloc, teste si ton pointeur n'est pas nul
2) le prototype de main n'est pas void main(void) mais int main(void) ou bien int main(int argc,char ** argv)
3) rajoute un return EXIT_SUCCESS à la fin du main
4) il faut que tu fasses un free(tete), tete=NULL;
19/03/2007, 09h11
poche
Citation:
Envoyé par salseropom

Salut, il faut que tu mettes

Code:

1 2 Creer(tete);

car tete est est Voiture *

1) après un malloc, teste si ton pointeur n'est pas nul
2) le prototype de main n'est pas void main(void) mais int main(void) ou bien int main(int argc,char ** argv)
3) rajoute un return EXIT_SUCCESS à la fin du main
4) il faut que tu fasses un free(tete), tete=NULL;
Je ne comprends pas ton petit 4), pourtant je fais bien :

Code:

Voiture *tete=NULL;

Sinon quand je fais ça, ça se compile, mais sa mais une erreur lors de l'application rendu au printf !!!!:

Code:

1
2
3
4
5
6
7
8
9
int main(void)                 
{                      
      Voiture *tete=NULL;
	  tete = malloc(sizeof *tete);
	  Creer (tete);	
	  printf("%s",tete->marque);
 //     ecrire (&tete);
	  return EXIT_SUCCESS; 
 }

Citation:

Envoyé par poche

Sinon quand je fais ça, ça se compile, mais sa mais une erreur lors de l'application rendu au printf !!!!:

Code:

1
2
3
4
5
6
7
8
9
int main(void)                 
{                      
      Voiture *tete=NULL;
	  tete = malloc(sizeof *tete);
	  Creer (tete);	
	  printf("%s",tete->marque);
 //     ecrire (&tete);
	  return EXIT_SUCCESS; 
 }

je corrige

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
int main(void)                 
{                      
      Voiture *tete = malloc(sizeof *tete);
     if(tete==NULL)
    {
     fprintf(stderr,"Erreur malloc\n");
     exit(EXIT_FAILURE);
    }
	  Creer (tete);	
	  printf("%s\n",tete->marque);
       free(tete); tete=NULL; /* il faut liberer la memoire. C'est tres imporant */
	  return EXIT_SUCCESS; 
 }

J'ai fais ça, est ce que ça marche?ai je bien utilisé la bonne syntaxe?

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
#include <stdio.h>
#include <stdlib.h>
#include <conio.h>
#include <string.h>
 
 
typedef struct Voiture        //Définition de la structure Voiture
{
        char *marque;
        char *modele;
        char *puissance;
        struct Voiture *suivant;
}Voiture;
 
 
void Creer (Voiture **p);
 
 
int main(void)                 
{                      
      Voiture *tete=NULL;
	  tete = malloc(sizeof *tete);
	  if (tete=NULL) exit(EXIT_FAILURE);
	  Creer (&tete);
	  printf("%s\n",&(tete->marque));
       free(tete); tete=NULL; /* il faut liberer la memoire. C'est tres imporant */
	  return EXIT_SUCCESS; 
 
 
 
 
 //     ecrire (&tete);
	 // return EXIT_SUCCESS; 
 }
 
void Creer (Voiture **p)        //Définition de la fonction Creer qui est de type Voiture
{
		Voiture *voit=NULL;
		voit = malloc(sizeof *voit);
		if (voit==NULL) exit(-1);
        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;
 
}

19/03/2007, 11h07
salseropom
comme dans ton autre mesage, tu dois allouer de l'espace pour tes chaines. Dans la fonction Creer, il faut que tu fasses qqch du genre
Code:

1 2 3 4 5 6 p->marque=malloc(20*sizeof(*p->marque)); /* tester is p->marque == NULL */ p->modele=malloc(20*sizeof(*p->modele)); /* tester si p->modele == NULL */
et après ca devrait marcher... et n'oublie pas de libérer la mémoire à la fin de ton prgm

Citation:
Envoyé par poche
J'ai fais ça, est ce que ça marche?ai je bien utilisé la bonne syntaxe?
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 #include <stdio.h> #include <stdlib.h> #include <conio.h> #include <string.h> typedef struct Voiture //Définition de la structure Voiture { char *marque; char *modele; char *puissance; struct Voiture *suivant; }Voiture;
je ne suis pas sûr que cette syntaxe soit acceptée...
typedef struct Voiture {blabla} Voiture; ??

Citation:

Envoyé par poche

J'ai fais ça, est ce que ça marche?ai je bien utilisé la bonne syntaxe?

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
#include <stdio.h>
#include <stdlib.h>
#include <conio.h>
#include <string.h>
 
 
typedef struct Voiture        //Définition de la structure Voiture
{
        char *marque;
        char *modele;
        char *puissance;
        struct Voiture *suivant;
}Voiture;
 
 
void Creer (Voiture **p);
 
 
int main(void)                 
{                      
      Voiture *tete=NULL;
	  tete = malloc(sizeof *tete);
	  if (tete=NULL) exit(EXIT_FAILURE);
	  Creer (&tete);
	  printf("%s\n",&(tete->marque));
       free(tete); tete=NULL; /* il faut liberer la memoire. C'est tres imporant */
	  return EXIT_SUCCESS; 
 
 
}

non, ce serait plutôt :

Code:

1
2
 
printf("%s\n",tete->marque);

pourquoi sa marche toujours pas quand j'écris ça :

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
#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);
 
 
int main(void)                 
{                      
      Voiture *tete=NULL;
	  tete = malloc(sizeof *tete);
	  if (tete=NULL) exit(EXIT_FAILURE);
	  Creer (tete);
	  printf("%s\n",tete->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=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;
 
}

19/03/2007, 16h44
Médinoc
Code:

1 2 3 4 5 void Creer (Voiture *p) //Définition de la fonction Creer qui est de type Voiture { <snip> p=voit; }
Ne sert à rien car p est une copie locale du pointeur...

Comme on l'a déjà dit, si tu veux modifier le pointeur lui-même dans Creer(), tu dois passer un pointeur de pointeur...
19/03/2007, 16h53
poche

ok merci