Discussion :

[telemarker05]

bonjour,

je fais un prog qui utilise une base sqlite. Le nom de cette base est déterminé (fichier ini) dès l'ouverture (splash dans main.cpp).

Comment rendre cette variable (le nom du fichier contenant la base de données accessible depuis n'importe quel autre partie du code.
ou pour le dire autrement, comment créer une variable globale ?

Merci à vous

[LittleWhite]

Bonjour,

Actuellement, ma technique est ( et aussi pour tout les nombres "magiques" ) de faire un fichier global.hpp et d'y mettre mes variables en passant par le préprocesseur ( certains dirant qu'il faut mieux faire un const .... )
Comme ceci

Code :

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

1
2
 
#define FILENAME "./data/filename.ini"

[telemarker05]

Merci à toi
L'idée est bonne, mais sauf erreur de ma part, cela crée une constante que tu globalises.
Or je souhaite une variable, qui sera initialisée dans le code et non en dur...

[yan]

Bonjour,

Actuellement, ma technique est ( et aussi pour tout les nombres "magiques" ) de faire un fichier global.hpp et d'y mettre mes variables en passant par le préprocesseur ( certains dirant qu'il faut mieux faire un const .... )
Comme ceci

Code :

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

1
2
 
#define FILENAME "./data/filename.ini"

En c++, autant utiliser une variable const :

Code :

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

1
2
 
     const std::string FILENAME = "./data/filename.ini";

au moins c'est type safe.

Pour telemarker05, pourquoi as tu besoin de faire

le nom du fichier contenant la base de données accessible depuis n'importe quel autre partie du code.

Sinon, c'est avec le pattern singleton qu'il faut faire cela. Mais attention, c'est un peu le pattern du diable

[telemarker05]

pour yan
Dans mon main.cpp, je lance un splash, durant lequel je lis mon fichier de conf, je récupère le nom du fichier contenant ma base sqlite.

J'en profite pour la créer si elle est manquante.

Du coup je souhaite transmettre ce nom de fichier à ma mainwindow pour éviter d'avoir à relire le fichier de conf... et pour qu'elle puisse traiter la base de données.

Merci de ton aide.

[LittleWhite]

Donc ça devient du global non constant ... ou alors ... on peut aussi jouer sur d'autre détail. Mais je ne me vois pas faire un singleton pour ça.

Pour revenir à ma suggestion ... le #define, bien géré peut ne poser aucun problème (type safe):

Code :

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

1
2
3
4
 
#define MON_INT 12i;
#define MON_FLOAT 1.2f;
#define MA_CHAINE "fichier.txt"

Je pense que c'est assez type safe, grâce aux indicateurs. De plus contrairement aux const, je ne prend pas de mémoire ( juste de la place dans l'executable ). ( Bon ça c'était l'argument nul ). Mais je pense que les deux sont corrects.

[yan]

[quote=LittleWhite;4775261]Pour revenir à ma suggestion ... le #define, bien géré peut ne poser aucun problème (type safe):[/code]
oui et non
http://www.comeaucomputing.com/techtalk/#definevsconst

Je pense que c'est assez type safe, grâce aux indicateurs

dans ton cas surement, mais pour une personne qui reprendrais ton code...

Après, faut juste savoir le risque que cela comporte.

Pour revenir au problème, pourquoi ne pas faire une fonction dans ton mainwindow qui prend en paramètre le nom du fichier?

[Alp]

J'ai tendance à condamner l'utilisation du Singleton dans ce genre de situations, idem pour les macros ou quoi. On est en OO, donc c'est plutôt soit ta fenêtre principale directement, soit un objet qui sera maintenu par ta fenêtre principale (ma solution de prédilection) qui a la responsabilité, mais pas un truc réparti entre 2343 fichiers, on ne sait où.

[Aurelien.Regat-Barrel]

Note qu'il charge dynamiquement sa donnée depuis un fichier de config donc de toutes façons un #define ne convient pas.

Une solution courante est de mettre ce genre d'info dans ton objet Application (qui dérive de QApplication), qui est déjà un singleton. Tu peux aussi instancier à chaque fois que tu en as besoin un QSettings qui va lire à nouveau la valeur depuis ton fichier ini (je ne pense pas que ce soit pénalisant), ou conserver ton QSettings initial comme donnée membre de ton objet Application par exemple.

[telemarker05]

merci à vous.

Ma solution reste simple :
dans le main.cpp je valide ma base durant l'affichage du splash.

Dans ma mainWindow, je relis mon fichier ini pour trouver le nom du fichier qui contient la base, que je stocke dans un attribut privé de ma classe mainWindow.
Il devient accessible, via un accesseur en lecture, à tout objet instancié dans ma mainWindow.

Merci encore à vous tous