Discussion :

[ando0098]

Bonjour,

J'ai une question concernant les performances liées à la déclaration et l'instanciation des objets.
Je me demande si la déclaration d'un objet à chaque cycle de boucle est une bonne solution ou le contraire.

Code :

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

1
2
3
4
5
 
                String str;
		for (int i = 0; i < 10000000; i++) {
			str = new String("Bonjour");
		}

ou bien

Code :

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

1
2
3
4
 
		for (int i = 0; i < 10000000; i++) {
			 String str = new String("Bonjour");
		}

Merci d'avance.

[thelvin]

Déclarer un objet n'effectue rien du tout. Seule son affectation (str = quelquechose;) fait quelque chose.

Décider où on fait la déclaration est une question de propreté du code, pas de vitesse d'exécution. L'exécution sera la même.

Et donc, pour la propreté du code, les variables doivent être déclarées le plus tard possible.

(Bon, par contre, instancier avec new une nouvelle String de même valeur que les précédentes à chaque boucle, ça c'est très mauvais question performances.)

[ando0098]

concernant l'objet String, c'est juste un exemple. Ce que j'ai voulu savoir, est ce que la déclaration à l'intérieur de la boucle n'a pas d'effet sur la pile.

[thelvin]

Ma foi, je vais vérifier, mais il n'y a pas de raison que ça fasse une différence. Et le boulot du compilateur, c'est de s'en rendre compte et faire au mieux.

[adiGuba]

Salut,

concernant l'objet String, c'est juste un exemple. Ce que j'ai voulu savoir, est ce que la déclaration à l'intérieur de la boucle n'a pas d'effet sur la pile.

Tu peux vérifier cela en décompilant le code avec javap -c : le bytecode généré est identique (ou presque, puisque ne change que les index des variables).


Ce genre de truc c'est de l'optimisation de bout de chandelle...
Déjà avant de chercher à optimiser il faudrait cibler le problème. Sinon ce n'est que pur perte de temps source de problème potentielle à cause d'optimisation bidon !


Comme le dis thelvin le principal reste la lisibilité et la cohérence du code.Donc si la variable n'est utilisé que dans la boucle, autant la déclarée dedans...

A moins bien sûr que ce soit une constante dont la valeur est identique pendant toute la boucle.


a++

[Beginner.]

Bonsoir,

Question de débutant : ce genre de boucle n'alloue-t-il pas 10000000 d'emplacements pour un objet de type String ?

Si oui quel est l'intérêt ? Car en plus je suppose qu'on obtient uniquement une référence pour le dernier emplacement, non ?

[thelvin]

Question de débutant : ce genre de boucle n'alloue-t-il pas 10000000 d'emplacements pour un objet de type String ?

Ça instancie 10000000 objets String, mais vu que la référence vers chacun d'eux est perdue à la fin de la boucle, il devient immédiatement garbage-collectable et il n'est pas nécessaire de les garder tous en même temps.
A priori, donc, on ne peut pas dire que ça "alloue 10000000 emplacements".

Si oui quel est l'intérêt ?

Comme l'a dit ando0098, cet exemple n'a pas d'intérêt. Il veut simplement savoir si, pour l'exécution, il vaut mieux déclarer une variable objet qui sera écrasée à chaque boucle, à l'intérieur ou à l'extérieur de cette boucle.

Et la réponse est que ça n'a pas d'importance pour l'exécution, mais qu'il est plus propre pour le code, de la déclarer à l'intérieur.

Car en plus je suppose qu'on obtient uniquement une référence pour le dernier emplacement, non ?

Je sais pas trop ce que tu veux dire, mais à la fin de la boucle, il ne reste de référence que vers la dernière String créée, oui. Et encore, ça c'est si la String est déclarée à l'extérieur de la boucle. Si elle l'est à l'intérieur, plus aucune de ces String n'est atteignable après la boucle. Elles sont toutes garbage-collectables, même la dernière.