Assegnazione di valori all'interno di un LINQ Selezionare?

Question

Ho la seguente query:

drivers.Select(d => { d.id = 0; d.updated = DateTime.Now; return d; }).ToList(); 

driver è una lista che viene fornito con diversi ID e valori aggiornati, quindi sto cambiando i valori nel Select, ma è il modo corretto per farlo. So già che non sto riassegnazione ai conducenti di piloti a causa ReSharper lamenta, quindi credo che sarebbe stato meglio se fosse stato:

drivers = drivers.Select(d => { d.id = 0; d.updated = DateTime.Now; return d; }).ToList(); 

, ma questo è ancora il modo in cui qualcuno dovrebbe assegnare nuovi valori per ciascun elemento l'elenco dei conducenti?

Answer 1

Anche se questo sembra innocente, specialmente in combinazione con una chiamata ToList che esegue immediatamente il codice, non starei assolutamente a modificare qualcosa come parte di una query: il trucco è così insolito da far inciampare i lettori del programma , anche quelli esperti, soprattutto se non l'hanno mai visto prima.

Non c'è niente di sbagliato con foreach loop - il fatto che si può farlo con LINQ non significa che si dovrebbe essere facendo.

Answer 2

MAI FARE QUESTO. Una query deve essere una query ; dovrebbe essere non distruttivo fare domande su un'origine dati. Se si desidera causare un effetto collaterale, utilizzare un ciclo foreach; questo è quello che è per Usa lo strumento giusto per il lavoro.

Answer 3

Ok, risponderò io stesso.

Le query Xaisoft, Linq, che si tratti di espressione lambda o espressione di query, non devono essere utilizzate per modificare l'elenco. Quindi il tuo Select

drivers = drivers.Select(d => { d.id = 0; d.updated = DateTime.Now; return d; }).ToList(); 

è in cattivo stile. Confonde/illeggibile, non standard, e contro la filosofia Linq. Un altro stile poveri di raggiungere il risultato finale è:

drivers.Any(d => { d.id = 0; d.updated = DateTime.Now; return false; }); 

Ma questo non vuol dire ForEach su List<T> è inappropriato. Trova usi in casi come il tuo, ma non mischiare la mutazione con la query Linq, tutto qui. Preferisco scrivere qualcosa del tipo:

drivers.ForEach(d => d.updated = DateTime.Now); 

È elegante e comprensibile. Dal momento che non si occupa di Linq, non confonde troppo. Non mi piace la sintassi per le affermazioni multiple (come nel tuo caso) all'interno del lambda. È un po 'meno leggibile e più difficile da eseguire il debug quando le cose diventano complesse. Nel tuo caso preferisco un ciclo dritto foreach.

foreach (var d in drivers) 
{ 
    d.id = 0; 
    d.updated = DateTime.Now; 
} 

personalmente mi piace ForEach su IEnumerable<T>as a terminating call to Linq expression (cioè, se l'assegnazione non intende essere una query ma un'esecuzione).