Discussion :

[lcoulon]

Bonjour,

Je suis en train de créer une paire de fichier .c et .h associé et j'ai besoin d'utiliser des fonctions de temps, afin de réaliser une tempo qui n'empêche pas le déroulement du programme.

Tout ce qui suit fonctionne, (testé) à condition que tout soit appelé depuis mon fichier principal : main.c

j'ai déclaré ceci dans entête de mon fichier principal (main.c)

Code :

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

1
2
3
TICK Timeout;
TICK last_TickSample;
TICK current_tickSample;

et que je les utilisent plus tard toujours dans mon fichier main.c , tout va bien :

Code :

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

1
2
3
 
TickInit();                			// Initialize tick manager
Timeout = TICK_SECOND * 2;        // on défini le délai au bout duquel la tempo se terminera

...

Code :

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

1
2
3
OUT1 = 1;
last_TickSample = TickGet();  		// Enregistre le temps actuel		
break;

...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
while(1)     // boucle infinie du programme
 
// Verifie que le delai demandé est atteint, 
//si oui execute le contenu de la boucle :
 
{
current_tickSample; = TickGet();
if (TickGetDiff(current_tickSample, last_TickSample) >= Timeout) 
 
			{
				OUT1 = 0;    		//  Turn off LED
 			}
}

Mon soucis est le suivant :

A présent, je voudrai appeler last_TickSample = TickGet(); ailleurs que dans main.c , j'ai besoin de l'utiliser le fichier .c que je suis en train de créer.

Lorsque last_TickSample = TickGet(); est situé en dehors de mon fichier main.c, le compilateur me dit ceci :

symbol 'last_TickSample' has not been defined
lvalue required

cependant, j'appelle bien tick.h et time.h depuis le fichier .h concerné
j'ai essayé d'ajouter ceci dans mon fichier .h, mais sans succes :

Code :

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

extern TICK last_TickSample;

-> le compilateur répond : Error: syntax error et pointe cette même ligne

Pourriez vous m'aider svp ?
Merci beaucoup;

[DeathMixer]

Bonjour,

Pour avoir accès à tes variables déclarées, il faut les déclarer dans un fichier (*.c ou *.h) séparé en variable 'extern' et inclure ce fichier dans le fichier source que tu es en train de créer.

Mets tes 3 lignes

Code :

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

1
2
3
TICK Timeout;
TICK last_TickSample;
TICK current_tickSample;

dans un fichier séparé, et inclus ce fichier dans l'en-tête de ton fichier *.c et tout devrait fonctionner...

Inutile de dire qu'il faut AUSSI l'inclure dans ton fichier principal (main.c)...
Sinon ton main n'aura plus accès à ces valeurs.

Cdt

[lcoulon]

Faut il mettre ' extern ' devant les 3 lignes commençant par TICK ?

[DeathMixer]

Vu que tu me sembles n'utiliser que la variable "last_TickSample", tu n'as pas besoin de déclarer les autres en 'extern'. Mais dans le doute, mets les toutes en 'extern', on sait jamais. Tu risques d'en avoir besoin...

[lcoulon]

j'ai crée comme tu me l'as conseillé, un nouveau fichier .h, que j'ai appelé (decla_time.h) , il ne contient donc que mes déclarations de temps :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
#ifndef decla_H
#define decla_H
 
 
#include "time/include/time.h"
#include "net/include/tick.h"
 
TICK Timeout;
TICK TickSample;
TICK TickSample;
 
#endif  // decla_H

J'appelle ce fichier decla_time.h depuis mon main.c et aussi dans le fichier .c que je suis en train de créer.


A présent, le compilateur m'indique cette erreur :
'Timeout' has multiple definitions

Est ce parce qu'il est declaré dans mon nouveau fichier decla_time.h et également dans main.c sous la forme :

Code :

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

Timeout = TICK_SECOND * 2;        // on défini le délai au bout duquel la tempo se terminera

[ram-0000]

Ton problème est que dans un fichier .h, tu définis des variables.

La différence que je fais (et que beaucoup de monde fait aussi) entre déclaration et définition, est la suivante :

• une déclaration est informative et dit au compilateur "quelque part, il existe une variable nommée toto et dont le type et tata
• une définition génère du code et occupe de la mémoire.

Dans un .h on ne doit que déclarer les variables, les définitions doivent se faire dans un .c. Si tu définis des variables dans un .h et que 2 sources (ou plus) de incluent le fichier .h, tu auras une erreur lors du link avec "Variable 'machin' has multiple definitions".

Donc pour revenir a ton problème, il te faut faire 2 fichiers, un .h et un .c

Code decla.h :

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
#ifndef decla_H
#define decla_H
 
 
#include "time/include/time.h"
#include "net/include/tick.h"
 
extern TICK Timeout;
extern TICK TickSample;

// a ce propos, c'est normal d'avoir 2 variables nommées TickSample ?
extern TICK TickSample;

 // c'est un exemple, tu n'es pas obligé d'avoir une fonction
extern void ma_fonction(const char *Message);
 
#endif  // decla_H

Code decla.c :

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

1
2
3
4
5
6
7
8
9
10
11
 
 
#include "decla.h"
 
TICK Timeout = 12;
TICK TickSample = 12345;
 
void ma_fonction(const char *Message)
{
   printf("%s", Message);
}

Au fait, ces variables Timeout et TickSample sont des variables globales et les bonnes pratiques (quelque soit le langage) veulent que l'on cesse d'utiliser des variables globales (même si parfois, c'est dur de faire autrement).