Pointeur dans une fonction ?

**sliiim6184** · 28/12/2006, 00h52

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

**Emmanuel Delahaye** · 28/12/2006, 01h03

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 :

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

**Thierry Chappuis** · 28/12/2006, 01h08

Parce que le type realisateur est définit par:

Code :

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

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 :

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

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