Discussion :

[pot2yaourt]

Bonjour à tous,

Voilà, aujourd'hui j'ai une classe MaClasse que je stocke dans une List<MaClasse> et ensuite j'ai plusieurs méthodes qui parcours cette liste afin de faire quelques modifications au sein de plusieurs éléments, du style :

Code :

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

1
2
3
4
5
6
7
foreach(MaClasse m in MaListeDeMaClasse)
{
    if(m.Propriété_1 == Id_recherché)
    {
        m.Propriété_2 = valeur;
    }
}

Je me demandais donc s'il n'était pas plus "optimisé" de passer tout simplement à un tableau à plusieurs dimensions que je pourrais manipuler ainsi :

Code :

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

1
2
tab['id']['propriété_1'] = valeur_1;
tab['id']['propriété_2'] = valeur_2;

Ceci éviterait du coup de parcourir la liste à chaque coup étant donné que je dispose déjà de l'id.

Du coup, j'aurai aimé savoir comment le déclarer un tel tableau et comment ajouter les éléments.

Merci d'avance pour votre aide !
Lionel.

[ostenhard]

Bonjour,

Il y a également la possibilité d'utiliser un dictionnaire de type Dictionary<Type de l'identifiant, MaClasse>.

Il sera alors possible d'accéder à l'élément correspondant à l'id quasiment comme vous le souhaitez :

Code :

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

1
2
 
monDictionnaire[idRecherché].LaPropriété = valeur;

Edit : Attention, dans un dictionnaire un id ne peut être en doublon

[Sehnsucht]

Bonjour,

Pour compléter la réponse précédente; quand la "clef" appartient à l'objet j'ai tendance à utiliser KeyedCollection<TKey, TItem> (sous certaines réserves dépendant du cas concret)

Cordialement !

[Bluedeep]

Il est possible qu'il y ait un problème de design ici.

Sous réserve de voir le besoin fonctionnel précis, je ne vois pas a priori la finalité de conserver en dehors des instances les valeurs des propriétés.

[tomlev]

Je me demandais donc s'il n'était pas plus "optimisé" de passer tout simplement à un tableau à plusieurs dimensions que je pourrais manipuler ainsi :

Code :

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

1
2
tab['id']['propriété_1'] = valeur_1;
tab['id']['propriété_2'] = valeur_2;

D'un point de vue POO, c'est une aberration... tu sors les données des objets auxquels elles appartiennent, ça va être inmaintenable ! En plus tu perds le typage des propriétés avec ce système...

La solution du dictionnaire (ou éventuellement KeyedCollection) proposée plus haut me semble beaucoup plus adaptée

[pot2yaourt]

Merci à tous pour vos réponses !

Au final, le code a été entièrement revu, et du coup on garde List<MaClasse> et il n'y a plus qu'une méthode qui fait un foreach sur la liste. D'ailleurs, la liste ne contenant, en moyenne, qu'une 15aine d'éléments, côté perf ça reste très correct.

Encore merci à tous pour votre aide !
Lionel.

[ostenhard]

S'il n'y a qu'un élément qui correspond à l'id, le foreach sur toute la liste n'est pas idéal, en faisant un Find(x=> x.LaPropriété == valeur) cela serait mieux !