Discussion :

[scorbo]

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 :

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

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

[D[r]eadLock]

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

[Anomaly]

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

Code :

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

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

[scorbo]

voilà le fichier .h

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

[D[r]eadLock]

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 :

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

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

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 :

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

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 :

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

[sylvain114d]

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

Code :

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

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