Définition multiple

Bonjour les gars ^^
J'ai un petit problème, le compilo me signale une définition multiple je crois que j'ai mal inclus mes fichiers mais ca m'a l'air correcte, tenez regardez:
dans main.c j'ai: include "module.h"
dans functions.c j'ai aussi include "module.h"
et dans le module.h:

Code:

---

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

#ifndef H_MODULE #define H_MODULE   #include <stdio.h> #include <stdlib.h> #include <string.h> #include "functions.h"   #define DONE 0 #define MAX_CHAR 255   typedef struct {   unsigned char car; int occur; }CHAR;   unsigned char Set_Bit[8] = {128, 64, 32, 16, 8, 4, 2, 1}; CHAR CharTable [MAX_CHAR];   #endif

---

dans le functions.h au cas ou:

Code:

---

1 2 3 4 5 6 7

#ifndef H_FUNCTIONS #define H_FUNCTIONS   /*prototypes de mes fonctions*/   #endif

---

et le compilo me rend:

Citation:

---

Envoyé par code::blocks

.objs\functions.o:functions.c:(.data+0x0): multiple definition of `Set_Bit'
.objs\main.o:main.c:(.data+0x0): first defined here
collect2: ld returned 1 exit status
Process terminated with status 1 (0 minutes, 0 seconds)
0 errors, 0 warnings

---

Merci d'avance

Voir le site d'Emmanuel Delahaye: http://emmanuel-delahaye.developpez....s.htm#globales

En générale on initialise pas les variables, tableaux, .... dans les headers, peut-être que l'erreur viens de là (j'ai deja eu le problème) !

Citation:

---

Envoyé par CSoldier

En générale on initialise pas les variables, tableaux, .... dans les headers, peut-être que l'erreur viens de là (j'ai deja eu le problème) !

---

L'erreur vient de là. On déclare les variables/tableaux en extern dans les .h, et on définit les variables/tableaux dans une fonction d'init par exemple.

De toute façon, attention, ces variables/tableaux sont alors des variables globales, à manipuler avec précaution.

Merci beaucoup, problème résolu :) (comme d'hab :p)
Merci merci ^-^

Citation:

---

Envoyé par Goundy

dans main.c j'ai: include "module.h"
dans functions.c j'ai aussi include "module.h"
et dans le module.h:

Code:

---

1 2 3 4 5 6 7 8

#ifndef H_MODULE #define H_MODULE   #include <stdio.h> #include <stdlib.h> #include <string.h> #include "functions.h"

---

---

Erreur de conception : Ces 4 headers ne sont pas utilisés ici. A retirer.

Citation:

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

#define DONE 0 #define MAX_CHAR 255   typedef struct {   unsigned char car; int occur; }CHAR;   unsigned char Set_Bit[8] = {128, 64, 32, 16, 8, 4, 2, 1}; CHAR CharTable [MAX_CHAR];

---

---

Erreur de conception : On ne met pas de définitions d'objet dans un header.

Citation:

---

Code:

---

1 2 3

#endif

---

dans le functions.h au cas ou:

Code:

---

1 2 3 4 5 6 7

#ifndef H_FUNCTIONS #define H_FUNCTIONS   /*prototypes de mes fonctions*/   #endif

---

et le compilo me rend:

Citation:

---

Envoyé par code::blocks

.objs\functions.o:functions.c:(.data+0x0): multiple definition of `Set_Bit'
.objs\main.o:main.c:(.data+0x0): first defined here
collect2: ld returned 1 exit status
Process terminated with status 1 (0 minutes, 0 seconds)
0 errors, 0 warnings

---

---

Ben oui. Si tu tiens à avoir des variables globales (qui ferait ça...) il faut suivre les regles de conception...

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

Citation:

---

Envoyé par CSoldier

En générale on initialise pas les variables, tableaux, .... dans les headers, peut-être que l'erreur viens de là (j'ai deja eu le problème) !

---

Qu'on initialise ou non n'est pas la question. On ne définit pas d'objet dans un header. C'est tout.

Merci Emm, c'est noté :)