Discussion :

[Zhitoune]

Bonjour à tous!

dans mon projet, je dois faire communiquer deux classes cameraSettings et camera qui sont créées en même temps.
camera dialogue avec une camera qui lui est associé, et cameraSettings est une interface graphique pour régler les paramètres d'utilisation de la caméra. Il est donc impératif que ces deux classes puissent s'échanger des informations.
Pour ce faire j'ai déclaré dans chaque classes une variable du type de l'autre classes :

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
//cameraSettings.h
#include "camera.h"
class cameraSettings :public QWidget
{
Q_OBJECT
public :
cameraSettings ();
camera *cam;
};
 
//camera.h
#include "cameraSettings.h"
class camera 
{
public :
camera();
cameraSettings *gui;
};

de cette façon, une fois les classes créées, je fais le lien les objets :

Code :

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

1
2
3
4
5
camera *cameraS = new camera();
cameraSettings *displayS = new cameraSettings();
 
cameraS->gui = &displayS;
displayS->cam = &cam;

et ...... ca marche!

par contre là où ça ne marche plus c'est quand il y a plusieurs cameras : chaque camera étant controlées par un couple camera/cameraSettings, je dois donc créer autant de couple qu'il y a de caméras. J'utilise donc des tableaux d'objets :

Code :

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

1
2
3
4
5
6
7
8
//les headers des deux classes ne changent pas.
camera *cameraS[nbCamera];
cameraSettings *displayS[nbCamera];
for (int i = 0; i < nbCamera ; ++i)
{
	cameraS[i]->gui = displayS[i];
	displayS[i]->cam = cameraS[i];
}

tout compile correctement, mais le programme plante dès que l'une des classes utilise l'autre.... et évidement je ne sais pas comment faire... donc help si je puis me permettre.

Merci d'avance!

[bacelar]

C'est pas beau comme code mais bon, on n'est pas là pour ça.

Code :

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

1
2
3
4
5
6
7
8
9
10
camera *cameraS[nbCamera];
cameraSettings *displayS[nbCamera];
for (int i = 0; i < nbCamera ; ++i)
{
	cameraS[i] = new camera();
	displayS[i] = new cameraSettings();
	cameraS[i]->gui = displayS[i];
	displayS[i]->cam = cameraS[i];
}

C'est juste pour la forme car, entre autre, la gestion mémoire est pourrie.

[Zhitoune]

merci pour ta réponse.
j'ai rajouté les lignes mais le programme plante toujours.
sinon je me doute que cette méthode est bancale mais je n'en vois pas d'autre, pour éviter de mélanger mes caméras et mes boutons...
est-ce que tu pourrais m'indiquer une autre manière de faire??

[3DArchi]

A mon avis, il pensait entre autres aux pointeurs nus : Présentation des pointeurs intelligents en C++ (de Loïc Joly) et Les pointeurs intelligents de Qt de Thiago Macieira et Harald Fernengel, traduction Thibaut Cuvelier).
Au passage tu devrais regarder les conteneurs STL.

Ensuite, juste avec le nom de tes classes, je comprend que cameraSettings appartiennent à camera, mais je ne comprend pas trop pourquoi camera appartient à cameraSettings ?

[Zhitoune]

merci je me documente de ce pas.

sinon par rapport au nom, ils ne suivent aucun protocole, ils reflètent juste la fonction de la classe.
d'autre part camera doit connaitre cameraSettings car les caméras revoient parfois des informations utile pour l'utilisateur que j'aimerai pouvoirafficher dans cameraSettings.
J'espère avoir répondu correctement à ta question

[bacelar]

j'ai rajouté les lignes mais le programme plante toujours.

Quelle ligne ?
Quel type d'erreur ?