Discussion :

[ZaaN]

Hello,
C'est surement tres con, mais voici mon problème :

pour un dictionnaire de type

Code :

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

protected IDictionary<Maclasse, uint> m_Dico;

je tente de modifier la VALUE (et non la clé), mais le compilo me dis que VALUE est read only... !?

Code :

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

1
2
3
4
5
6
7
foreach (var Entry in m_Dico)
{
    if (Entry.Value > uiRefValue)
    {        
        Entry.Value--;//Erreur Ici
    }
}

Comment feriez vous pour faire cette ecriture ?

Error 1 Property or indexer System.Collections.Generic.KeyValuePair<Maclasse,uint>.Value' cannot be assigned to -- it is read only

[neptune]

Comment feriez vous pour faire cette ecriture ?

Code :

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

1
2
3
4
5
6
7
8
foreach (var Entry in m_Dico)
{
    if (Entry.Value > uiRefValue)
    {        
        //Entry.Value--;//Erreur Ici
        m_Dico[Entry.Key]--;
    }
}

Voila une des raisons pour lesquelles je n'aime pas le mot-clé var... Si tu avais su immédiatement de quel type était Entry tu trouveras facilement où est ton erreur...

Le problème de cette boucle c'est que tu modifie la collection et le foreach ne pourra plus continuer :-(

[Guulh]

Code :

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

1
2
3
4
5
6
7
foreach (var Entry in m_Dico)
{
    if (Entry.Value > uiRefValue)
    {        
        m_Dico[Entry.Key] = Entry.Value - 1;
    }
}

C'est parce que la valeur de ton dico est de type Valeur (une struct, quoi )

[edit] La solution de neptune est mieux, en effet.

[neptune]

C'est parce que la valeur de ton dico est de type Valeur (une struct, quoi )

Non, pas vraiment. C'est surtout que KeyValuePair<T> ne définit qu'un Get pour Value et non un Set. ;-)

[ZaaN]

merci bcp !

Autre question sur le meme exemple : Aurais-je une moyen de filtrer mes valeur dans l'entete de la boucle, plustot que dans un if à l'interieure de la boucle ?

un truc du genre

Code :

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

1
2
3
4
foreach (var Entry in m_Dico.FILTERonVALUEbigger(uiRefValue))
{
    m_Dico[Entry.Key]--;   
}

[neptune]

Non ce n'est pas possible, il faut imbriquer une boucle supplémentaire.

A moins qu'en 3.x... Mais pas en 2.0.

[Guulh]

Non, pas vraiment. C'est surtout que KeyValuePair<T> ne définit qu'un Get pour Value et non un Set. ;-)

Ce que je veux dire, c'est que si sa Value était de type classe, il aurait pu faire ça :

Code :

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

1
2
3
4
5
6
Dictionary<int, Classe> dico;
foreach(KeyValuePair<int, Classe> entry in dico)
{
if (truc)
  entry.Value.UneProp += 1;
}

Alors que là, comme c'est une struct, on ne peut ni le réaffecter (comme les classes) ni le modifier (contrairement aux classes).

Bon, bref

Pour Zaan : un dico est aussi un ICollection de KeyValuePair, tu pourrais imaginer de faire un équivalent du FindAll de la List<T> qui prendrait ça en entrée et ressortirait une liste filtrée en sortie, mais ça vaut pas la peine.

Ou avec des yield return... mais c'est inutilement compliqué ()

[ZaaN]

Le problème de cette boucle c'est que tu modifie la collection et le foreach ne pourra plus continuer :-(

Dans ce cas quelle est la meilleure solution pour faire un traitement tel que je le desire ? ( modifier plusieurs element du dico)

Faire un nouveau dico en recopiant les clé et en modifiant les valeurs ?
autre solution ?

[neptune]

Au lieu d'énumérer ta liste, parcourt la (for plutot que foreach).

[ZaaN]

je trouve pas le moyen d'acceder à un element bien precis avec un for.

J'ai pensé qu'avec un compteur basé sur .Count puis un acces aux elements que j'ai pas trouvé ... ! (ElementAt ... pas trouvé)

Bref j ai trouvé une autre methode plus simple et rapide à coder!

Code :

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

1
2
3
4
5
6
7
8
var DicoTemp = new Dictionary<Maclasse, uint>(m_Dico);
foreach (var Entry in DicoTemp)
{                
      if (Entry.Value > uiRefValue)
      {                    
           m_Dico[Entry.Key]--;                     
      }                                            
}