Discussion :

[poukill]

Salut à tous,

J'ai juste une petite question qui me dérange... Dans ma conception j'utilise un pattern Proxy. Mon proxy se contente, comme tout bon Proxy de renvoyer l'info vers le "vrai" objet, en ajoutant son petit quelque chose. Là il se trouve que les fonctions surchagées par mon proxy sont protected. Alors il ne peut pas simplement accéder à l'objet stocké par composition.
Camera.h simplifié:

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
 
class Choc;
class CameraProxy;
struct FabriqueCameraIR;
 
class Camera {
public:
	friend CameraProxy; // Déclaration FRIEND obligatoire
	friend FabriqueCameraIR;
 
	Camera(Choc & choc, std::string nomCamera);
	virtual ~Camera() {}
 
	// Fonctions d'accès aux images
	image_type LoadImageByNumber(int number);
	image_type LoadImageByTime	(double time);
	image_type LoadLastImage	() const {return m_Image;}
 
protected:
	// A implémenter dans les classes Filles
	virtual void ImportImage(int numero) = 0; 
	virtual void CalculateNumberImages(void) = 0; 
	virtual void Initialisation () {}
};

CameraProxy.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
 
class CameraProxy :	public Camera {
public:
 
	CameraProxy(Camera * camera);
	virtual ~CameraProxy() {}
 
	virtual void ImportImage(int numeroImage);
	virtual void CalculateNumberImages();
	virtual void Initialisation();
 
private:
	Camera * m_Camera;
	static boost::mutex m_Mutex;
};

CameraProxy.cpp :

Code :

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

1
2
3
4
5
6
7
 
void CameraProxy::ImportImage(int numeroImage)
{
	// On transmet l'appel en sécurisant le réseau
	boost::mutex::scoped_lock locker( m_Mutex );
	return m_Camera->ImportImage(numeroImage); // Méthode PROTECTED
}

J'ai à peu près la même chose dans la fabrique (elle fait appel aux méthodes du proxy en fait)

Au final, ce friend CameraProxy, ça vous choque ou pas ?

[Médinoc]

Déjà, est-ce normal qu'un proxy doive donner accès à des méthodes Protected ?

[poukill]

Effectivement c'est pas courant. Pour moi le Proxy est donné au client de manière invisible (puisqu'il le reçoit comme une classe de base). Je surchage donc juste les fonctions membres qui utilisent le réseau (pour le sécuriser). Et certaines de ces méthodes sont protected.

EDIT : J'utilise aussi un peu le template method, c'est à dire que dans mes classes filles j'implémente des parties d'une méthode générale. Et ces fonctions sont protected...

Bon, ça vaut ce que ça vaut....

[Luc Hermitte]

Non (cela ne me choque pas).
Tu ne voudrais quand même pas exposer des détails internes ?

[poukill]

Non (cela ne me choque pas).

Ah OK. Cool.

Tu ne voudrais quand même pas exposer des détails internes ?

En même temps ce sont des méthodes sans danger pour la classe, et puis elle sont toujours protected dans CameraProxy... Donc à moins d'hériter, personne ne pourra les appeler non plus de l'éxtérieur.
Et puis la classe CameraProxy, personne n'est censé l'utiliser, car c'est la fabrique qui l'utilise :

Extrait du code Fabrique :

Code :

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

1
2
3
 
Camera * cam = new CameraProxy( new CameraIR(m_Choc, numberCamera, m_NomCamera) );
return cam;

Au final, si ça choque pas grand monde, tant mieux, je peux pourquivre tranquillement dans cette voie !

à Médinoc et Luc !

[Luc Hermitte]

Par contre, mes proxys sont généralement manipulés par valeur sur la pile et ne contiennent d'une référence vers l'objet véritable.

[poukill]

Ok donc tu penses plutôt à ça :

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
 
class CameraProxy :	public Camera {
public:
 
	CameraProxy(Camera & camera);
	virtual ~CameraProxy() {}
 
	virtual void ImportImage(int numeroImage);
	virtual void CalculateNumberImages();
	virtual void Initialisation();
 
private:
	Camera m_Camera;
	static boost::mutex m_Mutex;
};

C'est à dire : une référence à l'objet en question. J'ai juste en plus un mutex pour sécuriser les appels réseaux...

EDIT : Mais du coup j'aurai une copie lors de l'appel :

Code :

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

1
2
3
 
CameraIR cameraIR(m_Choc, numberCamera, m_NomCamera);
Camera * cam = new CameraProxyIR(cameraIR);

Je vais perdre en performance...

[poukill]

OK je suis passé par référence comme dans le post ci dessus.

Merci pour le coup de main.