multiple definition

Hi,

J'ai un petit problème:
j'utilise dev-c++ 4.9.8.0, j'ai créer un projet qui inclus 3 fichiers c et un fichier h. Dans chaque fichier c j'ai inclue le fichier h. Au tout début de ce dernier j'ai mis:

Code:

---

1 2 3 4 5

#ifndef _PROG_H #define _PROG_H 1 .... .... #endif

---

Le problème c'est que lors de la compilation il écrit "multiple definition of"...

a priori ca a l'air bon, mais :
- c'est quoi le message en entier ? (copier/coller apprecie :wink:)
- tu ne le definis (_PROG_H) pas ailleurs par hasard ?
- t'es sur que c'est bien PROG_H qui est "multiple definition..." ?

Ton message d'erreur n'est pas normal, cependant, pour éviter une multi-inclusion, on ne définit pas de valeur au #define :

Code:

---

1 2 3 4 5

#ifndef _PROG_H #define _PROG_H .... .... #endif

---

Cela t'évitera le message d'erreur, mais cela ne résoudra pas le problème : si le problème est sur _PROG_H, c'est qu'il n'est pas unique dans ton programme (et donc ton .h risque de ne pas être compilé).

voilà le fichier .h

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

#ifndef _PROG_H #define _PROG_H 1   #include <stdio.h> #include <stdlib.h> #include <gtk-2.0/gtk/gtk.h>   #define W_DAREA 300 #define H_DAREA 300   GtkWidget* window; GtkWidget *darea; /* drawing area */ GdkPixmap *darea_pxm; /* backing pixmap for darea */ GdkGC *darea_gc; /* attached Graphics Context */ GdkPixmap *brique1_xpm;       void key_press_event (GtkWidget *widget, GdkEventKey *event); int expose_event(GtkWidget *widget, GdkEventExpose *event); void load_pixmaps(void); void init_var(void);   int level = 0;   #endif

---

- La variable _PROG_H n'est définit que dans le fichier .h
- Les messages d'erreurs sont:

multiple definition of 'level'
multiple definition of 'window'
multiple definition of 'darea'

Et ainsi de suite pour toutes les variables déclarées dans le fichier .h

Dans ton .h, il faut definir les variables comme extern (extern int i;). Et il faut definir dans un (et un seul) des .c ces memes variables.

Explications:
le .h est normalement uniquement la pour definir des types/variables, mais pas pour les "implementer". Or toi, avec cette definition (sans extern), lorsque tu compiles tes 3 .c independamment, tu ajoute dans les .o associes les variables du .h
Le include n'est qu'une recopie d'un fichier. Le #ifndef n'est la que pour eviter les inclusions recursives (lors de la compilation d'un seul .c), pas pour gerer les variables "globales".

Exemple (mauvais):

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

<commun.h> #ifndef __COMMUN_H__ #define __COMMUN_H__ int glob_var; #endif   <toto.c> #include "commun.h" int main(void){ tata(3); printf("glob_var=%d\n",glob_var); return 0; }   <tata.c> #include "commun.h"   void tata(int i){ glob_var=i; return; }

---

Lors de la compilation de toto.c, le compilo voit ca (apres le pre-processeur (gcc -E)) :

Code:

---

1 2 3 4 5 6 7 8

int glob_var;   int main(void){ tata(3); printf("glob_var=%d\n",glob_var); return 0;   }

---

Et lorsqu'il compile tata.c,

Code:

---

1 2 3 4 5 6

int glob_var;   void tata(int i){ glob_var=i; return; }

---

Et donc lors de l'edition des liens (mettre ensemble les deux .o generes), la variable glob_var est definie 2 fois.

Exemple (correct):

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22

<commun.h> #ifndef __COMMUN_H__ #define __COMMUN_H__ extern int glob_var; #endif   <toto.c> #include "commun.h" int glob_var; /* glob_var sera dans ce .o */ int main(void){ tata(3); printf("glob_var=%d\n",glob_var); return 0; }   <tata.c> #include "commun.h"   void tata(int i){ glob_var=i; return; }

---

Dans ce cas, la variable glob_var sera dans le code de toto.o (espace reserve pour cette variable).

Ce que dit D[r]eadLock est vrai en C++.
En C c'est un peu plus compliqué, il existe des definitions potentielles:

Code:

---

1 2 3 4

int x; /* definitions potentiel */ int x = 0; /* definition */ extern int x; /* declaration */

---

Une variable peu etre declaré autant de fois que l'on veut mais doit etre definie qu'une seule fois.
Il peu y avoir plusieurs defintions potentielles. L'editeur de lien prend alors la premiere qu'il rencontre comme definition.

Je vois donc que tu compile avec un compilateur C++ uniquement ou que tu as nomé tes fichiers *.cpp car sinon tu ne devrai avoir qu'une seule erreur :
multiple definition of 'level'
car
int level =0; est la seul vraie definition.

GtkWidget* window;
GtkWidget *darea;
sont de definitions potentielles