質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

Q&A

解決済

4回答

1744閲覧

【C#】オブジェクトリストの一部を条件付きで更新【LINQ】

nyahonyaho

総合スコア83

C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

0グッド

0クリップ

投稿2018/11/06 07:51

編集2018/11/06 07:54

フィールドしかないクラスがあります。

public class Human { public string Name{ get; set; } public string Age{ get; set; } public bool Otona{ get; set; } }

これをいっぱいnewして入れた new List<Human>() があるとします。

"20才以上の人だけ大人フラグをオンする"方法がわかりません。

humanList.Where(e => e.AGE >= 20).Select(e => e.Otona = true);

ってやると、boolのリスト、しかも条件を満たしたレコードのみのができあがってしまうのです。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

len_souko

2018/11/06 10:22

× フィールドしかないクラス / 〇 プロパティしかないクラス
len_souko

2018/11/06 10:23

フィールドとプロパティは全く異なるものですので間違わないようにしましょう
len_souko

2018/11/06 10:34

この質問ですが、条件を満たしたデータに対するデータ更新が目的ですか?それとも質問に挙げている固定条件を元にフラグを立てることが目的ですか?前者ならtomato氏の回答になりますし、後者ならYamakawaJunichi氏の回答が解決案となるかと思います
guest

回答4

0

こんにちは。

「フラグをオン」という操作は「クエリ」ではなく「アクション」に該当するため、Linqクエリ内で実行するのは推奨されません。作用はforeachを利用して処理するべきものです。

csharp

1var adults = humanList.Where(e => e.AGE >= 20); 2 3foreach (var adult in adults) 4 adult.Otona = true;

投稿2018/11/06 08:06

tamoto

総合スコア4103

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

大人かどうかはHumanの属性と考えられるため、Linqで設定するのではなくプロパティとして持たすのが適切かと思います。

C#

1public class Human 2{ 3 public string Name { get; set; } 4 public string Age { get; set; } 5 public bool Otona => int.Parse(Age) >= 20; // 例外処理省略 6}

投稿2018/11/06 08:24

YamakawaJunichi

総合スコア630

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

len_souko

2018/11/06 10:31 編集

一般的な意味から行くと外部からフラグを立てるよりもプロパティのgetterで作り出す方があってますもんね 僕もどちらかというとこの手法をとります ただ、今回の質問が単純にフラグを立てたいだけなのならプロパティのgetterで処理をする出良いと思いますが、「linqでデータをこねくり回したい」が目的で、本当にやりたいことは公開できない為(お仕事で作ってるもの等)にその一例として出しただけならこれでは解決できないかもしれません
len_souko

2018/11/06 10:42

補足 固定条件でフラグを立てる場合なら、外部から変更を行うようにすると、フラグを立てた後にデータ追加を行う際に変更漏れが起きたり再度同じ処理を記述する必要があったりと面倒ごとの原因となりそうですし、そもそもの条件を使用場所で把握しないといけないというデメリットがあります その為、プロパティのgetter内で処理した結果を返すようにすることで条件判断によるフラグの設定はgetterの一か所だけで済みますし、使用する側からは条件のことをいちいち気にせずにフラグを見るだけで済むようになります なお、何かしらの条件(国籍や生年月日など)で成人条件が変わる場合はそういった情報もプロパティへ追加してgetter内で条件判断に追加すれば対応可能です
guest

0

SelectではなくList.ForEachを使うかforeachでループするかどちらかに変えてください

投稿2018/11/06 08:03

papinianus

総合スコア12705

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

ベストアンサー

humanList.Where(e => e.AGE >= 20).ToList().ForEach(e => e.Otona = true);

投稿2018/11/06 16:18

teramon3333

総合スコア102

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

nyahonyaho

2018/11/06 23:36

できました! ありがとうございまする。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問