Discussion :

[Quetzalcoatl]

Bonjour à tous,

comment peut-on creer une instance d'une classe passée en paramètre ? Y'a t'il une méthode facile ? Voici ce que j'ai trouvé :

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
 
   protected final void setMover(Class agentMoverClass) {
        Class [] paramsType = new Class[2];
        Object [] params = new Object[2];
        Constructor  constructor;
        AgentMover mover;
 
        paramsType[0] = Class.forName("Agent");
        paramsType[1] = Class.forName("Simulator");
 
        constructor = agentMoverClass.getConstructor(paramsType);
 
        params[0] = this; /* an instance of Agent */
        params[1] = simulator; /* an instance of Simulator */
 
        mover = (AgentMover) constructor.newInstance(params);
        this.mover = mover;
   }

Je trouve ca assez lourd. Une idée ?

Merci

[adiGuba]

Salut,

Je trouve ca assez lourd. Une idée ?

Si tu veux faire appel au constructeur avec des paramètres tu peux difficilement faire plus court (et encore dans ton code il n'y a aucune gestion des erreurs).


Par contre si tu utilises le constructeur par défaut et que tu définis des accesseurs dans ton interface AgentMover, tu peux encore simplifier le tout :

Code :

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

1
2
3
4
5
6
7
8
9
10
protected final void setMover(Class agentMoverClass) throws Exception {
 
	Object object = agentMoverClass.newInstance();
	if (object instanceof AgenMover) {
        	AgentMover mover = (AgentMover) object;
		mover.setAgent(this);
		mover.setSimulator(simulator);
		this.mover = mover;
	}
}

a++

[Quetzalcoatl]

Tant pis, merci beaucoup.

[natha]

Plus court, plus simple (Java 5) :

Code :

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

1
2
3
4
5
6
7
8
protected final void setMover(Class agentMoverClass) {
    try {
        Constructor constructor = agentMoverClass.getConstructor(Agent.class, Simulator.class);
        this.mover = constructor.newInstance(this, simulator);
    } catch (NoSuchMethodException e) {
        // TODO ...
    }
}

Il doit y avoir une toute petite variante pour que ça marche en 1.4, je pense qu'il ne faut pas modifier grand chose (peut-être même rien du tout).

[adiGuba]

Avec Java 1.4 et inférieur cela revient à faire ceci :

Code :

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

1
2
3
4
5
6
7
8
protected final void setMover(Class agentMoverClass) {
    try {
        Constructor constructor = agentMoverClass.getConstructor(new Class[]{Agent.class, Simulator.class});
        this.mover = (AgentMover) constructor.newInstance(new Object[]{this, simulator});
    } catch (NoSuchMethodException e) {
        // TODO ...
    }
}

Ce qui revient au même code que celui d'origine, mis à part qu'ici il est plus compact (mais moins lisible).

Attention aussi aux ClassCastException puisque le type n'est pas vérifié...

a++

[natha]

Moins lisible ? Je trouve que c'est plus lisible au contraire, on voit directement qu'on appelle le constructeur avec tel et tel paramètres.

Pour le class cast, en Tiger il y a les génériques (que je n'ai pas précisés dans mon exemple), sinon rajouter un catch n'est pas très méchant.

[adiGuba]

Moins lisible ? Je trouve que c'est plus lisible au contraire, on voit directement qu'on appelle le constructeur avec tel et tel paramètres.

Pour le class cast, en Tiger il y a les génériques (que je n'ai pas précisés dans mon exemple), sinon rajouter un catch n'est pas très méchant.

Je parle bien sur du code Java 1.4 avec l'initialisation du tableau directement dans l'appel de méthode, ce que je ne trouve pas très lisible (en même temps cela reste une question de goût personnel)...

L'elipse de Java 5.0 apporte une bien meilleure lisibilité dans ce cas

Pareil pour le classcast... mais je préfère préciser

a++

[Quetzalcoatl]

Beaucoup mieux, merci beaucoup