Ne pas utiliser les "extern"

Version imprimable

01/07/2014, 18h35
ThomasAU

Ne pas utiliser les "extern"

Bonjour à tous

Je vois à gauche et à droite que les le mot-clé extern est "moche". Ou plutôt que sont emploi l'es. Pourquoi ?
Comment puis-je le remplacer ? Je l'utilise sur des sémaphore entre différente partie de mon code de plus dans un fichier header bien identifier (du genre GlobalVariable.h)
01/07/2014, 19h24
Bktero
L'utilisation d'un correcteur orthographique (de nos jours, intégré à tous les navigateurs internet) est nécessaire ! Merci de faire un effort ;)

Il y a 2 emplois de extern :
1. Devant un prototype de fonction : extern void f(int i);
2. Devant une déclaration de variable globale définie dans une autre unité de compilation : extern int i;
Dans le cas 1, le mot-clé extern est inutile car il est implicitement rajouté si besoin. Cela peut éventuellement servir à dire que c'est bien une fonction définie dans une autre unité de compilation et non simplement le prototype d'une fonction définie plus bas dans le fichier courant.

Dans le cas 2, c'est obligatoire pour utiliser une variable globale qui est définie dans un autre fichier.

Dans les 2, c'est effectivement peu intéressant / moche / contournable :
1. Un .h doit contenir les déclarations des variables et on inclue le .h du module qu'on souhaite utiliser. extern est comme je l'ai dit inutile et l'éventuellement petit avantage cosmétique n'a pas lieu d'être.
2. On évite les variables globales et on passe par des fonctions getters / setters et on retombe dans le cas 1.
Donc oui, souvent, on utilise peu le mot-clé extern. Mais bon, je n'ai personnellement rien contre lui si ce n'est qu'il montre souvent un mauvais découpage en modules comme expliqué ci-avant.

Citation:

(du genre GlobalVariable.h)

*
Ceci est malheureusement le signe d'une mauvaise modularité de ton application. J'avais commencé à écrire un article là-dessus... Il faut vraiment que je le termine pour expliquer pourquoi...
01/07/2014, 20h23
Sve@r
Citation:
Envoyé par Bktero

Il y a 2 emplois de extern :

Devant un prototype de fonction : extern void f(int i);
Devant une déclaration de variable globale définie dans une autre unité de compilation : extern int i;
Bonjour

Moi j'en vois un 3°: pour spécifier à l'intérieur d'un bloc que l'on utilise une variable globale (même définie dans la même unité de compilation).

Exemple
Code:

1 2 3 4 5 6 int var=123; int main() { extern int var; printf("%d\n", var); }
Le gros problème des variables globales, c'est qu'elles sont accessibles/modifiables de partout. Et donc il n'y a rien de pire que d'avoir une globale accidentellement modifiée durant l'exécution. Parce qu'ensuite, pour trouver la fonction qui foire, c'est galère.
Si le programmeur qui a l'intention de taper dans une globale, prend soin de la redéclarer de cette façon dans la fonction qui l'utilise, alors s'il y a un problème la recherche de la fonction incriminée en sera facilitée.

Mais bien entendu, si on peut ne pas avoir de globales/extern c'est encore mieux...

Citation:

Envoyé par ThomasAU

Comment puis-je le remplacer ? Je l'utilise sur des sémaphore entre différente partie de mon code de plus dans un fichier header bien identifier (du genre GlobalVariable.h)

Si une variable n'est utilisée que dans une fonction, alors elle doit être locale à la fonction.
Si une variable doit être utilisée dans plusieurs fonctions, alors tu passes la variable aux fonctions incriminées.
Cela peut t'amener, dans le cadre d'un gros projet, à avoir alors 15/20/50 variables. Dans ce cas tu crées une structure dédiée à la gestion de ton projet (sa configuration en somme) qui contiendra tous les éléments nécessaire à ton projet et que tu remplis dans le main. Ensuite, te suffit de passer la structure (une seule variable) aux fonctions qui en ont l'utilité.

Dans 99% des cas, les globales sont inutiles/évitables. La seule exception que je connaisse est dans le cas des kill/signaux. La fonction utilisée pour trapper un signal ne peut contenir qu'un seul paramètre: la valeur du signal trappé. Si cette fonction doit interagir sur un autre élément du programme, alors cet élément doit-être mis en global...
01/07/2014, 22h25
Bktero

Du coup..... http://gradot.wordpress.com/2014/07/...globales-en-c/

Citation:

cet élément doit-être mis en global...

Pourquoi ne pas passer par des getters et des setters ? ;)

Citation:

Envoyé par Bktero

Pourquoi ne pas passer par des getters et des setters ? ;)

Euh ben parce qu'on n'est pas en C++ quoi...:P

Nan, plus sérieusement, un getter et un setter reste une fonction quoi. Et comment cette fonction pourrait accéder à ma variable qui se trouve, elle, dans une autre fonction ?

Démo
Code:

1 2 3 4 5 6 7 8 9 10 11 int main() { int x=100; signal(SIGUSR1, trap); while (1) sleep(10); } int trap(int sig) { // Je voudrais afficher x lorsque mon prog reçoit un kill -SIGUSR1 depuis l'extérieur. Comment faire ??? Quel getter me permettra ce tour de passe-passe sans mettre x en global ??? }

01/07/2014, 22h58
ThomasAU

Citation:

Envoyé par Sve@r

Dans 99% des cas, les globales sont inutiles/évitables. La seule exception que je connaisse est dans le cas des kill/signaux.

Justement dans le projet que je code, j'utilise une queue (qui est en somme une file de signaux) et un sémaphore qui gère l'accès. J'ai plusieurs machines à état (pour le moment 7) qui interagissent avec plusieurs sous-systèmes. Ils sont indépendants dans des thread gérés indépendamment (par FreeRTOS). Les extern variables sont donc sur ce type de variable pour le moment.

Mon problème, que je tente de régler avec ces externes, est que lorsque j'essaye d'accéder à la queue depuis un autre thread, cela me renvoie sur une autre adresse mémoire.:aie:

Citation:

Envoyé par Sve@r

Si une variable doit être utilisée dans plusieurs fonctions, alors tu passes la variable aux fonctions incriminées.
Cela peut t'amener, dans le cadre d'un gros projet, à avoir alors 15/20/50 variables. Dans ce cas tu crées une structure dédiée à la gestion de ton projet (sa configuration en somme) qui contiendra tous les éléments nécessaire à ton projet et que tu remplis dans le main. Ensuite, te suffit de passer la structure (une seule variable) aux fonctions qui en ont l'utilité.

J'aime bien cette idée, mais il faut que je teste si cela prend plus de mémoire ou non (plus de pointeurs ?).

P.S : Excusez-moi pour ces malencontreuses faute d'orthographe.

Code:

1
2
3
4
5
static int x;
int getX()
{
    return x;
}

02/07/2014, 09h36
Sve@r
Citation:
Envoyé par Bktero

Code:

1 2 3 4 5 static int x; int getX() { return x; }
Ben oui mais tu as mis x en global là !!! Tu disais passer par un getter justement pour ne pas avoir à mettre de global...?:mouarf:

en fait non.

L'exemple plus complet, c'est:

L'en-tête suivant
Code:

1 2 int ageDuCapitaine(); void anniversaireCapitaine();
Le fichier d'implémentation correspondant
Code:

1 2 3 4 5 6 7 static int ageCapitaine=42; int ageDuCapitaine() { return ageCapitaine; } void anniversaireCapitaine(){ ++ageCapitaine; }

02/07/2014, 13h03
transgohan

Citation:

Envoyé par ThomasAU

Mon problème, que je tente de régler avec ces externes, est que lorsque j'essaye d'accéder à la queue depuis un autre thread, cela me renvoie sur une autre adresse mémoire.:aie:

C'est donc que tu as plusieurs déclarations de ta variable au lieu d'en avoir une seule et les autres en extern.

Cela peut arriver aussi il me semble si tu as mis toutes les déclarations en extern.
Certains compilateurs ne vont pas broncher au link mais ne vont rien linker du tout et tu vas te retrouver avec X adresses.
02/07/2014, 15h54
ThomasAU

Le problème est résolut parfaitement avec les déclarations en extern. Pour le moment j'utilise le compilateur gcc et non pas un compilateur exotique (ce que je pense pas faire). Toutefois si je dois faire appel à un tel compilateur qui ne reconnaitrait pas les extern, alors je pense faire une transmission de variable.
02/07/2014, 16h26
Sve@r

Citation:

Envoyé par ThomasAU

Le problème est résolut parfaitement avec les déclarations en extern. Pour le moment j'utilise le compilateur gcc et non pas un compilateur exotique (ce que je pense pas faire). Toutefois si je dois faire appel à un tel compilateur qui ne reconnaitrait pas les extern, alors je pense faire une transmission de variable.

Déjà un compilo C qui ne connaitrait pas les extern ne peut pas exister. Le extern est incontournable de la programmation modulaire. En effet, chaque module doit se compiler séparément mais chaque module doit déclarer les variables qu'il utilise. Or extern sert justement à indiquer qu'une variable déclarée dans un module X existe déjà ailleurs et qu'il ne faut alors pas lui réserver de mémoire.
Si tu as des soucis, c'est soit que ta variable globale est mise en extern de partout (donc en fait elle n'est définie nulle part), soit tu as oublié, dans un module, de la mettre en extern. Dans ce cas elle est définie deux fois et le linkeur gueule à l'éditions de liens.
Pour éviter ce genre de soucis, on définit les variables dans le module qui a le main() et on les déclare en exten dans tous les autres modules.

Mais bien entendu c'est encore mieux de ne jamais utiliser de globales !!!
02/07/2014, 16h32
ternel

Précision pour la culture.

Ces modules sont appelés "unités de compilation" dans la norme.
Chaque module est un des .c initial, une fois préprocessé (donc, entre autre, quand tous les #include ont été faits, récursivement)
02/07/2014, 20h02
Bktero

Citation:

Envoyé par Sve@r

Ben oui mais tu as mis x en global là !!! Tu disais passer par un getter justement pour ne pas avoir à mettre de global...?:mouarf:

Elle n'est pas globale, elle est statique au fichier et donc non visible depuis les autres unités de compilation. Tu ne peux pas y accéder autrement qu'avec le getter (sauf dans l'unité de compilation courante bien sûr).

leternel montre en effet un exemple plus complet.

A ma connaissance, tu ne peux pas réexposer une variable automatique à l'extérieur. Tu es obligé de passer par des variables a linkage externe (comme ma statique ou alors une vraie globale).
03/07/2014, 08h53
Kannagi

J'ouvre une petite parenthèse , je pense que utilisation de getter/setter soit pas le plus approprié en C ,en C++ get/set pour ma part c'est souvent utilisation de variable qui sont en privé et donc qu'on doit passer par les méthodes pour les modifier/lire.
En C il n'y a pas ce souci , les structures ont peut les modifier/lire , pour static je le vois seulement pour des variables temporaires mais qui doivent gardé la même valeurs (pas initialisation a chaque appel de la fonction).
03/07/2014, 09h40
ternel

Ca sert si on veut proposer un singleton, ou un pool.
Avec cette méthode, ta "classe" est automatiquement un singleton, et son instance est l'unité de compilation. (par contre, il faut gérer la destruction).

C'est exactement ce qu'il se passe avec la SDL, par exemple, qui demande un appel à SDL_Init() et SDL_Quit().
03/07/2014, 16h06
Sve@r

Citation:

Envoyé par Bktero

Elle n'est pas globale, elle est statique au fichier et donc non visible depuis les autres unités de compilation. Tu ne peux pas y accéder autrement qu'avec le getter (sauf dans l'unité de compilation courante bien sûr).

Ouais c'est un peut tiré par les cheveux. Bon je suis d'accord, ça limite les risques mais pas pour l'unité de compilation courante dans laquelle elle reste synonyme de globale. C'est mieux que rien quoi. Mais pas de globale/statique c'est encore mieux non ?:aie:

Citation:

Envoyé par leternel

C'est exactement ce qu'il se passe avec la SDL, par exemple, qui demande un appel à SDL_Init() et SDL_Quit().

Bah, c'est un peu le même principe pour tous les outils un peu complexes (malloc/free, fopen/fclose, shmat/shmdt)...
L'initialisation sert à "initialiser" tous les éléments internes de l'outil (qui peuvent eux-même se trouver dans une structure telle que je l'expliquais dans mon premier post) et la fermeture permettra de "libérer" toutes les ressources éventuellement réservées...