Discussion :

[pierre-y]

Bonsoir,
J'ai une question un peu bête sur les bonnes pratiques en programmation et c'est un peu du pinaillage mais bon. Je code en ce moment en CSharp et je dois souvent faire des bouclages de tableau.

Code :

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

1
2
3
4
5
for (int i=0 ; i<=worldSize.x-1 ; i++)
{
       //
}

La question porte sur "worldSize.x-1" a savoir si ça n'a pas d'importance de le mettre comme ça ou si c'est plus efficace de l'initialiser dans une variable avant? Comme ceci :

Code :

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

1
2
3
4
5
6
7
int size_x = worldSize.x-1;

for (int i ; i<=size_x ; i++)
{
       //
}

Oui je sais c'est pas grandiose comme question^^, mais je me pose la question depuis tellement longtemps.

Bonne soirée,

[Sun.Wu.Kong]

Dans le premier cas worldSize.x - 1 est calculé à chaque itération de ta boucle. Donc, ça fait du temps de calcul.

Dans le secong cas, worldSize.x - 1 n'est calculé qu'une seule fois.

Sur de petites boucles, ce n'est pas très important l'une ou l'autre. Sur plusieurs millions d'itérations, cela peut te faire gagner du temps de prendre la seconde solution.

[wolinn]

pierre-y :

Comme déjà dit, avec la première écriture, worldSize.x-1 risque d'être recalculé à chaque itération, encore qu'un compilateur moderne devrait être capable de s'apercevoir que worldSize.x est invariant, et ne faire le calcul qu'une fois.
La seconde solution introduit une nouvelle variable, ce n'est pas particulièrement clair et élégant.

J'aurais écrit :
for (i = 0; i < worldSize.x; i++)
...

Après, c'est une question de goûts et d'habitudes, d'autres n'aiment pas non plus cette écriture parce que l'indice worldSize.x figurant dans la définition de la boucle n'est jamais atteint.

[dourouc05]

Dans pas mal de langages, tu as des itérateurs et des boucles de type for-each (range en Python, par exemple). Pour moi, ça serait le meilleur style à employer ici. Surtout que, apparemment, ça existe en C# (http://stackoverflow.com/questions/5...-integer-range).

Dans un autre langage, Julia, on a même une fonction pour itérer sur tous les index d'un tableau : eachindex().

Ces deux techniques donneraient ça en Julia :

Code julia :

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

1
2
3
4
5
6
7
for i in 1:worldSize.x
    # ...
end
 
for i in eachindex(worldSize)
    # ...
end

Par contre, réfléchir en termes d'optimisation du compilateur, pour moi, ce n'est tout simplement pas important dans la plupart des applications — sans oublier qu'un compilateur moderne est très intelligent.

[pierre-y]

Merci pour tout ces avis.

Après, c'est une question de goûts et d'habitudes, d'autres n'aiment pas non plus cette écriture parce que l'indice worldSize.x figurant dans la définition de la boucle n'est jamais atteint.

Et ça peut poser probleme que worldSize.x ne soit jamais atteint?

[wolinn]

Et ça peut poser probleme que worldSize.x ne soit jamais atteint?

Absolument aucun, c'est juste éventuellement une question de lisibilité.

for (i = 0; i < worldSize.x; i++)
...
est bien équivalent à :
for (i = 0; i <= worldSize.x-1; i++)
...

En ce qui me concerne, je préfère la "sobriété" de la première forme, mais on peut aussi considérer que la seconde est plus lisible, parce qu'on voit directement que i varie de 0 à worldSize.x-1 inclus.

[pierre-y]

Merci pour ces précision. Bonne soirée,

[Médinoc]

On peut aussi considérer que la première forme est plus lisible, car plus utilisée et donc plus "attendue": Pratiquement personne n'utilise la seconde forme, ce qui cause une "pause" dans la lecture du code par un être humain.