Discussion :

[poukill]

Avoir un accès non contant vers une donnée privée, cela veut un peu dire que la donnée n'a pas à être privée.

Il m'arrive de faire de telles choses. Mais je passe avant du temps à voir si les accès que je veux réaliser ne correspondent pas en fait à un nombre restreint de fonctions qui devraient être au niveau de l'objet directement englobant, ou si la donnée doit vraiment être stockée à cet endroit.
Bref, je n'ai pas de recette miracle.

Les recettes miracles, c'est quand on débute... après ça se gâte!
Effectivement, c'est un peu comme si l'attribut était public... Je vais voir s'il n'y a pas moyen de placer des fonctions qui pourraient être au niveau de Choc, mais je n'y crois pas.
CFP est une classe très importante. Les manipulations doivent être faite d'ici je pense!

Merci à toi...

[poukill]

Peut-on instancier une classe (hors liste d'initialisation) dans une fonction membre si cette classe ne fait pas parti des attributs de cette classe?

J'aimerai écrire:

Code :

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

1
2
3
4
5
 
void CFP::lancerWarping()
{
<div style="margin-left:40px">warping =new Warping (image_IR_ref, image_visible);
// bla bla bla</div>}

où Warping est une classe "non membre" de CFP?

[Matthieu Brucher]

Ben oui, pourquoi pas ? Tu fais bien ça tout le temps avec des int, non ?

[poukill]

Je suis d'accord!
Mais tu m'avais dit que l'instanciation de classe devait se faire dans la liste l'initialisation :

Code :

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

1
2
3
4
5
 
void CFP::CFP() : warping()
{
     std::cout << "nouvel objet cree";
}

C'est valable que pour un objet membre?

[Matthieu Brucher]

Oui, un objet qui n'appartient pas à la classe ne peut pas se faire initialiser à la construction de la classe puisqu'elle n'en a pas connaissance.

[poukill]

Est-ce que ce code est correct ?

Code :

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

1
2
3
4
std::vector< std::pair <int, int> >& PDC::getTableauDepart() const
{
	return &tableau_depart;
}

Puis dans la classe principale je l'appelle comme ceci:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
void Warping::affichageDepart()
{
	std::vector< std::pair <int, int> >::const_iterator i;

	std::vector< std::pair <int, int> >& depart = pdc.getTableauDepart();

	for (i=depart.begin(); i != depart.end(); i++) {
		int x = i->first;
		int y = i->second;
        depart.draw_rectangle(x-1,y-1,x+1,y+1,red,1);
		depart.display(depart_disp);
    }
}

[Eusebe]

si tableau_depart est bien un pointeur vers un std::vector< std::pair <int, int> >, je pense que c'est bon.

Edit : si comme dans ce post, tableau_depart n'est pas un pointeur vers un vector mais un vector, il ne faut pas que tu fasses

Code :

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

return &tableau_depart;

mais

Code :

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

return tableau_depart;

dans getTableauDepart()

[Matthieu Brucher]

Est-ce que ce code est correct ?

Code :

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

1
2
3
4
std::vector< std::pair <int, int> >& PDC::getTableauDepart() const
{
	return tableau_depart;
}

Pas de & devant, on renvoie l'objet, pas un pointeur.

[Luc Hermitte]

Si ta fonction est const et que l'objet que tu renvoies est un attribut de l'objet PDC, alors il te faudra renvoyer une référence constante.

[poukill]

Merci à vous trois. C'est sur ce point que j'hésitais!
On renvoit bien une référence, c'est à dire l'objet lui même en fait...

Je viens de résoudre de mon côté un autre problème, je vous le soumet rapidement, pour voir si c'est pas complètement débile. Je crois qu'il y a peu de règles générales, donc bcp de possibilités, mais peut-être y a t-il des choses à ne pas faire?

J'ai une classe Warping qui fait son boulot dans le cadre de mon programme. Elle possède un constructeur avec plusieurs arguments, qui va à son tour appeler quelques fonctions membres assez spécialisées pour sauvegarder des Points de controle dans un fichier d'un nom précis, etc...
Le problème, c'est que j'aurai bien besoin d'appeler cette Classe Warping pour autre chose plus tard! Et passer des arguments que l'on a pas, ça pose problème!

J'ai donc pensé à une surchage de constructeur, qui va appeler à son tour des fonctions membres, toutes surchargées, car il ne faut pas faire exactement la même chose...

Ces surcharges sont-elles une bonne idée?

Merci