Discussion :

[karen8304]

Bonjour,

J'ai un morceau de code que j'aimerais compléter sauf que je ne comprend pas comment et à quel endroit je peux initialiser mes variables.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
 
#ifndef VARIABLE_H
#define VARIABLE_H
 
#ifdef BUILD_DLL_VARIABLES
	#define DllExport _declspec(dllexport)
#else
	#define DllExport _declspec(dllimport)
#endif
 
 
 
struct DllExport Globals
{
	int change_view;
	static Globals & instance();
 
private:
 
	Globals()
		: change_view(0)
	{}
};
 
#endif

Code :

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

1
2
3
4
5
6
7
8
9
 
#include "stdafx.h"
#include "globals.h"
 
Globals & Globals::instance()
{
	static Globals inst;
	return inst;
}

A quel endroit je peux définir la valeur de change_view à 5 par exemple ? Au début je pensais que c'était lorsque j'écrivais change_view(0) que j'initialisais ma variable mais ce n'est pas le cas.

Merci pour votre aide.

Karen

[ternel]

en fait, si, c'est bien le cas, mais il faudra recompiler tous les programmes qui incluent cet en-tête.
C'est probablement pour ca que tu voudrais que la définition du constructeur soit dans le fichier d'implémentation.

[karen8304]

Merci pour votre réponse

[karen8304]

Bonjour,

J'ai une nouvelle question sur la même portion de code.

Je souhaiterais déclarer ces nouvelles variables :

Code :

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

1
2
3
4
 
static int Idx_integ =0;
int Tinteg[6] = {1000,2000,3000,4000,7000,10000};	
int TI_band1_2 = Tinteg[Idx_integ];

Déjà est-ce possible de le faire (un tableau peut-il être déclaré dans ce genre de structure?) ? Et si oui pouvez-vous m'indiquer comment je dois procéder s'il vous plait?

Je vous remercie par avance.

Karen.

[ternel]

J'imagine que ce doit être des variables de la structure Globals, n'est-ce pas?

Si c'est bien le cas, ce sont seulement des variables membres, que tu devras initialiser dans le constructeur.
Tout pouvant être membre d'une classe.

Comment t'y prendrais-tu?

[karen8304]

Oui il s'agit bien de variables de la structure Globals.

Je ne comprend pas bien ce que vous voulez dire lorsque vous écrivez :

Si c'est bien le cas, ce sont seulement des variables membres, que tu devras initialiser dans le constructeur.
Tout pouvant être membre d'une classe.

Le tableau que je veux initialiser devrait être une variable membre ou je me trompe?

[ternel]

Bonne question. D'après toi?

[karen8304]

Je dirais oui ça doit être une variable membre.

[ternel]

Donc, vas-y, écrit ces membres, et montre nous comment tu le fais.
Comme ca, on pourra t'aider à comprendre.

Par contre, nous ne te donnerons pas la réponse toute faite (en vertu de l'article IV-N de notre charte).
C'est pour cela qu'il est important que tu essaies toi-même.

[karen8304]

Voici le code que j'ai essayé mais j'ai des erreurs :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
 
#ifndef VARIABLE_H
#define VARIABLE_H
 
#ifdef BUILD_DLL_VARIABLES
	#define DllExport _declspec(dllexport)
#else
	#define DllExport _declspec(dllimport)
#endif
 
 
 
struct DllExport Globals
{
        static int Idx_integ;
        int Tinteg[6];	
        int TI_band1_2;
 
	static Globals & instance();
 
private:
 
	Globals()
		:  Idx_integ (0), Tinteg(1000,2000,3000,4000,7000,10000), TI_band1_2 (Tinteg[Idx_integ])
	{}
};
 
#endif

Cette erreur apparaît : impossible de spécifier un initialiseur explicite pour les tableaux .
J'ai donc essayer de déclarer mon tableau dans le fichier .cpp en ayant juste cela dans le .h

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
 
#ifndef VARIABLE_H
#define VARIABLE_H
 
#ifdef BUILD_DLL_VARIABLES
	#define DllExport _declspec(dllexport)
#else
	#define DllExport _declspec(dllimport)
#endif
 
 
 
struct DllExport Globals
{
        static int Idx_integ;
        int Tinteg[6];	
        int TI_band1_2;
 
	static Globals & instance();
 
private:
 
	Globals()
		:  Idx_integ (0), Tinteg(), TI_band1_2 (Tinteg[Idx_integ])
	{}
};
 
#endif

Cependant j'ai cette fois cela qui apparaît : les éléments du tableau 'Globals::Tinteg' vont être initialisés par défaut, j'en déduis qu'il faut que je l'initialise dans le .h. J'ai également essayé en mettant des crochets au lieu des parenthèses entre les différentes valeurs mais ce n'est toujours pas correct.

[ternel]

En fait, tu ne peux pas initialiser un tableau membre autrement qu'avec la valeur par défaut dans la liste d'initialisation.

Tu peux ruser un peu:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
struct DllExport Globals
{
public:
        static int Idx_integ;
 
        int Tinteg[6];	
        int const& TI_band1_2;
 
	static Globals & instance();
 
private:
	Globals() :  TI_band1_2 (Tinteg[Idx_integ]) {
		Tinteg[0]=1000;
		Tinteg[1]=2000;
		Tinteg[2]=3000;
		Tinteg[3]=4000;
		Tinteg[4]=7000;
		Tinteg[5]=10000;
	}
};

et dans le fichier d'implémentation:

Code :

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

int Globals::Idx_integ = 0;

En effet, avec static, la variable Idx_integ n'est membre d'aucun objet de la classe, donc pas à initier dans le constructeur.
Cette définition doit être unique dans le programme, donc non présente dans l'en-tête.

J'ai transformé TI_band1_2 est une référence constante vers la première case du tableau. ainsi, elle gardera systématiquement sa valeur.


Mais je devine que tu veux pouvoir changer cette valeur.


Du coup, tu aurais beaucoup plus intérêt d'avoir une fonction membre.
J'enlève ce fameux static devant Idx_integ, il n'a pas de sens ici.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
 
struct DllExport Globals {
public:
        typedef int Idx_type;
        typedef int values_type;
 
        Idx_type Idx_integ() const {return Idx_integ_value;}
        bool Idx_integ(Idx_type i) {
                if (i <=0 || i>=6) return false;
                Idx_integ_value = i;
                return true;
        }
 
        values_type Tinteg[6];
 
        values_type const& TI_band1_2() const {return Tinteg[Idx_integ];}
 
	static Globals & instance();
private:
        Idx_type Idx_integ_value;
private:
	Globals() : Idx_integ(0) {
		Tinteg[0]=1000;
		Tinteg[1]=2000;
		Tinteg[2]=3000;
		Tinteg[3]=4000;
		Tinteg[4]=7000;
		Tinteg[5]=10000;
	}
};

Je n'aime pas laisser la main sur le tableau Tinteg, je pense qu'une telle variable membre devrait être privée ou constante.

Petite remarque au passage: pour des raisons de lisibilité, si tu utilises un private: ou public:, je te recommande de spécifier le premier, même s'il est implicite.

[karen8304]

Merci beaucoup pour vos réponses et vos conseils, je vais essayer le code de suite