Discussion :

[kris1]

salut,
bon voila tout court mon problème j'ai trois fichiers :
irda.h
irda.c
test.c
j'ai deux variables globales déclaré dans irda.h :
int kode ;
int button_pressed ;
quand je compile ces fichiers le message suivant apparait :

Code :

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

1
2
3
4
5
6
[root@emblin remote_control]# arm-linux-gcc *.o irrecord.c test.c -o test -lpthread
/tmp/ccW73Iso.o:(.bss+0x0): multiple definition of `button_pressed'
/tmp/cccMUSim.o:(.bss+0x0): first defined here
/tmp/ccW73Iso.o:(.bss+0x4): multiple definition of `Newkode'
/tmp/cccMUSim.o:(.bss+0x4): first defined here
collect2: ld returned 1 exit status

à noter que j'ai bien vérifié que ces deux variables sont déclarés une seulle fois
merci

[Pierre Maurette]

No comment sur les globales. Juste pour votre problème, avez-vous mis un guard ou un #pragma once à votre .h ?

[kris1]

l'entête de mon fichier est la suivante

Code :

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

1
2
3
4
5
6
#if !defined(__IRRECORD_H__)
#define __IRRECORD_H__
//-----------------------------------------------------------------------------
#if defined(__cplusplus)
extern "c" {
#endif

[hegros]

On peut voir le fichier.c qui les utilise ces var glob

[kris1]

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
#include "irrecord.h"
#include "pthread.h"
pthread_t get_button_pressed ;
 
int main()
{
 
pthread_create(&get_button_pressed, NULL, get_remote_code, NULL);
while(1)
{
	if ( Newkode == 1  )
	{
	Newkode = 0 ;
	printf("la valeur de cccccces%d",button_pressed);
	}
}
}

[diogene]

Si tu definis tes variables globales dans un .h, tu vas avoir des définitions multiples (si ton .h est inclus dans plusieurs .c).

défini tes variables globales dans un .c et, dans le .h correspondant, déclare les en extern

[Emmanuel Delahaye]

<...>#pragma once à votre .h ?

Non standard. Aucun intérêt.

[Pierre Maurette]

<...>#pragma once à votre .h ?
Non standard. Aucun intérêt.

Ça, c'est du lourd... Relisez votre norme (§6.10.6) et épargnez-vous, ainsi qu'à moi, vos lapalissades grincheuses.

[Emmanuel Delahaye]

Ça, c'est du lourd... Relisez votre norme (§6.10.6) et épargnez-vous, ainsi qu'à moi, vos lapalissades grincheuses.

Je ne vois pas le rapport avec une 'LaPalissade'...

En tout cas, il n'y a aucun '#pragma once' dans n1124.

Quand à §6.10.6, oui, #pragma existe depuis C90 (voire avant, je n'en sais rien), et alors ? Soit le paramètre (ici, 'once', non standard C99) est connu et interprété, mais la norme ignore comment (donc comportement indéfini), soit il est inconnu et il est proprement ignoré (ça, c'est un comportement standard spécifié).

En définitive, c'est bien ce que je dis, c'est inutile, ou alors il faut préciser le contexte ce que tu as omis, d'où ma réaction.

[dj.motte]

Salut,

Il me semble que l'on définir des variables globales ( pas trop ) dans un un fichier entête .h à condition qu'elles soient statiques.

Code :

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

1
2
3
4
5
6
7
8
 
#ifndef DATA_H
#define DATA_H
 
static int G_kode ;
static  int G_button_pressed ;
 
#endif

A vérifier...

[Emmanuel Delahaye]

bon voila tout court mon problème j'ai trois fichiers :
irda.h
irda.c
test.c
j'ai deux variables globales déclaré dans irda.h :
int kode ;
int button_pressed ;

Mauvaise conception.
1 - On ne définit pas d'objet ni de fonctions dans un header.
2 - On utilise pas de globales sauf nécessité absolue[1].
3 - Si on y est obligé (à démontrer...), on le fait correctement en mettant un préfixe indiquant que c'est une globale (G_), en plaçant la déclaration dans un header protégé contre les inclusions multiples :

Code :

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

1
2
3
4
5
6
7
8
9
 
/* data.h */
#ifndef DATA_H
#define DATA_H
 
extern int G_kode ;
extern int G_button_pressed ;
 
#endif

et la définition dans un .c :

Code :

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

1
2
3
4
5
6
 
/* data.c */
#include "data.h"
 
int G_kode ;
int G_button_pressed ;

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

-----------------
[1] Je rappelle que les pthreads ont un champ (void*pdata) qui est fait pour passer des adresses de contextes...

http://emmanuel-delahaye.developpez....ds.htm#donnees