Discussion :

[Goundy]

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 :

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

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

1
2
3
4
5
6
7
 
#ifndef H_FUNCTIONS
#define H_FUNCTIONS
 
/*prototypes de mes fonctions*/
 
#endif

et le compilo me rend:

.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

[DaZumba]

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

[Franck.H]

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

[Trap D]

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.

[Goundy]

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

[Emmanuel Delahaye]

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

Code :

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

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.

Code :

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

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.

Code :

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

1
2
3
 
 
#endif

dans le functions.h au cas ou:

Code :

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

1
2
3
4
5
6
7
 
#ifndef H_FUNCTIONS
#define H_FUNCTIONS
 
/*prototypes de mes fonctions*/
 
#endif

et le compilo me rend:

.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

[Emmanuel Delahaye]

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.

[Goundy]

Merci Emm, c'est noté