[débutant]Organisation des déclarations de variables

Bonjour,

J'ai un problème d'organisation de déclaration de variable.
J'ai créé une bibliothèque (.h) dans laquelle j'ai mis quelles fonctions et également la déclaration de structures avec un typedef pour chacune des structures.
J'utilise ces structures dans mon pgm principal comme variables globales.

Durant la compilation, j'ai un message signifiant que ces variables ( ex. LPAR[nb].Oid_value) n'e sont pas déclarées dans une fonction contenue dans le .h

mon.h
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 typedef struct IBM_LPAR_MIB IBM_LPAR_MIB; struct IBM_LPAR_MIB { char Oid_value[30]; char Lpar_name[12]; char Cpc_name[12]; }; int GetMibLpar( char *Cmdsnmp) { ~~ while(pword !=NULL) { if (a==0) { strcpy(LPAR[nb].Oid_value,(pword+27)); a=a+1; } else { ~~ } }
Le main
Code:

1 2 3 4 5 6 7 8 9 10 #include "mon.h" ~~ ~~ IBM_LPAR_MIB LPAR[300]; int main(int argc, char *argv[]) { ~~ return 0; }
Ma question: dois je placer mes déclarations de variables globales avant l'include de mon.h, ou existe t'il une autre méthode plus approriée ?

Du code dans un [dot]h ....

declare tes globales en extern dans ton [dot]h

ton [dot]h

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
 
#ifndef __TON_H__
#define __TON_H__
typedef struct  s_ta_struct
{
int foo;
char bar;
}t_ta_struct;
 
extern t_ta_struct ta_struct;
 
#endif /*__!TON_H__ */

ton [dot]c

Code:

1
2
3
4
5
6
7
8
9
10
 
 
#include "ton.h"
t_ta_struct ta_struct;
 
int main(int ac, char **av)
{
...;
return (0);
}

Citation:

Envoyé par SPACHFR

Ma question: dois je placer mes déclarations de variables globales avant l'include de mon.h, ou existe t'il une autre méthode plus approriée ?

Il y a quelques règles simples à respecter pour organiser son code. Ca commence par écrire des .h corrects :

Protection contre les inclusions multiples
Uniquement des prototypes de fonctions et ce qu'il faut pour les définir (includes, macros, constantes, types)
pas de code, pas de variables

Ensuite, se pose la pertinence des variables globales. C'est rarement le cas et la programmation par contexte (structure, pointeurs sur structures) permet d'écrire du code plus fiable et plus lisible.

A lire de A à Z... : http://emmanuel-delahaye.developpez.com/codage.htm

Si néanmoins, tu tiens aux globales, fait le correctement :

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

Merci pour vos réponses.
Je n'avais pas mis la partie de code contre les inclusions multiples dans l'exemple de mon message, mais c'est codé dans mon fichier .h (d'ailleurs pas très bien au vue de vos exemples....
Code:

1 2 3 4 5 #if !defined (ZCMNG_H) #define ZCMNG_H #endif ~~
Je vais corriger est prendre la régle donnée dans : http://emmanuel-delahaye.developpez.com/codage.htm.

Pour les variables globales, j'ai pas trouvé d'autre solution simple( je fais de l'auto-formation ...seul dans mon coin).
Si j'ai bien compris les propos concernant l'organisation des variables globales
je fais un fichier de déclaration (.h)
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 /* H_SP_MIB_20071203.h */ #if !defined (H_SP_MIB_20071203) #define H_SP_MIB_20071203 typedef struct IBM_LPAR_MIB IBM_LPAR_MIB; struct IBM_LPAR_MIB { char Oid_value[30]; char Lpar_name[12]; char Cpc_name[12]; }; extern IBM_LPAR_MIB LPAR[300]; #endif
Un fichier .c
Code:

1 2 3 4 /* H_SP_MIB_20071203.c */ #include "H_SP_MIB_2071203" IBM_LPAR_MIB LPAR[300];
pgm principal .c
Code:

1 2 3 4 5 6 7 8 9 10 #include "H_SP_MIB_20071203.h" int main(void) { int a=0; char cmd[50]=""; ~~ a =GetMibLpar(cmd); ~~ return 0; }
Cependant j'ai toujours un doute concernant la place de ma fonction. Vue les régles données (pas de code, pas de variable dans les .h) ou dois je placer ma fonction ?
Code:

1 2 3 4 5 6 7 8 9 10 11 int GetMibLpar( char *Cmdsnmp) { ~~ while(pword !=NULL) { if (a==0) { strcpy(LPAR[nb].Oid_value,(pword+27)); a=a+1; } else { ~~ } }
doit elle être dans le H_SP_MIB_20071203.c, et le prototype de celle-ci dois t'il être dans le pgm principal ?
Désolé Emmanuel, j'ai relu plusieurs fois mais j'ai pas trouvé (probablement pas compris) où déclarer ma fonction.

Citation:

Envoyé par SPACHFR

Pour les variables globales, j'ai pas trouvé d'autre solution simple( je fais de l'auto-formation ...seul dans mon coin).
Si j'ai bien compris les propos concernant l'organisation des variables globales
je fais un fichier de déclaration (.h)

:!: Astuce : Dans le fichier .c, tu peux omettre la taille du tableau :

Code:

IBM_LPAR_MIB LPAR[];

car elle est déclarée dans le .h. Il est toujours plus facile de maintenir du code dont les définitions sont uniques (une seule fois '300' et non 2 fois...)

Citation:

Cependant j'ai toujours un doute concernant la place de ma fonction. Vue les régles données (pas de code, pas de variable dans les .h) ou dois je placer ma fonction ?

A l'évidence dans un .c

Citation:

et le prototype de celle-ci dois t'il être dans le pgm principal ?

il doit être défini dans un .h, inclus dans le .c qui définit la fonction et dans les .c qui l'utilisent.

Tout cela n'est pas du hasard ou un dogme quelconque. C'est du simple bon sens... "Tout ce qui est utilisé doit avoir été déclaré au préalable".