Discussion :

[Unnamed_]

Bonsoir à tous
tout est dans le titre je pense
j'ai un souci avec un namespace qui dans l'une des fonctions modifie une des variables.
Secret (cliquez pour afficher)

Code truc.hpp :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
 
// ... avant le nspace
 
namespace untruc {
  void modifvar () ;
 
  // ... du code ...
 
  int variable ;
}
 
// ... après le namespace ...

Code truc.cpp :

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

1
2
3
4
5
6
7
8
9
10
11
 
 
// toujours autant de code ...
 
namespace untruc {
  void modifvar () {
    untruc::variable = 4444 ; //ou une valeur x
  }
}
 
// ... la suite

Code main.cpp :

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

1
2
3
4
5
6
7
8
#include "untruc.hpp"
#include <iostream>
 
int main (void) {
  untruc::modifvar () ;
  std::cout << untruc::variable << std::endl ;
  return EXIT_SUCCESS ;
}

ça me créé bel et bien les fichiers (*.o) mais lors de l'édition des liens gcc me sort (je suis sous linux) :

Code console :

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

1
2
untruc.o:(.bss+0x0): multiple definition of `untruc::variable'
main.o:(.bss+0x0): first defined here

merci d'avance pour vos réponses

[Lavock]

Ta variable est clairement défini dans deux unité de compilation. Les variables globales doivent être déclarer une fois, puis récupérer grâce à externe... Mais il y a des approche plus c++(singleton par exemple).

[Unnamed_]

Ta variable est clairement défini dans deux unité de compilation. Les variables globales doivent être déclarer une fois, puis récupérer grâce à externe... Mais il y a des approche plus c++(singleton par exemple).

Thanx, je vais voir coté singleton
merci encore

[3DArchi]

Bonjour,
Il est inutile de coller du singleton à toutes les sauces.
Si tu as besoin d'une variable globale, alors utilises une variable globale et non un singleton en te disant que tu fais de l'objet ou du C++.
Lorsque tu écris

Code :

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

type ma_variable;

dans un .h, ma_variable est définie autant de fois que le .h est inclus. Syntaxiquement une des solutions est effectivement d'écrire :

Code :

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

extern type ma_variable;

dans le .h pour déclarer la variable. Ensuite il faut la définir une seule fois dans un seul .cpp :

Code :

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

type ma_variable;

Mais, la vraie question est pourquoi tu as une variable globale ? Les variables globales peuvent traduire des erreurs de conception.

[Alp]

Dans 90% des situations je bannis directement le singleton. Je préfère avoir une classe qui va en gérer d'autres, et qui contiendra ce que tu voulais mettre en singleton. Imaginons que j'ai un socket dont je me sers dans plusieurs points du programme. Bah je préfèrerais le mettre dans ma fenêtre principale par exemple, et le donner dans le constructeur des dialogues et autres fenêtres de mon programme afin qu'ils puissent s'en servir.

Bref, en règle générale, c'est tout simplement une flemme qui mène au singleton, et non pas les bonnes pratiques de design