Design Pattern en java

Bonjour,

je souhaite créer un chargement dynamique de mes objets au runtime. Je dispose de deux classes de bases dont héritent différentes classes.
JE souhaite définir une méthode "factory" qui me permettent de donner de la modularité à mon code. C'est a dire que pour rajouter une nouvelle classe dérivant d'une classe de base, je devrais juste écrire un .java pour implénter ma classe mais sans toucher au code propre à la factory.

Je sais faire ca en C++ mais je suis assez nouveau en Java donc si qqn peut m'aider

Merci

Xavier

Moi je veux bien t'aider, mais si tu ne nous dit pas avec précisions (genre un exemple) ce qui ne vas pas, je doute que bcp de gens puisse t'aider...

As-tu regarder dans les tutos présent sur le site? Il me semble, de souvenir fort vague, qu'il y a un article qui parle de factory...

je vais expliquer plus en détails:

j'ai une classe de base Solveur et j'ai deux classes Simmetrix et Samcef qui dérivent de Solveur
Puis j'ai aussi une classe de base Analysis et Static et Fracture sont des classes dérivées de Analysis.


J'ai une factory qui me permet d'instancier soit un objet Samcef soit un objet
Simmetrix.

Code:

---

1 2 3 4 5 6 7 8 9 10 11

public class Factory { public Factory() { }   public Solver getSolver(String s) { if (s=="Simmetrix") return new Simmetrix(s); else if(s=="Samcef") return new Samcef(s); } }

---

Mais j'aimerais avoir une factory qui me permette via des mots clés, d'instancier des objets Samcef, Simmetrix, Static, Fracture.

Je ne trouve pas comment faire cette factory capable d'instancier des objets de classes dérivant de classes de base différentes à l'aide de mots clés.

J'espère que j'ai été clair mais c difficile à expliquer comme ca.

Merci

Xavier


Edité par Lunatix : merci de penser aux balises CODE

Si je comprend bien, tu voudrais que ton Factory retourne autre chose que des Solver en fonction du parametre ? Que ce soit des Solver ou des Analysis?

Oui voila, comme ca je peux ajouter des nouvelles classes dérivant des classes de base Solveur et Analysis sans toucher au code de la factory.


Le paramètre serait une string que j'irai lire dans des fichiers texte:

si le mot clés "Simmetrix" apparait alors je dois instancier un solveur de type Simmetrix. De plus si un utilisateur veut rajouter un solveur il ne devrait que
coder la classe NewSolver extends Solver et définir le mot clés "newsolver" comme paramètre permettant d'instancier un objet de type NewSolver.

Voila je pense que ca explique un peu plus le mécanisme que je veux mettre en place.

Merci,

Xavier

Code:

---

1 2 3 4 5 6 7 8

Class maClasse = Class.forName(nomClasse); if (!TaSuperClasse.class.isAssignableFrom(maClasse)); throw new Exception ("La classe n'implemente pas TaSuperClasse");   //si constructeur sans arguments TaSuperClasse obj = (TaSuperClasse)maClass.new Instance(); //sinon TaSuperClasse obj = (TaSuperClasse)maClasse.getConstructor(tableauDeTypesDArguments).newInstance(tableauDArguments);

---

Salut,


Je pense que tu as tapé ton code ici comme exemple, mais il y a deux problèmes :

• 1. Il ne peux pas compiler car dans un cas il ne renvoit pas de valeur.
  2. Les comparaisons de String se font avec la méthode equals() et non pas == !

Maintenant si j'ai bien compris tu veux que la méthode getSolver() soit générique et marche quelque soit la valeur passé en paramètre (a condition qu'elle corresponde à un objet implémentant Solver).


Il te suffit d'utiliser l'API de reflection et d'utiliser les noms complets des classes (package + nom) en paramètres de la méthode :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

public class Factory { public Solver getSolver(String className) throws Exception {   // On récupère l'objet représentant la classe : Class pClass = Class.forName(className); // throw ClassNotFoundException   // On instancie une nouvelle instance : Object instance = pClass.newInstance(); // throw InstantiationException, IllegalAccessException   // Si l'instance crée n'est pas une instance de Solver if ( !(instance instanceof Solver) ) { throw new Exception ("La classe n'implemente pas Solver"); } return (Solver) instance; } }

---

Je n'ai pas géré les exception à toi de le faire...

a++

public class Factory {
public Solver getObject (String className) throws Exception {

// On récupère l'objet représentant la classe :
Class pClass = Class.forName(className);
// throw ClassNotFoundException

// On instancie une nouvelle instance :
Object instance = pClass.newInstance();
// throw InstantiationException, IllegalAccessException

// Si l'instance crée n'est pas une instance de Solver
if ( !(instance instanceof Solver) ) {
throw new Exception ("La classe n'implemente pas Solver");
}
return (Solver) instance;
}
}


mais ici l'objet que je récupère est une instance de la classe Solver. Or je veux pouvoir avec cette factory créer des instances de la classe Analysis:

J'ai une classe Fracture extends Analysis et Static extends Analysis et dans ces deux classes j'ai une méthode RUN() et je veux faire qqch du style:

Object object = getObject("fracture");
object.RUN(); --> appel la méthode RUN de la classe fracture

or avec ce que tu m'as donné, on obtient en retour une Instance de la classe Solver donc ca va pas.

Merci,

Xavier
[/b]

Bonjour,

n'oublie pas que toutes les classes java ont une classe mère "Object"

Code:

---

1 2 3 4 5 6 7 8 9 10

Class maClasse = Class.forName(nomClasse);   if (Solver.class.isAssignableFrom(maClasse) || Analysis.class.isAssignableFrom(maClasse) || ...) { //si constructeur sans arguments return maClass.new Instance(); //sinon return maClasse.getConstructor(tableauDeTypesDArguments).newInstance(tableauDArguments); } else throw new Exception ("...");

---

Tu ne pourras pas faire ca :

Code:

---

1 2	Object object = getObject("fracture"); object.RUN(); --> appel la méthode RUN de la classe fracture

---

utilise plutot ce genre de choses.

Code:

---

1 2	Analysis object = (Analysis)getObject("fracture"); object.RUN(); --> appel la méthode RUN de la classe fracture

---

Citation:

---

Envoyé par mencaglia

or avec ce que tu m'as donné, on obtient en retour une Instance de la classe Solver donc ca va pas.

---

Si Solver et Analysis sont deux classes totalement différents alors tu es obligé de retrouné un Object...

Par contre je ne vois pas trop l'intérêt si il n'y a aucune interface commune car il te faudra ensuite faire plusieurs cast et instanceof pour sécuriser ton code...

a++

Citation:

---

Envoyé par adiGuba

Citation:

---

Envoyé par mencaglia

or avec ce que tu m'as donné, on obtient en retour une Instance de la classe Solver donc ca va pas.

---

Si Solver et Analysis sont deux classes totalement différents alors tu es obligé de retrouné un Object...

---

Vrai. Pas d'interface commune ? Pas de super classe commune ?

Citation:

---

Envoyé par adiGuba

Par contre je ne vois pas trop l'intérêt si il n'y a aucune interface commune car il te faudra ensuite faire plusieurs cast et instanceof pour sécuriser ton code...

---

Autre solution, il faut typer un peu plus fortement les méthodes de ta factory.
Exemple :
- 1 méthode public Solver getSolver(String arg)
- 1 méthode public Analysis getAnalysis(String arg)

dans mon main j'ai ceci:


public static void main(String[] args) {
System.out.println("start main");

Factory factory = new Factory();
Solver object = (Solver)factory.getSolver("Simmetrix");
System.out.println(object.get_type());
System.out.println("finish main");
}
et il me met une exception :
unreported exception java.lang.Exception; must be caught or declared to be thrown at line 84 (84:50)

désolé, mais comme j'ai dis au début je suis assez novice en Java... j'ai fait que du c++ jusqu a présent,

Si tu peux m'aider encore u peu
merci
Xavier

Code:

---

public static void main(String[] args) throws Exception

---

Comme ca tu ne te poseras pas trop de questions pour commencer. Tu devras par contre t'interesser à la gestion des Exception par la suite.

Code:

---

1 2 3 4 5 6 7 8 9

public static void main(String[] args) throws Exception { System.out.println("start main"); /* Simmetrix simmetrix = new Simmetrix();   simmetrix.run();*/ Factory factory = new Factory(); Solver object = (Solver)factory.getSolver("Simmetrix"); // System.out.println(object.get_type()); System.out.println("finish main"); }

---

j'obtient cette exception:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

Exception in thread "main" java.lang.ClassNotFoundException: Simmetrix at java.net.URLClassLoader$1.run(URLClassLoader.java:199) at java.security.AccessController.doPrivileged(Native Method) at java.net.URLClassLoader.findClass(URLClassLoader.java:187) at java.lang.ClassLoader.loadClass(ClassLoader.java:289) at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:274) at java.lang.ClassLoader.loadClass(ClassLoader.java:235) at java.lang.ClassLoader.loadClassInternal(ClassLoader.java:302) at java.lang.Class.forName0(Native Method) at java.lang.Class.forName(Class.java:141) at Atelier_logiciel_fatigue.Factory.getSolver(Factory.java:32) at Atelier_logiciel_fatigue.Solver.main(Solver.java:85)

---

Je vais m''intéresser à la gestion des exceptions ... si tu as une idée pour cette exception car j'aimerais vérifier que la méthode que tu m'as donné correspond à mes besoins.

merci
Xavier

ta classe est dans un package ?
Si oui :

Code:

---

Solver object = (Solver)factory.getSolver("tonpackage.Simmetrix");

---

ou alors, tu prefixe dans la méthode getSolver :

Code:

---

1 2 3 4 5

public Solver getSolver(String s) { String className = "package."+s; Class classe = Class.forName(className); ... }

---

par contre, le pb c'est que ca implique que tes Solver soient tous dans le meme package dans ce cas là.

Autre solution, dans ta factory, tu initialises une HashMap de Class avec comme clé Simmetrix, ...
Tu récuperes ta classe en faisant tamap.get("Simmetrix") et tu utilises le code qui ne change pas derrière.

Bonjour,
un petit detail dans le pattern "Factory" la methode getUnObjetSelonLesParametre est declaré "static" pour l appeler sans declarer d'instance de cette classe. Par la suite l'exeption levé est un probleme d'inclde ou de path (ce qui reviens grosso-modo au meme).

Citation:

---

Envoyé par BainE

Bonjour,
un petit detail dans le pattern "Factory" la methode getUnObjetSelonLesParametre est declaré "static" pour l appeler sans declarer d'instance de cette classe.

---

Faux. Ce que tu dis est contraire aux patterns Abstract Factory et Factory Method. Comment pourrais tu substituer une factory par une autre en utilisant des méthodes statiques ?

Citation:

---

Envoyé par BainE

Par la suite l'exeption levé est un probleme d'inclde ou de path (ce qui reviens grosso-modo au meme).

---

Le classpath est sans doute bon mais il manque certainement le package pour que le Class.forName(...) fonctionne.

merci pour le tuyau, j'ai rectifier l'histoire du package:
mais quand je fais un get_type() sur mon objet Solver, j'ai en retour un type Solver or le mot clés que j'ai donné en paramètre de mon gerSolver est "Simmetrix" donc je devrais avoir un type Simmetrix.

Code:

---

1 2 3 4 5 6 7

public static void main(String[] args) throws Exception { System.out.println("start main"); Factory factory = new Factory(); Solver object = (Solver)factory.getSolver("Atelier_logiciel_fatigue.Simmetrix"); [b]System.out.println(object.get_type());[/b] System.out.println("finish main"); }

---

si vous pouvez m'aider encore un petit peu,

Merci
Xavier

re,
je ne veux rien substituer du tout, mais pour moi le shéma UML est pas facile a decrire mais en decoule dans le code ces lignes :

Code:

---

1 2	InterfaceObjet Factory.getMonObjet( param1, param2 );

---

Factory inclue l'objet mere de ceux qui peuve etre rendu par la Factory.
Je vois rien d'impossible dans ce concepte la.

Montre le code de ta méthode getSolver et reformule ta question, je ne comprends pas bien. Qu'est ce que te donne System.out.println(object.get_type()); ?

Citation:

---

Envoyé par BainE

re,
je ne veux rien substituer du tout, mais pour moi le shéma UML est pas facile a decrire mais en decoule dans le code ces lignes :

---

Je ne sais pas de quel pattern factory tu parles. Moi je te parle des patterns du GoF (Gang of Four), le livre de référence sur le sujet.

Pour plus de détails sur les patterns factory (et du GoF) regardes ces liens (attention, implémentations en C#) :
- http://www.dofactory.com/Patterns/PatternFactory.aspx
- http://www.dofactory.com/Patterns/PatternAbstract.aspx
- http://www.dofactory.com/Patterns/Patterns.aspx#list

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

public class Factory { public Solver getSolver(String className) throws Exception {   String classNametmp = "Atelier_logiciel_fatigue."+className;   Class maClasse = Class.forName(classNametmp);   if (Solver.class.isAssignableFrom(maClasse)) { return (Solver) maClasse.newInstance(); } return null; } }

---

Voila ma méthode getSolver

le type est une donnée membre de chaque classe Solver, Simmetrix
get_type() renvoie Solver pour la classe Solver et Simmetrix pour la classe Simmetrix, c'est juste pour voir le type de l'objet que j'instancie.

Code:

---

1 2 3 4 5 6 7 8 9

public static void main(String[] args) throws Exception { System.out.println("start main"); /* Simmetrix simmetrix = new Simmetrix();   simmetrix.run();*/ Factory factory = new Factory(); Solver object = (Solver)factory.getSolver("Simmetrix"); System.out.println(object.get_type()); System.out.println("finish main"); }

---

Mais la j'instancie un objet de type Solver alors que j'ai mis le mot clés Simmetrix donc je devrais avoir un objet de type Simmetrix.

J'espère que j'ai été clair ,
merci

Xavier

Remplace System.out.println(object.get_type()); par System.out.println(object.getClass()); et tu verras que ton objet est bien un objet de type Simmetrix.

oki dlemoing, le pattern strict ne fais pas ca, mais je suis pas sur que ce soit idiot, en tout ca merci pour le lien il me servira.

merci bcp à tout le monde,

je vais m'intéresser aux différences entre Java et C++ pour éviter certaines surprises et je vais m'intéresser aux gestions d'exception.

Bonne fin d'aprem à tout le monde,

Xavier

Citation:

---

Envoyé par BainE

oki dlemoing, le pattern strict ne fais pas ca, mais je suis pas sur que ce soit idiot, en tout ca merci pour le lien il me servira.

---

Désolé de t'avoir fait penser que je considérais comme idiot ce que tu proposais. C'est juste que j'ai tendance à considerer qu'on ne doit pas abuser de l'utilisation du "static".
Les membres et méthodes statiques, bien qu'ayant leur utilité, ne sont pas toujours le gage d'une conception objet de qualité et ils ont rarement leur place dans les designs patterns.
Ce n'est pas pour rien si dans une factory les méthodes ne sont pas statiques, mais pour comprendre tout ca je te conseille le livre du GoF (existe aussi en francais).

nan je suis pa vexé :wink:
je suis simplement parti de l idée que la factory est une classe dont la création d'instance est superflux, en effet une factory ne doit pas avoir beaucoup d attributs, et une fois crée et demandé une instance par un getter l instance est ramassé par le garbage collector. Il n'y a aucun interet de declaré plusieurs intance... Et je me basai sur mes souvenirs de cours...
houlala :oops: enfin voila
pour rajouté une betise il me semblai meme qu'il faille declaré le constructeur de la factory en private...

Donc il va falloir que je revise dur !!!
Voila bonne apres midi a tous
et :ave: dlemoing

Lu

je reviens, car j'ai un petit pb;




public static void main(String[] args) throws Exception {
System.out.println("start main");
Factory factory = new Factory();
Solver object = (Solver)factory.getSolver("Simmetrix");
System.out.println(object.getClass());
object.run();
System.out.println("finish main");
}

quand j'appelle une méthode de la classe Simmetrix avec mon instance objet , j'ai le mess suivant :
cannot find symbol; symbol : method run(), location: class Atelier_logiciel_fatigue.Solver at line 89 (89:16)
Pourtant j'ai bien la méthode run() car Simmetrix implémente l'interface Runnable. Je comprends pas pourquoi il va chercher la méthode dans la class Solver .

merci

Xavier

Il faut que ta méthode run() soit définie dans la classe Solver (ou déclarée en tant que méthode abstraite pour être redéfinie dans les sous-classes, mais alors Solver devient une classe abstraite).
Par contre, à l'execution, ce sera bien la méthode run définie dans Simmetrix qui sera executée (polymorphisme oblige).

OK, mais alors toutes les méthodes que je veux coder dans des classes dérivées doivent être mise dans la classe de base comme abstract méthode.

En C++, on peut faire ca, ajouter des méthodes dans des classes dérivées et qui ne sont pas déclarées dans la classe de base, il suffit juste de faire un dynamic_cast pour forcer le type et ainsi appeler la méthode.

Ca permet d'avoir une classe de base avec toutes les méthodes qui sont communes à l'ensemble des classes dérivées et de préciser le comportement des classes "filles" en ajoutant des méthodes...

Merci
Xavier

Tu peux également forcer le type si tu connais l'interface à utiliser :

Code:

---

1 2	Solver obj = factory.getSolver(...); ((Runnable)obj).run();

---

ou directement :

Code:

---

1 2	Runnable obj = (Runnable)factory.getSolver(...); obj.run();

---

Apparte : je suis desole mais les exceptions ca existe depuis longtemps en C++ et a ca ete finallement standardise entre '94 et '96 (pask'avant chacun faisait a sa sauce). Outre les exceptions propres a chacun des vendeurs (ex: CException dans MFC) il existe std::exception de base dans ISO C++.

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 25 26 27 28

#include <exception>   class MonException1 : public std::exception { . . . };   class MaClasse { public: . . . void moOp() throw (MonException1, MonException2); . . . };   void MaClasse::monOp() throw (MonException1, MonException2) { . . . if (testErreur) { throw MonException1("Ca plante paske"); } . . . }   try { monInstance.monOp(); } catch (MonException1& e1) { std::cout << "ERREUR : " << e1.what() << std::endl; }

---

La grosse diff c'est qu'en C++on peut faire un throw de n'importe quoi (un int, un pointeur, une ref, un objet quelconque, ...) ce qui rend difficile la gestion des erreurs. Bien qu'on puisse faire un :

Code:

---

1 2 3 4 5 6 7 8 9

try { . . . } catch (...) { // Oui il faut bien ecrire 3 points d'affilee. // Traitement de l'erreur. // Note : avec "..." impossible de savoir ce qui a ete catche). throw; // relancement eventuel de l'erreur. }

---

Egalement il n'existe pas de bloc finally.

En Java on ne peut lancer que des Throwable et classe heritee comme Exception et on dipose d'un bloc finally toujours execute (si ca marche ou si ca plante).

Code:

---

1 2 3 4 5 6 7 8 9 10 11

try { . . . } catch (Throwable t) { // Traitement de l'erreur. throw t; // relancement eventuel de l'erreur. } finally { // Traitement qui est tout le temps effectue. }

---