Discussion :

[jmclej]

Bonjour,
J'ai actuellement ceci qui fonctionne :

Code :

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

1
2
3
4
5
6
7
8
9
10
public void func(Division[] divisions)
{
    var currencies = new HashSet<string>(divisions.SelectMany(div => div.Locations).Select(e => e.CcyLoc));
    //currencies.UnionWith(new HashSet<string>(divisions.SelectMany(div => div.CurrencyCode)));
    //ou bien même
    //currencies.UnionWith(divisions.SelectMany(div => div.CcyDiv));
    foreach(CalcDivision div in divisions) {
        currencies.Add(div.CurrencyCode);
    }
}

Mais je ne comprends pas pourquoi je ne peux pas remplacer mon foreach par un des lignes commentée du dessus. Ça ne compile pas (invalid arguments).
Voyez-vous le problème?
D'avance merci,
Julien

[Shivaneth]

Bonjour,

essaye ceci :

Code :

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

1
2
3
4
    var currencies = new HashSet<string>(divisions.SelectMany(div => div.Locations).Select(e => e.CcyLoc));
    currencies.AddRange((from d in divisions select d.CurrencyCode).ToList());
    //ou
    currencies.AddRange(divisions.Select(d=>d.CurrencyCode));

[Sehnsucht]

@Shiva Skunk: et la méthode HashSet<T>.AddRange ; tu la sors du chapeau ?

en partant du principe que CurrencyCode et CcyDiv sont bien des string ; le fait de faire un SelectMany dessus renvoie alors un IEnumerable<char> car SelectMany sert à "aplatir" une collection de collections (et un string est un IEnumerable<char>) ; comme tu l'as fait pour "aplatir" l'ensemble des CcyLoc contenus dans tous les Locations. Du coup, a priori (parce qu'il faut quand même faire pas mal de "divination" pour savoir qui fait quoi avec le peu d'infos ), remplace ces SelectMany par des Select et ça devrait aller (il faudra peut-être un ajout de Cast ou OfType pour le CurrencyCode vu qu'il semblerait que tu l'obtiennes de CalcDivision plutôt que Division [et que je suppose que le premier hérite du second]).

Sur un autre plan, le foreach c'était déjà très bien à la base ; faut arrêter de vouloir à tout prix mettre du Linq parce que c'est "hype" (UnionWith fait juste un foreach avec un AddIfNotPresent (une méthode interne à la classe) même méthode appelée depuis Add ; donc tu gagnes juste un passage par Add).

Cordialement !

[Shivaneth]

Rho bah si on peut plus se réveiller tranquillement le matin aussi

désolée :red:

[worm83]

Bonjour,

Tu peux recréer la méthode d'extension ForEach comme ceci

Code :

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

1
2
3
4
5
6
7
8
9
 
 
 public static void Foreach<T>(this IEnumerable<T> li, Action<T> action)
        {
            foreach (var x in li)
            {
                action(x);
            }
        }

Ce qui donne à l'utilisation

Code :

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

1
2
3
4
5
6
 
 
public void UneAction(IEnumerable<string> li)
{
       li.ForEach( x => Console.Writeline(x));
}

Ça reste un foreach mais écrit de facon "Linq".


Edit : Je viens de remarquer que tu avais déjà posté la question et que quelqu'un avait répondu la même chose que moi.

[jmclej]

C'est pas que je cherche à faire du LINQ à tout prix, je cherche juste à comprendre.
Effectivement, je me suis trompé: CcyLoc, CurrencyCode et CcyDiv sont les mêmes string en fait mais je ne peux plus modifier mon post.
AddRange n'est effectivement pas accessible.
Effectivement Select fonctionne quand SelectMany ne fonctionne pas, c'est un peu tricky cette "mise à plat" que fait SelectMany pour passer d'un type string à un IEnumerable de char.. Dans ma 1ère ligne "SelectMany(div => div.Locations)", comment est-ce qu'il sait qu'il ne doit s'arrêter à retourner des Locations et pas d'autres types qui composent mes Locations?
Y a-t-il une différence à faire currencies.UnionWith(new HashSet<string>(divisions.Select(div => div.CurrencyCode))); ou currencies.UnionWith(divisions.Select(div => div.CurrencyCode)); ?