Généricité, ou presque..

Version imprimable

Hello,

J'ai un petit souci... J'ai un programme qui fonctionne bien! Tant mieux... Mais il est vraiment pas joli!

En gros, je fais un programme de configuration, qui prend des valeurs dans un XML (qui retourne des String) et qui doit les stocker dans une classe (grace au paquetage reflect)

Mon souci est le suivant: Quand je stock mes valeurs, je dois faire un traitement différent pour chaque type possible, et il y en a une floppée:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
    Class[] classes = {    boolean.class, Boolean.class, 
            byte.class, Byte.class, 
            short.class, Short.class, 
            int.class, Integer.class, 
            long.class, Long.class,
            float.class, Float.class,
            double.class, Double.class,
            String.class,
            boolean[].class, Boolean[].class,
            byte[].class, Byte[].class,
            short[].class, Short[].class,
            int[].class, Integer[].class,
            long[].class, Long[].class,
            float[].class, Float[].class,
            double[].class, Double[].class,
            String[].class};

Cela car je n'ai pas réussi (et pourtant j'ai essayé un bon moment) à envoyer un objet dans la méthode set de reflect... Attention à ne pas oublier les tableaux :lol:
Sinon il me met une exception du genre (pour u boolean):
java.lang.IllegalArgumentException: Can not set boolean field Configuration.Config01.booleanP to java.lang.String

Voici donc mon programme (une partie), si quelqun a une idée, je suis vraiment preneur!!

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
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
 
c=this.o.getClass();
 f=c.getFields();
...
public void endElement...    
 
    // Type of current field
    int intType = searchArray(classes, type);
    switch(intType){                    
        case BOOLEAN_PRIMITIVE: 
        case BOOLEAN: val = Boolean.parseBoolean(this.value.get(0)); break;
        case BYTE_PRIMITIVE:
        case BYTE: val = Byte.parseByte(this.value.get(0)); break;
        case SHORT_PRIMITIVE:
        case SHORT: val = Short.parseShort(this.value.get(0)); break;
        case INTEGER_PRIMITIVE: 
        case INTEGER: val = Integer.parseInt(this.value.get(0)); break;
        case LONG_PRIMITIVE: 
        case LONG: val = Long.parseLong(this.value.get(0)); break;
        case FLOAT_PRIMITIVE: 
        case FLOAT: val = Float.parseFloat(this.value.get(0)); break;
        case DOUBLE_PRIMITIVE: 
        case DOUBLE: val = Double.parseDouble(this.value.get(0)); break;
        case STRING: val = this.value.get(0); break;
        case BOOLEAN_P_ARRAY:
            boolean[] arrayBooleanP = new boolean[this.value.size()];
            for(int i=0;i<this.value.size();i++)
                arrayBooleanP[i]=Boolean.parseBoolean(this.value.get(i));
            val=arrayBooleanP;
            break;
        case BOOLEAN_ARRAY:
            Boolean[] arrayBoolean = new Boolean[this.value.size()];
            for(int i=0;i<this.value.size();i++)
                arrayBoolean[i]=Boolean.parseBoolean(this.value.get(i));
            val=arrayBoolean;
            break;
        case BYTE_P_ARRAY:
            byte[] arrayByteP = new byte[this.value.size()];
            for(int i=0;i<this.value.size();i++)
                arrayByteP[i]=Byte.parseByte(this.value.get(i));
            val=arrayByteP;
            break;
        case BYTE_ARRAY:
            Byte[] arrayByte = new Byte[this.value.size()];
            for(int i=0;i<this.value.size();i++)
                arrayByte[i]=Byte.parseByte(this.value.get(i));
            val=arrayByte;
            break;
        case SHORT_P_ARRAY:
            short[] arrayShortP = new short[this.value.size()];
            for(int i=0;i<this.value.size();i++)
                arrayShortP[i]=Short.parseShort(this.value.get(i));
            val=arrayShortP;
            break;
        case SHORT_ARRAY:
            Short[] arrayShort = new Short[this.value.size()];
            for(int i=0;i<this.value.size();i++)
                arrayShort[i]=Short.parseShort(this.value.get(i));
            val=arrayShort;
            break;
        case INTEGER_P_ARRAY:
            int[] arrayIntP = new int[this.value.size()];
            for(int i=0;i<this.value.size();i++)
                arrayIntP[i]=Integer.parseInt(this.value.get(i));
            val=arrayIntP;
            break;
        case INTEGER_ARRAY:
            Integer[] arrayInt = new Integer[this.value.size()];
            for(int i=0;i<this.value.size();i++)
                arrayInt[i]=Integer.parseInt(this.value.get(i));
            val=arrayInt;
            break;
        case LONG_P_ARRAY:
            long[] arrayLongP = new long[this.value.size()];
            for(int i=0;i<this.value.size();i++)
                arrayLongP[i]=Long.parseLong(this.value.get(i));
            val=arrayLongP;
            break;
        case LONG_ARRAY:
            Long[] arrayLong = new Long[this.value.size()];
            for(int i=0;i<this.value.size();i++)
                arrayLong[i]=Long.parseLong(this.value.get(i));
            val=arrayLong;
            break;
        case FLOAT_P_ARRAY:
            float[] arrayFloatP = new float[this.value.size()];
            for(int i=0;i<this.value.size();i++)
                arrayFloatP[i]=Float.parseFloat(this.value.get(i));
            val=arrayFloatP;
            break;
        case FLOAT_ARRAY:
            Float[] arrayFloat = new Float[this.value.size()];
            for(int i=0;i<this.value.size();i++)
                arrayFloat[i]=Float.parseFloat(this.value.get(i));
            val=arrayFloat;
            break;
        case DOUBLE_P_ARRAY:
            double[] arrayDoubleP = new double[this.value.size()];
            for(int i=0;i<this.value.size();i++)
                arrayDoubleP[i]=Double.parseDouble(this.value.get(i));
            val=arrayDoubleP;
            break;
        case DOUBLE_ARRAY:
            Double[] arrayDouble = new Double[this.value.size()];
            for(int i=0;i<this.value.size();i++)
                arrayDouble[i]=Double.parseDouble(this.value.get(i));
            val=arrayDouble;
            break;
        case STRING_ARRAY:
            String[] arrayString = new String[this.value.size()];
            for(int i=0;i<this.value.size();i++)
                arrayString[i]=this.value.get(i);
            val=arrayString;
            break;
    }
 
     f[index].set(this.o, val);
 
...

Ah, et BOOLEAN_PRIMITIVE, .... sont des constantes entières qui référencent le tableau dans le code en dessus... (des indices de tableaux)

Et la, je voudrais ajouter une vérification sur le range de val... Donc je dois l'ajouter 50 millions de fois... Vraiment pénible...

Help please :?

18/04/2007, 13h17
adiGuba

Salut,

Citation:

Envoyé par ploxien

Et la, je voudrais ajouter une vérification sur le range de val...

Traduction ????

Que veux-tu faire exactement ? Ce n'est pas clair du tout !
Penses que nous ne sommes pas plongé dans ton code comme toi tu peux l'être !

a++
18/04/2007, 13h21
ploxien

Ouais, t'as raison, désolé...

Mais le truc du range, c pas important...

Ce que je veux, c'est vraiment simplifier ce code! Faire un traitement générique pour n'importe quel type! Si c'est possible...

Merci ;)
18/04/2007, 13h26
adiGuba

Citation:

Envoyé par ploxien

Ce que je veux, c'est vraiment simplifier ce code! Faire un traitement générique pour n'importe quel type! Si c'est possible...

Mais que doit faire ce code ??
Qu'est-ce qu'il reçoit en entrée ? A quoi correspondent les variables classes, type, this.value, etc...

Comment pourrait-on simplifier un code sans savoir ce qu'il fait !?!

a++
18/04/2007, 13h35
ploxien

Je précise:
Pour chaque type, je fait cela:

- Je prend this.value.get(0), ce qui me donne la String qui provient du XML (p.ex, "true", "3.4", "abc"). Donc n'importe quel type est retourné en String...

- Ensuite je transforme cette valeur dans le type de destination (ici Byte)

Code:

val = Byte.parseByte(this.value.get(0));

- Ensuite je fais un

Code:

f[index].set(this.o, this.value.get(0));

ce qui stock la valeur (prise précédement dans le XML) dans une variable d'une autre classe (java.lang.reflect). C'est ici que se situe le prob, l'objet que j'envoie doit être du bon type (ouais, forcément).

Donc ce que je voudrais faire, c'est ça: j'ai un type dans une variable du genre de Class type=...; Je voudrais donc transformer un String dans ce type, qu'il soit n'importe quoi!

C'est plus clair?? Encore des précisions??

Merci
18/04/2007, 13h39
ploxien

Donc remplacer tout ca par un truc du genre:

Code:

val=classes[intType].cast(this.value.get(0));

(classes[intType] vaut int.class, Byte.class, Boolean.class, boolean.class, ...)
Sauf que ca ca me donne un exception:java.lang.ClassCastException
at java.lang.Class.cast(Unknown Source)

:bug:
18/04/2007, 13h44
chtig

Il me semble avoir compris ton souci.
L'idée, c'est qu'en entrée tu as une liste de String, et que tu désires utiliser cette liste pour setter la valeur d'un attribut dans une classe qui peut être à peu près de n'importe quel type (tableau ou non).

Je n'ai pas vraiment d'idée plus "propre" que ce que tu as fait. Mais si j'étais à ta place, j'irais jeter un coup d'oeil au code Struts qui permet de populer un Form à partir d'une requête. Ils ont du avoir la même problématique (les attributs du Form peuvent être de type quelconque et les valeurs dans la requête sont sous la forme chaîne de caractères). Ils ont peut être eu une idée sympa 8-)

Edit : il semblerait que tes précisions pendant que je répondais confirme ce que j'avais compris :D
18/04/2007, 13h48
ploxien

Oui, c'est exactement ça!!

Je vais regarder du coté de Struts...
Merci pour l'info et certainement à bientôt ;)

++
18/04/2007, 14h08
g0up1l

Et oui, l'éternel fantasme du constructeur virtuel...tellement pratique...pouvoir instancier un objet sans connaître son type !

Au pire une solution consiste à rajouter dans ton fichier XML une info supplémentaire sur le type de la valeur.
Cette info peut-être le type même de la valeur juxtaposé à la valeur mais pas uniquement. On peut aussi mettre en place un canevas. Par exemple, on sait que l'on va trouver dans le fichier : 3 'String' puis deux 'int' puis 3 'float'
18/04/2007, 14h12
ploxien

Héhé et oui...

Mais en fait, le truc, c'est que le type je le connais, c'est classes[intType]. Ca me retourne un "Class"... Donc je l'ai, c'est juste que c'est pas toujours le même...

Tu vois ce que je veux dire?

Merci

Citation:

Envoyé par ploxien

C'est plus clair??

Je pense : tu veux convertir des données de type String vers le type de l'attribut de ta classe, pour ensuite pouvoir l'affecter sans ClassCastException...

C'est normal que la méthode cast() ne fonctionne pas, car un cast se contente de changer le type de la référence de l'objet, mais pas le type lui même. Bref tu es obligé de convertir l'objet.

Le plus simple pour éviter de faire de gros switch/case c'est de définir une interface pour les objets qui se chargeront de cette conversion, par exemple :
Code:

1 2 3 public interface Parser<T> { public T parse(String source); }
L'idée étant d'utiliser une classe pour la conversion des données. Cette classe contiendra une Map affectant un Parser à un type Java. Le constructeur de cette classe pourrait définir un certain nombre de parser par défaut.
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 29 30 31 32 33 34 35 class Converter { private final Map<Class,Parser> parsers = new HashMap<Class, Parser>(); // Ajout d'un parser pour un type public <T> void addParser(Class<T> type, Parser<T> parser) { this.parsers.put(type, parser); } // Recher d'une parser selon le type public <T> Parser<T> getParser(Class<T> type) { return (Parser<T>) this.parsers.get(type); } public Converter() { // Ajout de parser par défaut // Par exemple pour Integer et int : // Creation du parser : Parser<Integer> intParser = new Parser<Integer>() { public Integer parse(String source) { return Integer.parseInt(source); } }; // Association avec les type Integer et int : addParser(Integer.class, intParser); addParser(int.class, intParser); // etc. pour les autres types } ...
Ensuite il suffit de faire une méthode de conversion qui utilisera le bon parser selon le type :
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 29 30 31 32 33 34 35 36 37 38 39 public <T> T convert(Class<T> type, List<String> value) throws Exception { Object result = null; if (type.isArray()) { // Cas particulier pour les tableau // 1. On récupère le parser pour le type de base du tableau : Parser parser = getParser(type.getComponentType()); if (parser==null) { throw new Exception("No parser for type " + type.getComponentType()); } // 2. On crée le tableau via la reflection : result = Array.newInstance(type, value.size()); Object[] array = (Object[]) result; // 3. On affecte chaque élément du tableau après conversion : int i=0; for (String source : value) { array[i] = parser.parse(source); i++; } } else { // Cas standard (affectation simple) Parser parser = getParser(type); if (parser==null) { throw new Exception("No parser for type " + type); } result = parser.parse(value.get(0)); } return ignoreTypeSafetyWarning(result); } @SuppressWarnings("unchecked") private <T> T ignoreTypeSafetyWarning(Object o) { return (T) o; // warning type safety qui peut être ignoré }
Citation:

Envoyé par ploxien

Encore des précisions??

En fait tu nous as dit ce que tu voulais faire... mais on ignore encore pourquoi ! Bref s'il existe des solutions toutes faites on ne peut pas te les donner car on ignore le but exact de tout cela mis à part la conversion des types.

Il ne faut pas hésiter à donner un max d'info pour améliorer la qualité des réponses que l'on obteindra ;)

a++

18/04/2007, 14h42
g0up1l

Ben, si tu as l'objet Class c'est bien partit !
tu as la méthode 'getConstructor(null)' pour obtenir un constructeur sur cette classe, il te restera un 'newInstance(null)' à faire pour avoir ton instance et c'est bon !
18/04/2007, 14h44
ploxien

Ce que je dois faire:

Un programme de gestion de configuration.
J'entend pas la, que j'aurai une classe avec des valeurs par défaut et que quand je l'instancie elle va chercher des valeurs de configuration dans un XML...

ALors si tu connais une autre façon de faire... C'est volontiers!

C'est du genre de wwAppConfiguration en .NET...

A+ et merci!
18/04/2007, 15h22
chtig

J'aime bien l'idée des Parser avec les génériques. Effectivement, ça permet de bien dissocier la partie "généricité" de la partie traitement...

Sinon, ben en termes d'idées, pas mieux en ce qui me concerne pour remplir cette tâche de loader de configuration ...

Citation:

Envoyé par ploxien

Un programme de gestion de configuration.
J'entend pas la, que j'aurai une classe avec des valeurs par défaut et que quand je l'instancie elle va chercher des valeurs de configuration dans un XML...

Tu peux utiliser les classes XMLDecoder/XMLEncoder pour "sérialiser" tes classes en XML.

Par exemple avec la classe suivante :

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
29
30
public class MyObject {
	private String name;
	private int[] value;
	private boolean state;
 
 
	public String getName() {
		return name;
	}
	public void setName(String name) {
		this.name = name;
	}
	public boolean isState() {
		return state;
	}
	public void setState(boolean state) {
		this.state = state;
	}
	public int[] getValue() {
		return value;
	}
	public void setValue(int[] value) {
		this.value = value;
	}
 
	@Override
	public String toString() {
		return name + " " + Arrays.toString(value) + " " + state;
	}
}

Tu génères une fois le fichier XML correspondant aux valeurs par défaut :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
		// Création d'un objet en lui associant des attributs :
		MyObject obj = new MyObject();
		obj.setName ("hello");
		obj.setValue (new int[]{15, 0, 22});
		obj.setState (true);
 
		// On le sauvegarde en XML :
		XMLEncoder encoder = new XMLEncoder(new FileOutputStream("out.xml"));
		try {
			encoder.writeObject(obj);
		} finally {
			encoder.close();
		}

Cela te générera un fichier XML du style :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
<?xml version="1.0" encoding="UTF-8"?> 
<java version="1.5.0_10" class="java.beans.XMLDecoder"> 
 <object class="MyObject"> 
  <void property="name"> 
   <string>hello</string> 
  </void> 
  <void property="state"> 
   <boolean>true</boolean> 
  </void> 
  <void property="value"> 
   <array class="int" length="3"> 
    <void index="0"> 
     <int>15</int> 
    </void> 
    <void index="2"> 
     <int>22</int> 
    </void> 
   </array> 
  </void> 
 </object> 
</java>

Ensuite il suffit d'utiliser un XMLDecoder pour récupérer l'objet en question, exemple :

Code:

1
2
3
4
5
6
7
8
9
	@SuppressWarnings("unchecked")
	public static <T> T loadFromXML(String fileName) throws FileNotFoundException {
		XMLDecoder decoder = new XMLDecoder(new FileInputStream("out.xml"));
		try {
			return (T) decoder.readObject(); // unchecked cast
		} finally {
			decoder.close();
		}
	}

Que tu peux utiliser de la manière suivante :

Code:

MyObject another = loadFromXML("out.xml");

a++

C'est incroyable!!!

En tout cas merci bcp!!

Sinon, lors du encoder.writeObject(obj);, il n'écrit que les valeurs des attributs modifiés (obj.setName ("hello");, ...). Ne serait il pas possible de générer le XML en prenant les valeurs par défaut définies dans l'objet?

Objet:

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
29
30
31
32
33
public class Config01{
 
    private boolean booleanP = false;
    private Boolean booleanS = false;
 
    private byte byteS = 0;
 
    private Byte byteP = 0;
 
    private short shortS = 0;
    private Short shortP = 0;
 
    private int integerS = 0;
    private Integer integerP = 0;
 
    private long longS = 0;
 
    private float floatS = 0;
 
    private double doubleS = 1.0;
    private Double doubleP = 1.0;
 
    private String string = "aaa";
 
    private int[] tabIntP = {1,2,3};
 
    private Double[] tabDoubleS = {1.1,2.2,3.3};
 
    private String[] tabString = {"I","am","nothing"};
...
}
 
Merci d'avance et A+

19/04/2007, 09h06
adiGuba

Citation:

Envoyé par ploxien

Sinon, lors du encoder.writeObject(obj);, il n'écrit que les valeurs des attributs modifiés (obj.setName ("hello");, ...). Ne serait il pas possible de générer le XML en prenant les valeurs par défaut définies dans l'objet?

Non je ne crois pas...

Par contre si tu veux manipuler plus précisément le fichier XML, tu peux te tourner vers des APIs externes comme XStream (voir le tutoriel).

a++

PS : c'est pas pour rien que je demandais plus de détail ;)