Discussion :

[bourbaki2003]

Bonjour,

J'ai une petite question d'architecture. J'ai une interface Communication et deux implementations: CarCommunication et HotelCommunication. Ces deux classes font la communication avec des fournisseurs de voitures et d'hotels. Ces classes sont instanciees depuis une fabrique (CommunicationFactory). Maintenant, j'aimerai que suivant certaines variables d'environnement, la fabrique instancie une autre classe qui joue un peu le role d'un mock object. Grosso modo, cette classe lit des fichiers XML et les renvoie aux classes clientes. Donc cette classe peut etre independante du produit (Hotel, Car...). J'ai pense a une premiere solution qui consistait a cree une nouvelle classe par produit (ex: HotelMockCommunication..) qui herite de la vrai classe de communication, le probleme est que le code entre HotelMockCommunication et CarMockCommunication va etre le meme, donc il est un peu bete d'avoir deux classes differentes qui font la meme chose.

Si vous avez des idees. J'espere avoir ete assez clair.

Merci d'avance.

[denisC]

J'ai pense a une premiere solution qui consistait a cree une nouvelle classe par produit (ex: HotelMockCommunication..) qui herite de la vrai classe de communication, le probleme est que le code entre HotelMockCommunication et CarMockCommunication va etre le meme, donc il est un peu bete d'avoir deux classes differentes qui font la meme chose.

Cette solution me parait effectivement être la bonne. Après, rien ne t'empeche de mutualiser le code dans une class MockCommunication, par exemple, et de faire simplement de la délégation dans HotelMockCommunication.

En fait, la vrai solution propre, serait que HotelCommunication soit une interface (idem pour CarCommunication), et d'avoir un MockCommunication qui implémente les deux, plus des vrais implementation HotelCommunicationImpl et CarCommunicationImpl.

Mais bon, à toi de voir jusqu'à quelle compléxité de la chose il est intéressant d'aller....

[had35]

Pour quelle raison as-tu écris les classes HotelMockCommunication et CarMockCommunication ?

Si c'est pour répondre à l'écriture de ton "mock object", à mon avis, il aurait fallu y répondre soit en modifiant la hiérarchie d'héritage de CarCommunication et HotelCommunication pour qu'elles héritent de ta classe mock object et non l'inverse, soit que ces classes implémentent une interface commune déclarant les méthodes appelées ailleurs (apparemment Communication).

[fabszn]

Pour quelle raison as-tu écris les classes HotelMockCommunication et CarMockCommunication ?

Si c'est pour répondre à l'écriture de ton "mock object", à mon avis, il aurait fallu y répondre soit en modifiant la hiérarchie d'héritage de CarCommunication et HotelCommunication pour qu'elles héritent de ta classe mock object et non l'inverse, soit que ces classes implémentent une interface commune déclarant les méthodes appelées ailleurs (apparemment Communication).

Hello,

Je pense qu'un Mock Object ne doit pas intervenir dans la hiérarchie des classes de l'application. cela doit être une classe qui vient se greffer temporairement et supprimer lors de mise en production.

qui herite de la vrai classe de communication, le probleme est que le code entre HotelMockCommunication et CarMockCommunication va etre le meme, donc il est un peu bete d'avoir deux classes differentes qui font la meme chose.

Tu indiques que le code va être le même, donc effectivement il n'y a pas grand intéret. Mais normalement une classe mock Object n'est pas implémentée, normalement, imiter une source de données (ou similaire)? donc le code doit être forcement différent..

En fait, la vrai solution propre, serait que HotelCommunication soit une interface (idem pour CarCommunication), et d'avoir un MockCommunication qui implémente les deux, plus des vrais implementation HotelCommunicationImpl et CarCommunicationImpl.

Est ce que l'on ne perd pas l'interet de l'interface si on créé deux interfaces (hotel et car)? L'interet d'avoir une interface commune entre les deux classes permet de manipuler les classes sans se soucier du type réel de la classe d'implémentation. non?

Ca s'appelle un post tire-groupé