Discussion :

[poche]

voici mon 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
#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.

[Emmanuel Delahaye]

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

[poche]

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

[poche]

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

fallait mettre ça :

Code :

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

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

à la place de ça:

Code :

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

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 :

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

Creer(&tete);

ou

Code :

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

Creer(tete);

[salseropom]

Salut, il faut que tu mettes

Code :

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

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;

[poche]

Salut, il faut que tu mettes

Code :

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

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 :

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

Voiture *tete=NULL;

[poche]

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

Code :

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

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

[salseropom]

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

Code :

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

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 :

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

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

[poche]

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

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

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

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

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

[salseropom]

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

Code :

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

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

[salseropom]

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

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

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

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

[poche]

pourquoi sa marche toujours pas quand j'écris ç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
#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;
 
}

[Médinoc]

Code :

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

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

[poche]

ok merci