[Sonar] Comment corriger un "Avoid instantiating objects in loops" ?

Version imprimable

07/10/2011, 11h21
womannosky

[Sonar] Comment corriger un "Avoid instantiating objects in loops" ?
Bonjour à tous,

J'utilise Sonar (version 2.2) et j'ai un nombre important de violations intitulées "Avoid instantiating objects in loops" mais je n'ai aucune idée de comment les corriger.
J'instancie mes objets dans une boucle car j'ai besoin de créer un nouvel objet à chaque itération de boucle.

Voici mon code :
Code:

1 2 3 4 5 List liste = null; for (int i = 0; i < limite; i++) { liste = new ArrayList(); }
Pour faire sauter la violation, je ne peux pas réutiliser ma liste comme ci-dessous car j'ai besoin à chaque itération de boucle d'une nouvelle liste :
Code:

1 2 3 4 5 6 7 List liste = new ArrayList(); for (int i = 0; i < limite; i++) { liste.clear(); liste.setObjet(); ... }
Auriez-vous des idées pour corriger ce type de violations ?

Une idée comme ça, est-ce que le design pattern fabrique pourrait m'aider ?
09/10/2011, 19h03
miaous
Bonjour

Dans ton code la variable list est affecté au début de chaque itération de la boucle , et elle ne doit pas être lu en dehors de la boucle.

Ici la durée de vie de ta variable est équivalent à un parcours de ta boucle , Dans ces cas là ,il est conseillé en général de la déclarer dans la boucle, comme ceci :
Code:

1 2 3 4 5 for (int i = 0; i < limite; i++) { ArrayList liste = new ArrayList(); ... }
Dans ton exemple ,où tu utilise une ArrayList qui posséde un clear , il vaut mieux faire comme tu la écris dans ton code pour des questions de performance ( Tu créera et détruira moins d’objet ... ).

Citation:
Envoyé par miaous
Ici la durée de vie de ta variable est équivalent à un parcours de ta boucle , Dans ces cas là ,il est conseillé en général de la déclarer dans la boucle, comme ceci :
Code:

1 2 3 4 5 for (int i = 0; i < limite; i++) { ArrayList liste = new ArrayList(); ... }
Cela ne fait pas sauter la violation Sonar :?. De plus, pour des problèmes de performances, n'est-ce pas encore pire de créer l'objet dansl a boucle en plus de l'instancier ? C'est pourquoi j'ai sorti la création de l'objet de la boucle.

Citation:

Envoyé par miaous

Dans ton exemple ,où tu utilise une ArrayList qui posséde un clear , il vaut mieux faire comme tu la écris dans ton code pour des questions de performance ( Tu créera et détruira moins d’objet ... ).

Ou quoi ? Quelle est la deuxième solution que tu préconises ?

Mon exemple n'est pas assez parlant. En voici un nouveau avec une méthode qui supprime des utilisateurs en base de données (seulement les utilisateurs cochés dans un tableau par l'utilisateur) :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 /** * Supprime les utilisateurs cochés dans le tableau par l'utilisateur. * * @param utilisateursCoches * les utilisateurs cochés à supprimer */ public void suppressionUtilisateurs(List<UtilisateurBean> utilisateursCoches) { UtilisateurDTO utilisateurASupprimer = null; for (UtilisateurBean util : utilisateursCoches) { // On crée l'élément à supprimer utilisateurASupprimer = new UtilisateurDTO(); utilisateurASupprimer.setUser(util.getUser()); utilisateurASupprimer.setNom(util.getNom()); utilisateurASupprimer.setPrenom(util.getPrenom()); // Supprime l'utilisateur en base supprimerUtilisateur(utilisateurASupprimer); } }
Ici, je suis obligé d'instancier l'utilisateur à supprimer dans la boucle et Sonar me sort donc une violation de type "Avoid instantiating objects in loops" sur la ligne :
Code:

1 2 utilisateurASupprimer = new UtilisateurDTO();
Comment puis-je dans ce cas éviter la violation Sonar ?

Citation:

Envoyé par womannosky

Comment puis-je dans ce cas éviter la violation Sonar ?

Comme ceci ?

Code:

1
2
3
4
5
6
7
8
9
10
11
public void suppressionUtilisateurs(List<UtilisateurBean> utilisateursCoches) {
	UtilisateurDTO utilisateurASupprimer = new UtilisateurDTO();
	for (UtilisateurBean util : utilisateursCoches) {
		utilisateurASupprimer.setUser(util.getUser());
		utilisateurASupprimer.setNom(util.getNom());
		utilisateurASupprimer.setPrenom(util.getPrenom());
 
                // Supprime l'utilisateur en base
                supprimerUtilisateur(utilisateurASupprimer);
	}
}

17/10/2011, 09h33
womannosky

Bonjour Hervé et merci pour la réponse.

Ce code fonctionnerait si je supprimais directement les utilisateurs en base de données. Cependant, je garde ces objets (utilisateurs) dans une autre liste par la suite avant de tous les supprimer (l'une de mes contraintes). Si j'utilisais ce code, j'aurais dans ma liste tous les mêmes objets égaux. Leur user, prénom et nom serait ceux du dernier utilisateur parcouru dans la boucle "for" (puisque je modifie les références des objets à chaque itération).

Ma question s'en détache : Est-il possible de ne pas créer l'objet utilisateur dans la boucle "for" si je dois garder ces utilisateurs par la suite ?
J'ai l'impression que la réponse à cette question devient de plus en plus évidente, cela doit être "non". :(

Citation:

Envoyé par womannosky

Ce code fonctionnerait si je supprimais directement les utilisateurs en base de données. Cependant, je garde ces objets (utilisateurs) dans une autre liste par la suite avant de tous les supprimer.

En effet, c'est problématique :cry:
Bon, si c'est juste pour contourner Sonar, et dans l'hypothèse où cet outil ne fait pas une analyse statique "trop" approfondie, je propose ceci :mouarf:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
public void suppressionUtilisateurs(List<UtilisateurBean> utilisateursCoches, UtilisateurDTOFactory utilisateurDTOFactory) {
	for (UtilisateurBean util : utilisateursCoches) {
		UtilisateurDTO utilisateurASupprimer = utilisateurDTOFactory.newInstance();
		utilisateurASupprimer.setUser(util.getUser());
		utilisateurASupprimer.setNom(util.getNom());
		utilisateurASupprimer.setPrenom(util.getPrenom());
 
                // Supprime l'utilisateur en base
                supprimerUtilisateur(utilisateurASupprimer);
	}
}
 
public interface UtilisateurDTOFactory {
	UtilisateurDTO newInstance();
}
 
...
 
gestionnaireUtilisateurs.suppressionUtilisateurs(utilisateursCoches, new UtilisateurDTOFactory() {
	public UtilisateurDTO newInstance() {
		return new UtilisateurDTO();
	}
});

voire

Code:

1
2
3
4
5
6
7
 
private UtilisateurDTOFactory utilisateurDTOFactory; // injecté par Spring par exemple
 
public void suppressionUtilisateurs(List<UtilisateurBean> utilisateursCoches) {
	for (UtilisateurBean util : utilisateursCoches) {
		UtilisateurDTO utilisateurASupprimer = utilisateurDTOFactory.newInstance();
...

19/10/2011, 09h53
womannosky

Bonjour hervé,

Merci pour la réponse qui m'est plus que satisfaisante :ccool:. En effet, cela fonctionne grâce à la Factory.