Discussion :

[raf64flo]

Bonjour, depuis un moment je cherchais à afficher l'état d'un objet en affichant la liste de ses champs et leur valeur en surchargeant la méthode toString() (ou en utilisant un autre nom de méthode).

Je me suis penché sur le problème aujourd'hui, et j'en suis arrivé au code suivant. Je suis dans le cas où j'utilise des objets qui héritent d'une classe mère qui contient, elle, les attributs intéressants.

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
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
	@Override
	public String toString() {
		String thisName = this.getClass().getSuperclass().getName();
		String objectName = thisName.substring(thisName.lastIndexOf("."), thisName.length());
 
		StringBuffer buf = new StringBuffer();
		buf.append("--- Object " + objectName + " Begin ----\n");
 
		Field[] fields = this.getClass().getSuperclass().getDeclaredFields();
 
		for (int index = 0; index < fields.length; index++) {
			Field field = fields[index];
			String value = null;
			try {
				Object obj = field.get(this);
				if (obj != null)
					value = (String) obj.toString();
				else
					value = null;
			}
			catch (IllegalArgumentException e) {
				// TODO Auto-generated catch block
				e.printStackTrace();
			}
			catch (IllegalAccessException e) {
				// TODO Auto-generated catch block
				e.printStackTrace();
			}
			if (value != null)
				buf.append(field.getName() + ": " + value.toString() + "\n");
			else
				buf.append(field.getName() + ": NULL\n");
		}
		buf.append("--- Object " + objectName + " End ----\n");
 
		return buf.toString();
	}

SI vous avez des suggestions pour améliorer ce code et le rendre encore plus générique (par exemple en allant chercher les attributs de l'objet courant si besoin, ou d'objet de classes supérieures), c'est pas de refus.

En tout cas, cette portion de code est là pour ceux à qui elle conviendrait. Une fois le code optimisé, il pourrait être intéressant de l'ajouter à la FAQ ?

Existe-t-il une méthode plus simple pour en arriver au même résultat ?

[adiGuba]

Salut,


toString() n'est pas trop adapté à cela je trouve...

Sinon tu peux faire quelque chose d'encore plus générique avec l'introspection :

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
	public static void print(Object object) throws IntrospectionException {
		String objectName = object.getClass().getSimpleName();
		BeanInfo info = Introspector.getBeanInfo(object.getClass());
		System.out.println("--- Object " + objectName + " Begin ----");
		for (PropertyDescriptor property : info.getPropertyDescriptors()) {
			String name = property.getDisplayName();
			Object value = null;
			try {
				value = property.getReadMethod().invoke(object);
			} catch (Exception e) {
				value = "Erreur - " + e.getMessage();
			}
			System.out.println(name + ": " + value);
		}
		System.out.println("--- Object " + objectName + " End ----");
	}

Cela te récupère toutes les valeurs en utilisant les accesseurs

a++

[Heimdal]

Sinon tu peux utiliser la classe ToSringBuilder d'Apache commons-lang

[Pill_S]

Sinon tu peux utiliser la classe ToSringBuilder d'Apache commons-lang

+1

ça évite de se casser la tête...

[raf64flo]

Eh bien, visiblement, je me suis compliqué la vie.

Je regarderai ces deux propositions dés que possible, merci bien.

[dbugfree]

Eh bien, visiblement, je me suis compliqué la vie.

Je regarderai ces deux propositions dés que possible, merci bien.

Et bien merci de t'être compliqué la vie, car moi ça va me servir

En fait j'ai un soucis avec BeanUtils.describe() et ToStringBuilder, ils tournent en rond vu mes objets métiers bidirectionnels (dossier<->client)

Après pourquoi n'est-ce pas adapté a toString() ? J'ai plutôt l'impression du contraire (transformer un objet en String) ...

[Rei Ichido]

On a rarement besoin de tous les champs pour identifier un objet.

- Ou bien il s'agit de logger des choses, auquel cas c'est à charge du codeur de savoir ce qu'il veut logger (et c'est très peu probablement toutes les propriétés de l'objet) ;

- Ou bien c'est pour faciliter le mode debug, mais là encore pas vraiment besoin de voir toutes les propriétés, ça peut rapidement rendre illisible le toString.

[dbugfree]

Je n'utilise pas toString pour identifier un objet, mais plutot le getter de son id/numéro. Et le mode debug sert a loguer toutes les données en général, du moins dans les appli sur lesquels je bosse et dans les DAO. Sinon je vois pas trop l'intérêt de créer un toString restreint en fait, car comme tu dis si on veut juste quelques données il suffit de les loguer individuellement.

Et pour ce qui est de ne pas rendre un toString illisible, j'ai finalement trouvé le ToStringBuilder vraiment bien une fois associé avec un ToStringStyle ! On peut dans ce cas formater les données désirées sans utiliser le toString des sous-objets via un test de la classe et l'utilisation du getter de l'id par exemple

sources :
http://commons.apache.org/lang/api/o...ngBuilder.html
http://commons.apache.org/lang/api/o...ringStyle.html