Discussion :

[MatP]

Pour parcourir une collection (arrayList dans mon cas), il y a les trois méthodes suivantes :

- Avec l'index de l'item
- Avec For Each
- Avec l'objet IEnumerator

Bon, c'est bien tout ça, mais c'est quoi l'avantage d'utiliser plus une méthode qu'une autre ? En utilisant For Each, est-ce que ça va les passer dans l'ordre indexé ? Est-ce plus rapide avec l'enumerator?

Pour l'instant, j'utilise toujours la première méthode.

Merci

[neguib]

Je vote pour la simplicité du For Each. Pour ma part, le vrai souci avec un Arraylist c'est le fait de récupérer un objet non typé. Ayant le framework 2.0, pourquoi se priver des générics ?

[del-dongo]

le foreach n'est qu'une facilité du langage, derrière il n'y a rien d'autre que

Code :

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

1
2
 
   IEnumerator.MoveNext()

donc pour la simplicité, c'est vrai, le foreach...
ps : il apparaitrait qu'une boucle for serait plus appropriée si l'on recherche de hautes performances, mais dans la majorité des cas il n'y a presque pas de différrences....

[Piotrek]

Ienumerator et For each c'est idem

Apres lequel choisir, bin tout depend du code:

Code :

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

1
2
3
4
5
6
7
8
 
For i=0 to n
    Collec(i).truc = b
    Collec(i).bidule = c
    Collec(i).fairekekchose
    Collec(i).truc = c
    Collec(i).machin = d
Next

Bin la forcement on repasse a chaque fois par l'appel a la collection, la recup de l'objet par son index: dans ce cas vallait mieux faire un for each

Code :

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

1
2
3
4
 
For Each truc in Collec
    truc.fairekekchose
Next

Rappatrier une reference dans truc pour fairekekchose n'est pas utile. Vallait mieux passer par l'index dans ce cas ci.

[Maniak]

Code :

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

1
2
3
4
5
6
7
For i=0 to n
    Collec(i).truc = b
    Collec(i).bidule = c
    Collec(i).fairekekchose
    Collec(i).truc = c
    Collec(i).machin = d
Next

Bin la forcement on repasse a chaque fois par l'appel a la collection, la recup de l'objet par son index: dans ce cas vallait mieux faire un for each

Là de toute façon, la première chose à faire est de déclarer une variable à la première ligne pour récupérer l'objet directement. La duplication, c'est Mal :)
(et là c'est aussi un exemple de méthode manquante sur la classe des objets contenus dans la collection :)

Code :

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

1
2
3
For Each truc in Collec
    truc.fairekekchose
Next

Rappatrier une reference dans truc pour fairekekchose n'est pas utile. Vallait mieux passer par l'index dans ce cas ci.

Pas spécialement utile non plus. Il n'y a pas de différence de perfs et

Code :

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

1
2
3
4
foreach ( Bleh truc in machins )
{
  truc.Chose();
}

est toujours plus lisible que

Code :

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

1
2
3
4
for ( int i = 0 ; i < machins.Count ; ++i )
{
  machins[i].Chose();
}

Le plus utile des deux est le plus lisible. Et le for gagne rarement à ce jeu :)

De manière générale, si on a juste à parcourir linéairement, du premier au dernier, sans avoir besoin de l'index -> foreach.
Si on a besoin de l'index, qu'on ne va pas de un en un ou qu'on ne démarre pas du début -> for.

De manière encore plus générale, tant que foreach fait l'affaire -> foreach.
Quand on tombe sur un cas où ça ne va pas, on utilise autre chose.

La question des performances est complètement ignorable.

[Piotrek]

La question des performances est complètement ignorable.

Tolérable, nuance! on ignore pas: on tolère

[Maniak]

La question des performances est complètement ignorable.

Tolérable, nuance! on ignore pas: on tolère :P

On ignore jusqu'à ce que ça pose un problème. Et le commun des mortels ne verra probablement jamais cette situation se produire. Donc on ignore, on n'y pense plus, et ça fait un peu plus de place dans le crâne pour stocker des choses vraiment utiles :)

Où est le problème d'ignorer quelque chose de négligeable ? :)

[Piotrek]

Tu pars vraiement au 1/4 de tour

Bien sur je suis d'accord, et j'en profite pour passer le lien qui va bien pour ceux qui connaissent pas:
http://msdn.microsoft.com/library/default.asp?url=/library/en-us/dnpag/html/scalenet.asp

[Maniak]

Tu pars vraiement au 1/4 de tour :D

1/8
Mais juste 1/6 en fin de semaine :)

[MatP]

Ok, j'utilisais pas le For Each quand j'avais besoin de parcourir ma collection dans l'ordre des index, car me semble qu'une fois j'avais fait ça et le For Each n'avait pas parcouru ma collection dans l'ordre que je m'attendais.

Mais si vous dites qu'un for Each va parcourir dans le même ordre qu'un for de 0 à .count - 1, ça fait mon affaire aussi.

Merci pour les réponses