Discussion :

[jmclej]

Bonjour,
J'avais le code suivant

Code :

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

1
2
public CalcDivision[] divisions;
divisions = new[] { new CalcDivision(excelData, iterations, db) };

et je me retrouve à devoir faire ça :

Code :

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

1
2
3
4
5
6
public CalcDivision[] divisions;
List<CalcDivision> l = new List<CalcDivision>();
foreach (TC tc in excelData.tcs) {
	l.Add(new CalcDivision(constructionParameters, excelData, iterations, db, tc.divisionId));
}
divisions = l.ToArray();

N'y a-t-il pas une écriture plus concise possible (sachant que je n'ai pas le "droit" de changer la déclaration "public CalcDivision[] divisions;") ?
Merci pour votre aide.

[J@ckHerror]

Bonjour,

Grâce à linq vous pouvez faire

Code :

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

1
2
 
List<CalcDivision> l = divisions.ToList<CalcDivision>();

bon code,
J@ck.

[myNameIsFlo]

Je ne vois pas l'utilité de le faire plus court... M'enfin.

Si excelData.tcs est une enumerable, y'a moyen de faire

Code :

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

1
2
 
excelData.tcs.ForEach(x -> ...)

La vrai est question et pourquoi vouloir le faire? Un code 10 vaut parfois moins que le même code sur 20 lignes. Si le code est lisible et compréhensible, c'est bien l'essentiel.

[Pol63]

linq est en effet parfois l'ennemi du bien
il est possible de faire une trentaine de choses différentes sur la même ligne ... de 2500 caractères
le tout est de doser

[jmclej]

Merci,
En fait j'aurais voulu me passer d'utiliser une List temporaire et remplir mon array d'un coup.
Là je me retrouve encore avec :

Code :

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

1
2
3
4
public CalcDivision[] divisions;
List<CalcDivision> l = new List<CalcDivision>();
excelData.tcs.ForEach(x => l.Add(new CalcDivision(constructionParameters, excelData, iterations, db, x.divisionId)));
divisions = l.ToArray();

En fonction de ce qui est possible de faire, ce qui est toujours bénéfique de connaître, je jugerai si c'est mieux ou non.

[jmclej]

C'est bon, j'ai trouvé, an ajoutant "use System.Linq" :

Code :

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

1
2
public CalcDivision[] divisions;
divisions = excelData.tcs.Select(x => new CalcDivision(constructionParameters, excelData, iterations, db, x.divisionId)).ToArray();