Pointeur dans une fonction ?

Version imprimable

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
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
#include <stdio.h>
#include <stdlib.h>
 
typedef struct Film* film;
 
struct Film
       {
       char titre[40];
       film filmNext;
       film allfilmNext;
       };
 
typedef struct Realisateur* realisateur;
 
struct Realisateur
       {
       char real[20];
       film filmNext;
       realisateur realisateurNext;
       };
 
int main()
{
    //Declaration des variables
	int choix;// entier, test,i = 0;
	realisateur premreal=NULL, dernreal=NULL, courreal=NULL;
	film premfilm=NULL, dernfilm=NULL, courfilm=NULL;
	//LISTE list = NULL;
 
    printf("\n");
	printf("\t                      VIDEOTHEQUE");
 
	printf("\n\n\n");
	do
	{
		printf("\t\t\t\n                                 MENU ");
		printf("\n\n");
 
        printf("\n        *****************************************************");
        printf("\n        *                                                   *");
        printf("\t\t\t    *    - Consulter la liste des realisateurs, tapez[1]*");
		printf("\t\t\t    *    - Consulter la liste des films, tapez.......[2]*");
		printf("\t\t\t    *    - Ajouter un realisateur, tapez.............[3]*");
		printf("\t\t\t  
	    printf("\n        *                                                   *");
        printf("\n        *****************************************************");
 
		printf("\n\n       Entrez votre choix (1-9) => ");
		scanf("%d", &choix);
		printf("\nchoix = %d", choix);
 
 
		printf("\n\n");
 
		// Selon le choix effectué, on execute les fonctions
		switch (choix)
        {
			case 3: AddRealisateur(premreal);
				break;
    	}
    }	
	while(choix != 9);
}
 
void AddRealisateur(realisateur tmp)
{
     //Créer un nouveau réalisateur
     realisateur temp;
     printf("Nom du réalisateur à insérer dans la base de données:\n");
     scanf("%s", temp->real);
     temp->filmNext = NULL;
     temp->realisateurNext = NULL;
     if(tmp!=NULL)
     {
                  //Pointer vers le dernier réalisateur connu
                  while(tmp->realisateurNext != NULL)
                  {
                                tmp = tmp->realisateurNext;
                  }
                  tmp->realisateurNext = temp;
     }
     else
     {
                  tmp=temp;
     }
     return tmp;
}

Bonsoir,

Bon j'ai cherché un bout de temps et c'est la premiere fois que j'utilise le langage C dans un projet informatique.
Je voudrais savoir pourquoi ya t'il une erreur lors de l'affection d'un nom de réalisateur à la variable temp->real ...

Ai je oublier une chose ?

Ce programme est un petit morceau du projet.

Merci d'avance

Citation:

Envoyé par sliiim6184

Je voudrais savoir pourquoi ya t'il une erreur lors de l'affection d'un nom de réalisateur à la variable temp->real ...

Il faut utiliser strcmp().

Tes données sont mal organisées. D'un coté on doit avoir ce qui concerne la 'charge utile', c'est à dire une structure contenant les données elle-mêmes, et de l'autre, les informations de chainage si on a affaire à une liste chainée. Le tout étant placé dans un noeud (node).

Autrement dit, je te conseille de recommencer sur ces bases :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 struct film { char titre[40]; }; struct film_node { struct film film; struct film_node *p_next; }; struct Realisateur { char real[20]; film_node *film_first; }; struct real_node { struct Realisateur real; struct real_node *p_next; };

Parce que le type realisateur est définit par:

Code:

typedef struct Realisateur *realisateur

Il s'agit donc d'un pointeur sur une structure Realisateur. C'est une très mauvaise pratique de masquer l'usage d'un pointeur derrière un alias de type, car on en viens à oublier la nature des variables qu'on manipule. Le code ci-dessous pose problème:
Code:

1 2 3 4 5 6 7 void AddRealisateur(realisateur tmp) { //Créer un nouveau réalisateur realisateur temp; printf("Nom du réalisateur à insérer dans la base de données:\n"); scanf("%s", temp->real); temp->filmNext = NULL;
temp est donc un pointeur sur une structure Realisateur. Ce pointeur n'est pas initialisé et la valeur contenue dans temp peut être n'importe quoi et ne correspond à coup sûr pas à une adresse valable en mémoire. En tout les cas, temp ne pointe pas sur une structure Realisateur et le déréférencement de temp en 2ème argument de la fonction scanf va probablement entraîner une erreur de segmentation. Idem pour temp->filmNext.

Thierry

28/12/2006, 11h00
sliiim6184

Merci beaucoup ...

Je vais revoir donc mon programme entierement :mouarf:

Auriez vous un exemple à me fournir d'un code source sur laquelle je pourrais en apprendre davantage sur le langage C ?
28/12/2006, 11h32
Thierry Chappuis

Tu trouveras de nombreuses sources écrites en langage C ici: http://c.developpez.com/sources/c/

Thierry