Discussion :

[grunk]

Bonjour à tous,

J'ai une vue custom pour une appli android qui est assez complexe et est amenée à se redessiner souvent.

Dans cette vue je fait beaucoup de calcul du genre :

Code :

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

1
2
3
int nbSeconde = ((i * interval) * MAX) / mUsableWidth;
int heure = nbSecond / 3600;
c.drawText(String.format("%02d:00",heure), getWidth - 150, getHeight,paint);

J'ai donc deux variable temporaires (nbSeconde , heure ) qui sont là uniquement pour la lisibilité du code.

Sachant que ce cas de figure est fréquent dans mon code et qu'il est appelé très souvent (redraw de la vue) est ce que ça vaut le coup de perdre en lisibilité pour gagner sur ces 2 allocations temporaire et donc limiter les passage du GC?

[adiGuba]

Salut,


Je doute fortement que cela ait un quelconque impact...


Par contre comment sont définies les éléments de ton calcul : ((i * interval) * MAX) / mUsableWidth ?
S'il y a des constantes dans le lot tu peux gagner à les déclarer comme tel (en final donc).


a++

[thelvin]

Le compilateur se fiche royalement qu'on définisse une variable pour chaque chose ou qu'on écrive la totalité sur une seule ligne sans aucun nom. Il calcule ce qu'on lui demande de calculer, que ce soit dans une variable ou en ligne n'y change rien.

Par ailleurs, "éviter le passage du GC," où ça un GC ? Ce sont des int, pas des objets. Ils existent soit dans la stack, soit dans un objet qui est déjà là de toute façon. Ici, vu que ce sont des variables locales, dont tu avais remarqué d'ailleurs que tu pouvais aussi bien ne pas les déclarer comme variables, elles sont dans la stack.
Il n'y a rien à collecter, la stack se dépilera comme elle se dépile d'habitude.