Un Problème d'instance

Version imprimable

21/04/2009, 19h30
kululu

Un Problème d'instance

Bonjour

J'ai un petit soucis:
Tout d'abord je vous explique ma situation :)
Je veux créer 2 agents communicants tout simple :
_un 1er Agent vérifie si la porte de la maison est ouverte et si non il appelle le portier
_Le 2eme Agent, le portier viens à la rescousse du 1er en lui ouvrant la porte si elle est fermée

Pour cela j'ai créé une classe Déménageur(le 1er agent), une classe Portier, et enfin une classe Maison

Maison n'est pas un agent mais possèdes les Attribut tel que le boolean qui dit si la porte est ouverte (que l'on récupère avec un geteur)

Or pour utiliser les méthodes de la classe Maison (les geteurs et les seteurs) je dois instancier la classe Maison dans mes 2 agents (en faisant new Maison() ) mais si je fais cela j'aurais donc une "maison" différente (donc 2 instances différente) de la maison

Quel pourrait-être la solution pour que mes 2 agents utilisent la même instance de Maison? (ou ptêt bien qu'c'est po possible?)

Merci d'avance 8-)

Citation:

Envoyé par kululu

Quel pourrait-être la solution pour que mes 2 agents utilisent la même instance de Maison? (ou ptêt bien qu'c'est po possible?)

Evidemment que c'est possible !
Sinon on n'irait pas bien loin avec la programmation objet :aie:

J'imagine d'après ta description que tu fais quelque chose de ce genre dans ton code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 public class Portier { private Maison maison; public Portier() { this.maison = new Maison(); } } public class Demenageur{ private Maison maison; public Demenageur() { this.maison = new Maison(); } } public static void main(String[] args) { Portier portier = new Portier(); Demenageur demenageur = new Demenageur(); }
Solution : au lieu de créer une nouvelle instance de maison dans les constructeurs des classes Portier et Demenageur, passe plutôt ta maison déjà instanciée en paramètre des constructeurs.
Ça te permet d'affecter le portier et le déménageur à la maison que tu veux, plutôt que de les affecter systématiquement à une nouvelle maison, inconnue.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 public class Portier { private Maison maison; public Portier(Maison maison) { this.maison = maison; } } public class Demenageur{ private Maison maison; public Demenageur(Maison maison) { this.maison = maison; } } public static void main(String[] args) { Maison maison = new Maison(); Portier portier = new Portier(maison); Demenageur demenageur = new Demenageur(maison); }
EDIT

Par ailleurs, tu peux aussi décider de ne pas affecter "définitivement" un agent à une maison (et donc supprimer le champ maison) mais de passer en paramètre la maison aux méthodes qui l'utilisent.
C'est à dire au lieu de faire :
Code:

1 2 3 4 5 6 7 8 9 public class Portier { private Maison maison; public Portier(Maison maison) { this.maison = maison; } public ouvrir() { // traitement sur la maison } }
faire :
Code:

1 2 3 4 5 6 7 public class Portier { public Portier() { } public ouvrir(Maison maison) { // traitement sur la maison } }
Ça dépend de la relation entre tes agents et les maisons : est-ce que tout au long de sa vie un agent est affecté à une maison (et alors il est logique de garder dans la classe une référence sur une instance de Maison) ou est-ce qu'un agent peut agir sur n'importe quelle maison (et alors il est logique de passer la maison en paramètre des actions que peut effectuer l'agent) ?

21/04/2009, 20h05
darrylsite
Citation:

Envoyé par kululu

Bonjour
Quel pourrait-être la solution pour que mes 2 agents utilisent la même instance de Maison? (ou ptêt bien qu'c'est po possible?)

Merci d'avance 8-)

Pour avoir la meme classe maison dans les deux agents, il te suffit de creer d'abord la maison et ensuite passer la reference de la classe maison à tes agents via une methode.
En gros,
Code:

1 2 3 4 5 6 7 Maison laMaison= new Maison(); Agent agent1= new Agent("nom", ...); Agent agent2=new Agent("non",...); agent1.setMaison(laMaison); agent2.setMaison(laMaison);
La classe Agent possede une proprité dont laquelle on va garder la refence de maison.
21/04/2009, 20h07
kululu

Merci Astartee :king: en effet j'y avais pensé à la solution de mettre en paramètre (dans la méthode main) la Maison lorsque j'instancie mes 2 agents

Mais le prolème c'est que j'utilise JADE (un Framework multi-agent) et donc je ne passe pas par un main (mais par une ligne de commande qui me créé mon agent)

Je suis débutant encore dans l'utilisation de JADE mais je me demande si ça pourrait marcher ça :

-gui jade.Boot totoDemenageur:Demenageur(maison) //<-- paramètre Maison
-gui jade.Boot titiPortier:Portier(maison)

Mais si oui comment le JVM saura que c'est de la MEME maison que l'on parle? :(
De plus si je ne m'abuse les parmètres entre parenthèse sont des String et non des objet :s
21/04/2009, 20h10
kululu

Bonne idée darrylsite un seteur de Maison :) Mais comment faire toujours si j'ai pas de main?
21/04/2009, 21h58
Astartee

Citation:

Envoyé par kululu

Mais le prolème c'est que j'utilise JADE (un Framework multi-agent) et donc je ne passe pas par un main (mais par une ligne de commande qui me créé mon agent)

Ah bah si tu utilises un framework spécial au lieu d'écrire directement du code Java, il faut le dire tout de suite, ça évite de perdre du temps à répondre à côté de la plaque ;)
Je ne connais pas JADE, il faudra attendre quelqu'un d'autre pour t'aider...
22/04/2009, 09h50
ArnaudDev

Sinon une autre solution consiste à mettre les variables importantes que tes deux classes démenageur et portier vont tout deux utiliser en static.
Cela signifie que même si tu as deux instances différentes de la classe maison, elle auront toutes les deux accès à la même variable static !
Mais sinon la première solution reste la mieux enfin cela dépend de ce que tu comptes faire !

Bonjour,

Sinon, tu peux utiliser le design pattern (=patron de conception) Singleton.
Le principe est simple : tu crées une classe où tu t'assures qu'une seule instance de ta classe sera crée.

Pour cela, tu rends le constructeur de ta classe Maison privé, tu utilises une instance privée statique de ta classe Maison, et tu crées une méthode statique getInstance qui :

crée une nouvelle instance de ta classe maison si elle n'a jamais été instanciée
renvoie l'instance statique si la méthode a déja été appellée.

Dans ton cas, tu auras le code suivant (un bout de code vaut toujours mieux qu'un long discours :king:) :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 public class Maison{ private static Maison monUniqueMaison; // Ton instance privée statique /* Le constructeur privé de ta classe Maison */ private Maison() { } /* La méthode statique que tes agents appelleront à chaque fois qu'ils veulent accéder à "la" maison */ public static Maison getInstance() { if (monUniqueMaison == null) { monUniqueMaison= new Maison(); } return monUniqueMaison; } }
Avec ca, tu peux être sûr qu'il n'y aura qu'une seule Maison dans ton application ;)

Pour + d'infos sur le design pattern Singleton, regarde ici

Bon pour le posteur original je suis désolé je ne connais pas JADE, je ne peux pas t'aider.

Par contre je souhaite rectifier ce qu'a dis mr.mams :

Citation:

Envoyé par mr.mams

Bonjour,

Sinon, tu peux utiliser le design pattern (=patron de conception) Singleton.
Le principe est simple : tu crées une classe où tu t'assures qu'une seule instance de ta classe sera créée.
(...)
Avec ca, tu peux être sûr qu'il n'y aura qu'une seule Maison dans ton application ;)

Il n'a jamais dit qu'il souhaitait une seule maison pour son application. Il a dit qu'il voulait qu'un portier et un demenageur ait une reference sur la meme maison.
Mais il est tout a fait possible qu'il ait plusieurs maisons, avec chacune un portier et un demenageur. Donc le singleton n'a pas beaucoup de sens.
Il faut faire attention de ne pas abuser des singletons. On tombe vite dans l'anti-pattern.

Sinon d'un point de vue purement technique : Ton code de singleton n'est pas thread-safe.
Pour les singletons en Java on utilisera plutôt ceci (quand ca a un sens de mettre un singleton evidemment !) :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 public class Maison{ private static Maison monUniqueMaison = new Maison(); // Ton instance privée statique /* Le constructeur privé de ta classe Maison */ private Maison() { } /* La méthode statique que tes agents appelleront à chaque fois qu'ils veulent accéder à "la" maison */ public static Maison getInstance() { return monUniqueMaison; } }
Cf (par exemple) : http://www.ibm.com/developerworks/library/j-dcl.html

07/05/2009, 17h47
mr.mams

Citation:

Envoyé par Ksempac

Il n'a jamais dit qu'il souhaitait une seule maison pour son application. Il a dit qu'il voulait qu'un portier et un demenageur ait une reference sur la meme maison.
Mais il est tout a fait possible qu'il ait plusieurs maisons, avec chacune un portier et un demenageur. Donc le singleton n'a pas beaucoup de sens.
Il faut faire attention de ne pas abuser des singletons. On tombe vite dans l'anti-pattern.

Bien vu ! Mea culpa, je ferai plus attention la prochaine fois :mrgreen:

Citation:

Envoyé par Ksempac

Sinon d'un point de vue purement technique : Ton code de singleton n'est pas thread-safe.

C'est vrai tu as raison. En plus tu gagnes au niveau perfo car tu n'as pas besoin de tester l'existence de ta maison à chaque appel de la méthode getInstance :bravo: