Discussion :

[Trivol]

Bonjour,

Je m'adresse à vous dans l'espoir de trouver une réponse à mon problème (bien entendu j'ai déjà fait le tour de google et de la FAQ).

J'ai un programme constitué de :

struct.h -> contient toutes les includes et les definitions des variables globales

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
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
 
#ifndef _STRUCT_H_
#define _STRUCT_H_
 
/* 	inclusion des bibliotheques necessaires */
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <time.h>
//#include <stdbool.h>
#include <pthread.h>
#include <ncurses.h>
 
/* 	Definition des constantes du programme */
 
#define TAILLE_TABLE 1009
#define TAILLE_DESTINATION 20
#define TAILLE_NOM 20
#define TAILLE_PRENOM 20
#define TAILLE_COMPAGNIE 30
#define TAILLE_NUMERO 2
#define TAILLE_MAIL 50
#define TAILLE_MODELE 10
#define TAILLE_DATE 8
#define TAILLE_DESCRIPTION 1000
#define TAILLE_NUMERO_VOL 5
 
/* Definition du temps minimum avant un echange */
 
// 10 jours en secondes
#define TEMPS_ANNUL 864000
 
// Constante pour les arbres AVL
 
#define EQUILIBRE 0
#define VERS_GAUCHE +1
#define VERS_DROITE -1
 
// Constante pour la table de hash
 
#define NON_TROUVE -1
 
// Pour l'identification
 
#define NON_IDENTIFIE -1
 
// Pour l'affichage
 
#define LARGEUR 100
#define HAUTEUR 50
 
/* Petite macro bien pratique */
 
#define ABS(a) ((a) > 0 ? (a) : (-(a)))
 
 
/****************************************************************************
 
        DECLARATION DES STRUCTURES UTILISEES DANS LE PROGRAMME
        
****************************************************************************/
 
 
/* 	Structure pour les clients avec les structures pour la gestion de la
	fidelite du client et celle de l'historique de ses vols */
 
struct histo{
 
	long date;
	char destination[TAILLE_DESTINATION];
	char num_vol[TAILLE_NUMERO_VOL];
	long km;
	bool statut;
	struct histo * next;
}histo;
 
typedef struct histo * histo_pt;
 
struct personne{
 
	char nom[TAILLE_NOM];
	char prenom[TAILLE_PRENOM];
	char numero[TAILLE_NUMERO];
	long points;
	long km;
	time_t anciennete;
	histo_pt historique;
};
 
typedef struct personne personne_t;
 
/////////////////////////////////////////////////////////////
 
/*	Structure pour charger la liste des modele d'avion existant et leurs infos
	comme une description */
 
typedef struct avion_load{
 
	char modele[TAILLE_MODELE];
	char description[TAILLE_DESCRIPTION];
	long capacite;
	struct avion_load * next;
}avion_load;
 
typedef struct avion_load * liste_avion;
 
////////////////////////////////////////////////////////////
 
/* 	Structure utiliser pour modeliser en memoire les passagers ayant deja
	reserver a un jour j */
 
typedef struct cell_passager{
 
	long hash;
	struct cell_passager * next;
}cell_passager;
 
typedef struct cell_passager * liste_passager; 
 
/* Structure en arbre pour ranger les vols a modifier en arbre AVL plus tard*/
 
typedef struct Noeud_vol{
 
	char numero[TAILLE_NUMERO];
	char horaire[TAILLE_DATE];
	char modele[TAILLE_MODELE];
	char compagnie[TAILLE_COMPAGNIE];
	long distance;
	long capacite;
	char destination[TAILLE_DESTINATION];
	long facteur_equilibre;
 
	liste_passager passager[31];
 
	struct Noeud_vol *droit;
	struct Noeud_vol *gauche;
 
}Noeud_vol;
 
typedef struct Noeud_vol NoeudAVL;
 
/*****************************************************************************
 
        DECLARATION DES VARIABLES GLOBALES
        
*****************************************************************************/
 
 
/* Racine de l'arbre des vols */
 
NoeudAVL * racine = NULL;
 
long rotation = 0;
 
/* Liste des avions charges en memoire */
 
liste_avion listing_avion = NULL;
 
/* Table de hashage */
 
personne_t table_hashage[TAILLE_TABLE];
 
/* Date de la derniere sauvegarde qui permet de dialoguer avec le thread de sauvegarde */
 
time_t temps_start = 0;
 
long avance_time = 0;
 
/* temps utilisé dans tout le programme comme temps courant, mit a jour par la thread */
 
time_t temps_courant = 0;
 
/* Variable pour verifier l'identification */
 
long identification = NON_IDENTIFIE;
 
/*****************************************************************************
 
        LISTE DES DECLARATION DE FONCTIONS UTILISEES DANS LE PROGRAMME
        
*****************************************************************************/
 
/* Fonctions relatives aux arbres AVL */
 
long insertion_noeud(NoeudAVL *s_arbre, NoeudAVL *nouveau_noeud, NoeudAVL *pere);
long rotation_gauche(NoeudAVL *s_arbre, NoeudAVL *pere);
long rotation_droite(NoeudAVL *s_arbre, NoeudAVL *pere);
 
///////////////////////////////////////////////////////////////
 
/*	Fonctions relatives aux tables de hashage */
 
void initialisation_table();
long code_hashage_1(char clef[]);
long code_hashage_2(char clef[]);
void insertion_element(personne_t nouvel_element);
void affichage_table();
long recherche_element(char clef[], char clef2[]);
....
....
 
/* Fonction des threads */
 
void *task_a (void *p_data);
 
#endif

et plusieurs fichiers :

load_client.c
load_vol.c
...
du type :

Code :

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

1
2
3
4
5
6
 
#include "struct.h"
 
fonction...
...
...

enfin mon makefile :

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
 
eflyer: avl.o reservation.o load_vol.o load_avions.o hash.o load_clients.o ncurses.o affichage_liste.o main.o
	gcc -o eflyer.exe main.o avl.o reservation.o load_vol.o load_avions.o hash.o load_clients.o ncurses.o affichage_liste.o -lncurses -lpthread
 
avl.o: avl.c struct.h
	gcc -c avl.c
 
reservation.o: reservation.c struct.h
	gcc -c reservation.c
 
load_vol.o: load_vol.c struct.h
	gcc -c load_vol.c
 
load_avions.o: load_avions.c struct.h
	gcc -c load_avions.c
 
hash.o: hash.c struct.h
	gcc -c hash.c
 
load_clients.o: load_clients.c struct.h
	gcc -c load_clients.c
 
ncurses.o: ncurses.c struct.h
	gcc -c ncurses.c
 
affichage_liste.o: affichage_liste.c struct.h
	gcc -c affichage_liste.c
 
main.o: main.c struct.h
	gcc -c main.c

et malheureusement a la compilation j'ai un résultat dans ce genre la :

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
 
gcc -c avl.c
gcc -c reservation.c
gcc -c load_vol.c
gcc -c load_avions.c
gcc -c hash.c
gcc -c load_clients.c
gcc -c ncurses.c
gcc -c affichage_liste.c
gcc -o eflyer.exe main.o avl.o reservation.o load_vol.o load_avions.o hash.o load_clients.o ncurses.o affichage_liste.o
/usr/bin/ld: multiple definitions of symbol _avance_time
main.o definition of _avance_time in section (__DATA,__data)
avl.o definition of _avance_time in section (__DATA,__data)
/usr/bin/ld: multiple definitions of symbol _identification
main.o definition of _identification in section (__DATA,__data)
avl.o definition of _identification in section (__DATA,__data)
/usr/bin/ld: multiple definitions of symbol _listing_avion
main.o definition of _listing_avion in section (__DATA,__data)
avl.o definition of _listing_avion in section (__DATA,__data)
...

J'ai bien compris que c'est un problème de variables globales définies plusieurs fois. Pourtant j'ai bien mis les ifndef. J'ai par ailleurs vu dans la FAQ quelque chose à ce propos mais je ne comprend pas très bien comment résoudre le problème.

Enfin deuxième problème :

Voici mon main :

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
 
#include "struct.h"
 
void *task_a (void *p_data){
 
	time_t temps_temp;
	// Cette fonction travail uniquement sur des variables globales
 
	while(1){
 
           // Toutes des variables de types time_t
		temps_courant = temps_courant + (time(NULL) - temps_temp);
		temps_temp = time(NULL);
 
		if(avance_time){
 
			// Si avance time a une valeur c'est que l'on veut avance le temps de x jours
			launch_save();
			temps_courant = temps_courant + (86400 * avance_time);
			avance_time = 0;
			temps_start = temps_courant;
		}
		else{
 
				// Si la différence de temps entre le temps a l'ouverture
				// ou celui de la derniere sauvegarde est superieur a une journe
				if(	difftime(temps_courant, temps_start) > 86400){
 
					// On lance la sauvegarde
					launch_save();
					temps_start = temps_courant;
				}
		}
	}
 
   (void) p_data;
   return NULL;
}
 
int main(){
 
	// Creation de la task de sauvegarde et de gestion du temps
	pthread_t ta;
 
	// Lancement de la task
   	pthread_create(&ta, NULL, task_a, NULL);
 
	// Redimentionnement de la fenetre
	win_size();
 
	// Initialisation de l ecran pour ncurses
	initscr(); 
 
	// Activation des touches de fonctions clavier
	keypad(stdscr, TRUE);
 
	// Empeche l'impression des touches non desire
	noecho();
 
	// Lancement du programme 
	////////////////////////////////////////////////////////////
 
	intro();
	clear();
 
	// Chargement des donnees en memoire et petite animation
	loading();
 
	// Lancement du menu principale
	main_menu();
 
	// On coupe ncurses proprement
	endwin() ; 
 
	// On lance une petite sauvegarde
	launch_save();
 
	// Desallocation memoire de tout ce qui a ete charger
	free_vol();
	free_avion();
 
	// On quitte
	return(0);
}

Le thread qui gere le temps communique avec le reste du programme par la variable globale avance_time normalement égale à 0 sauf lorsque que l'on veut avancer le temps.

Pourtant la moindre modification de la variable avance_time entraine un bus error (malgré mon problème de compilation j'ai pu tester mon programme en copiant le contenu de tous les fichiers dans un seul)

Si quelqu'un pouvait m'aider à corriger le problème se serait vraiment sympa .

Merci d'avance.

Trivol

[Emmanuel Delahaye]

struct.h -> contient toutes les includes et les definitions des variables globales

Déjà, pas de définition dans un .h.

http://emmanuel-delahaye.developpez....s.htm#globales

Une des nombreuses raisons pour lesquelles il ne faut pas utiliser les globales, c'est que c'est compliqué à mettre en oeuvre. Les threads ont un paramètre fait pour éviter ça. Il faut apprendre à l'utiliser...

http://emmanuel-delahaye.developpez.com/pthreads.htm
et surtout
http://emmanuel-delahaye.developpez....ds.htm#donnees

[Trivol]

Certe je vois que les variables globales ce n'est pas le top mais je n'ai pas le temps de changer tout mon code avant le rendu :/.

Donc je les déclare dans un fichier globals.c mais je ne vois pas ou je doit mettre ce fichier dans ma compilation, casiment toutes mes fonctions se servent des variables globales.

[Emmanuel Delahaye]

Donc je les déclare dans un fichier globals.c mais je ne vois pas ou je doit mettre ce fichier dans ma compilation, casiment toutes mes fonctions se servent des variables globales.

Tu fais un fichier globals.c avec la définition des variables et un fichier globals.h avec la déclaration des variables.

Tu inclues globals.h dans globals.c (vérification de cohérence) et dans tous les .c qui en ont besoin.

C'est pas ça qui est expliqué dans l'article sur les globales dont je t'ai passé le lien ?

[Trivol]

Si effectivement mais je ne l'avais pas compris comme cela. Merci beaucoup pour ton aide, il ne me reste plus qu'a corriger un peu mon makefile pour l'adapter.