Discussion :

[edgarjacobs]

Bonjour / bonsoir à tous,

J'ai un petit problème dont je n'arrive pas à me dépatouiller. Je vous explique.
J'ai plusieurs fichiers .c qui ont besoin (sans la modifier, c'est pour de l'info) d'une même variable: dayname.
Je souhaiterais définir celle-ci dans un .h (disons dayname.h) et l'y initialiser, puis inclure ce fichier .h dans les .c:

Code :

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

const char *dayname={"Dimanche","Lundi","Mardi","Mercredi","Jeudi","Vendredi","Samedi"};

Et mon linker râle (warning seulement) en me disant que

Code :

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

Warning: Public symbol '_dayname' defined in both module A.OBJ and B.OBJ

pour chaque .c dans lequel j'inclus le .h
Et j'ai tenté de mettre un extern devant la variable dans le .h, et de faire dans un module #define extern, mais rien n'y fait.

Quelqu'un aurait-il une petite idée pour résoudre ce problème ?

D'avance merci,

Edgar.

PS: j'ai bien sur pensé à déclarer la variable dans le .h en tant que char dayname[7][9], et de faire un strcpy() des valeurs dans le .c d'initialisation, mais je ne trouve pas ça particulièrement élégant.... mais si c'est la seule solution, il faudra bien y arriver !

[foetus]

C'est un tableau, et non pas une seule chaîne de caractères

Je te propose cela
dayname.h:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
#ifndef DAYNAME_H
#define DAYNAME_H
 
extern const char* DAYNAME[7];
 
 
namespace globals {
    extern const char* dayname[7];
};
 
#endif // DAYNAME_H

dayname.cpp:

Code :

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

1
2
3
4
5
6
const char* DAYNAME[7] = {"Dimanche", "Lundi", "Mardi", "Mercredi", "Jeudi", "Vendredi", "Samedi"};
 
 
namespace globals {
    const char* dayname[7] = {"Dimanche", "Lundi", "Mardi", "Mercredi", "Jeudi", "Vendredi", "Samedi"};
}

ou alors

Code :

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

1
2
3
4
5
#include "dayname.h"
 
const char* DAYNAME[7] = {"Dimanche", "Lundi", "Mardi", "Mercredi", "Jeudi", "Vendredi", "Samedi"};
 
const char* globals::dayname[7] = {"Dimanche", "Lundi", "Mardi", "Mercredi", "Jeudi", "Vendredi", "Samedi"};

La taille 7 n'est pas obligatoire

Ensuite l'utilisation:

• Faire des #include "dayname.h" dans les fichiers qui en ont besoin
• Construire le module dayname.o ainsi g++ -c dayname.cpp. Et linker tes sources avec ce module. Par exemple: g++ main.cpp dayname.o -o main.
• Directement linker tes sources avec le cpp. Par exemple: g++ main.cpp dayname.cpp -o main.

[Bktero]

@foetus : nous sommes sur le forum C, une solution en C aurait été plus adaptée...

@edgarjacobs : il s'agit ici de bêtement déclarer une variable globale. Comme ferais-tu pour un entier ? Il faut faire la même chose avec un tableau de chaines, à savoir :

• déclaration externe dans un fichier h
• déclaration + définition dans un fichier c
• utilisation dans d'autres fichiers en incluant le fichier h qui déclare (de manière externe) le tableau

[Sve@r]

Bonjour

Tu crées un "dayname.h" contenant

Code c :

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

1
2
3
4
5
6
#ifndef DAYNAME_H
#define DAYNAME_H
 
extern const char* dayname[];
void foo();			// Exemple
#endif // DAYNAME_H

Jusque là, foetus avait bon.

Ensuite, dans ton source principal (celui qui a le main)

Code c :

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

1
2
3
4
5
6
7
8
#include <stdio.h>
#include "dayname.h"
const char* dayname[]={"Dimanche","Lundi","Mardi","Mercredi","Jeudi","Vendredi","Samedi", NULL};
 
int main()
{
	foo();
}

Et dans les autres sources

Code c :

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

1
2
3
4
5
6
7
8
9
10
#include <stdio.h>
#include "dayname.h"
 
void foo()
{
	int i;
	const char **pt;
	for (pt=dayname, i=1; *pt; pt++, i++)
		printf("jour %d: %s\n", i, *pt);
}

L'include de dayname.h dans le main est facultatif mais ça permet de garder une homogénéité dans le code (ça dépend aussi de l'endroit où seront déclarées les fonctions car dans mon premier essai je déclarais foo() dans le main donc c'est selon ton choix). Et le rajout d'un NULL dans le tableau ne va pas te faire perdre beaucoup de place (juste une adresse) mais t'affranchira de devoir te coltiner la valeur "7" à chaque fois que tu devras traiter ton tableau.

[foetus]

@foetus : nous sommes sur le forum C, une solution en C aurait été plus adaptée...

Désolé, j'ai fait cela en C++

Mais, il ne faut que supprimer le namespace.

Jusque là, foetus avait bon.

Effectivement, s'il n'y a pas beaucoup de choses dans le fichier source dayname.c[pp] on peut le fusionner avec le main

Et le rajout d'un NULL dans le tableau ne va pas te faire perdre beaucoup de place (juste une adresse) mais t'affranchira de devoir te coltiner la valeur "7" à chaque fois que tu devras traiter ton tableau.

Soit la sentinelle, soit l'énumération avec une valeur terminale "NB_DAYS"

[Bktero]

Si la déclaration externe inclue le fameux 7, on doit pouvoir faire un sizeof pour connaitre le nombre d'éléments.

J'aime bien le concept du marqueur de fin également, qui sera peut-être plus pratique à l'usage.

[Daïmanu]

Une autre solution serait d'emprunter la notion d'encapsulation des langages objets.

On déclare le tableau static et on y accède à travers une ou plusieurs fonctions :

Code C :

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

1
2
3
4
5
6
7
8
9
10
11
12
//dayname.h
#ifndef DAYNAME_H
#define DAYNAME_H
 
static const char *dayname={"Dimanche","Lundi","Mardi","Mercredi","Jeudi","Vendredi","Samedi"};
 
const char *get_dayname(unsigned int num) {
	return dayname[num % 7]; //0 = dimanche
	//return dayname[(num + 1) % 7]; //0 = lundi
}
 
#endif //DAYNAME_H

Les avantages sont :
- on évite la lecture hors du tableau (avec un modulo) ;
- on peut facilement décider quel est le premier jour de la semaine ;

[Sve@r]

On déclare le tableau static et on y accède à travers une ou plusieurs fonctions :

Oui mais chaque source qui inclus le ".h" aura son propre dayname local ; identique aux autres daynames certes mais c'est bête (choquant ?) de définir x fois le même contenu. Surtout si (évolution) on se met à vouloir ensuite le modifier dans une fonction f1() pour le traiter dans une autre fonction f2()...

Sinon perso je préfère la sentinelle au define. Déjà ça évite de devoir se creuser la tête pour trouver un nom de macro (NB_DAYS, NB_DAYNAME, SIZE_DAYNAME, SZ_DAYNAME, DAYNAMELEN, DAYNAME_LEN, etc...). Surtout si on est limite maniaque et que l'on veut absolument que les noms des macros aient tous une certaine homogénéité les uns avec les autres (ça agace quand on développe à plusieurs de voir d'un cote NB_MONTH et de l'autre DAYNAME_LEN)...
En plus, la macro crée un lien logique entre deux éléments (le tableau et la macro contenant sa taille) et tout relecteur du code (autre que son créateur) devra alors lui-aussi créer ce lien dans son esprit pour comprendre le code (et même s'il met 1/2 seconde pour établir le lien, c'est toujours ça de perdu).
Alors que la sentinelle est directement intégrée au tableau et je trouve ça plus simple à relire.

Et pour sizeof je le trouve trop dangereux à utiliser. sizeof(dayname) renverra certes 7 mais char **pt=dayname; sizeof(pt) renverra 4 (voire 2)...

[edgarjacobs]

Hello,

A la lecture des posts de Bktero et Sve@r, je me dis qu'il y a parfois des moments où je dois être aussi buté qu'un âne pour m'être acharné à modifier MA solution pour qu'elle fonctionne.... alors qu'en laissant reposer les choses quelques heures et en reprenant à tête reposée, j'aurais sans doute trouvé la solution (en fait, je l'avais, mais je déclarais dayname dans le main, et non en gobale....)

Merci à tous pour vos réponses.,

Edgar.

[Bktero]

Une autre solution serait d'emprunter la notion d'encapsulation des langages objets.

On déclare le tableau static et on y accède à travers une ou plusieurs fonctions :

Code C :

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

1
2
3
4
5
6
7
8
9
10
11
12
//dayname.h
#ifndef DAYNAME_H
#define DAYNAME_H
 
static const char *dayname={"Dimanche","Lundi","Mardi","Mercredi","Jeudi","Vendredi","Samedi"};
 
const char *get_dayname(unsigned int num) {
	return dayname[num % 7]; //0 = dimanche
	//return dayname[(num + 1) % 7]; //0 = lundi
}
 
#endif //DAYNAME_H

Les avantages sont :
- on évite la lecture hors du tableau (avec un modulo) ;
- on peut facilement décider quel est le premier jour de la semaine ;

Tu voulais savoir doute faire ça dans un .c et faire un .h pour donner connaissance de la fonction ?

Sinon comme dit Sve@r, tu dupliques la variable statique et tu vas surtout te taper des références multiples sur la fonction.

@Sve@r :

sizeof(dayname) renverra certes 7

Plutôt 7 * sizeof(un pointeur).

[Daïmanu]

Tu voulais savoir doute faire ça dans un .c et faire un .h pour donner connaissance de la fonction ?

Sinon comme dit Sve@r, tu dupliques la variable statique et tu vas surtout te taper des références multiples sur la fonction.

En effet, mon C est un peu rouillé

C'était juste histoire de montrer une autre approche.