[C#] LINQ を使って独自クラスリストの中身を更新したい

Question

###前提・実現したいこと
LINQ を使って独自クラスの中身を更新したいです。

###該当のソースコード
適当なデータを持つデータクラス↓

public class Data
{
    public int A;
    public int B;
    ...
    ...
}

そのクラスのリストの中身を更新したいです。

var dataList = new List<Data>();
foreach (var data in dataList)
{
    ++data.A;
}

↑のforeachをLINQで書きたいです。
イメージはこんな感じですが、うまくいかないです。。。

var increment = dataList.Select(data => ++data.A);

ご助力おねがいします。

Answer 1

selectは本来、出力の形を変える時に使うものです。そのコードだと返り値がintぽいのでint型の配列ができちゃうのかな？
無理にやろうとすると以下のような形になりますが、正直、linqの使い方としてはあまり有意義ではないので素直にforループの方がいい気がします。（後、linqはforループに比べて大分遅いです）

var increment = dataList.Select(data => { 
    ++data.A;
    return data;
});

Comments

[Mentol]

ありがとうございます！
調べててなんか違うなぁと思ってたので、スッキリしました笑

Answer 2

List<T> になっているのなら，

dataList.ForEach(data => ++data.A);

でよいのではと思います。

Comments

[Mentol]

ああああ！これです。これがやりたかったやつです。
ありがとうございましたー！

Answer 3

LINQのSelectは、新しいインスタンスを生成するメソッドです。
今回質問に書かれたコード

var increment = dataList.Select(data => ++data.A);

だと、dataListの中身を更新するのではなく、Select句の中で指定した値のコレクションを生成するというコードになります。
それでもSelectで実装する必要があるなら

var updated = dataList.Select(d => new Data() {A = ++d.A, B = d.B});

で実現できますが、dataListとは別のインスタンスを作るのでメモリの無駄です。
（dataListの更新前と後が両方必要な場合は、こうすればよいです。）

更新前後の両方のオブジェクトが必要ないのであれば、Foreachを使うのがベターです。

Comments

[Mentol]

ありがとうございます！
更新後のデータと更新前のデータを持ちたいときに使わせていただきますｂ