[friend] et [proxy]

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:

---

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:

---

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:

---

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 ?

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

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.... :koi:

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

Citation:

---

Envoyé par Luc Hermitte

Non (cela ne me choque pas).

---

Ah OK. Cool.

Citation:

---

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:

---

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 !

:merci: à Médinoc et Luc ! :king:

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.

Ok donc tu penses plutôt à ça :

Code:

---

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:

---

1 2 3

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

---

Je vais perdre en performance... :(

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

Merci pour le coup de main. :king: