C# Linqの使い方

Question

person {  
        stringid { get; set; }
        string name { get; set; }
        DateTime birthday { get; set; }
        int sex { get; set; }
        string pairid { get; set; }
    }

List<person> obj = new List<person>();
obj.Add(new person {....});

sexには1=男,2=女
pairidにはsexが1の場合は2のどれか
2の場合は1のどれか
つまりカップルとしてペアリングされていて
pairidがない場合はフリーというようなデータが入ったobjがあり、
pairidがnullではない場合（つまりカップルとなっている）にカップルの誕生日が若い方に対してoderbyしたlistを取得したい。

var a = obj.Where(r=> r.sex == 1).Orderby(下記条件).ToList();

orderbyの条件
foreach(person p1 in obj) 
{
	if (p1.pairid != null)
	{
		foreach(person p2 in obj) 
		{
			if (p1.pairid == p2.id)
			{
				if (p1.birthday.CompareTo(p2.birtday))
				{
					p1.birthday
				}
				else
				{
					p2.birthday
				}
			}
		}		
	}
}

var a にはpersonのプロパティに加え、p2.birthdayが条件の対象となった場合に追加されたListとしたい。

ちなみに var aとなってるところの型は正しくはどうなるのでしょうか？
型でvarの使い方がいまいち理解できていません。

どうぞよろしくお願いします。

Comments

[tamoto]

「一発でとる」「joinしてbirthdayを比較して新しいほう」と言ってますが、最終的に「どのようなデータ集合を取得したいのか」が明記されていないので、回答が難しいです。

[kiichi54321]

何と比較するのかよくわからないので、何がしたいのか日本語として意味がわかりません。

[退会済みユーザー]

一度 https://teratail.com/help/question-tips を読んでみることをお勧めします。そこにも書いてありますが、"人に質問をするには、自分が何を尋ねたいかを知っている必要があります。これは、「自分が今『何がわからないのか』がわかっていて、言語化できている」ということです" が十分にできておらす、今は "何がわからないかわからない人から質問を受けても、答える側も困ってしまいます" という状況です。「一発」ってなんなのかわかりませんが、質問者さんが二発、三発で取れるならそのコードをアップして、それをどうしたいのか書くというような配慮が必要です。

[ebiryo]

カップルを対象にペアの若いほうを取り出し、取り出したものを若い順に並べる、これをメソッドチェーンを使って行う　ってことですかね？　回答より何を言ってるのか理解するほうが難しいです。　

[takagi.1994]

pairにはidが入っている。idは一意の値である。ほしい情報は上記のプロパティとさらにpairidに入っているidに紐づくプロパティです。　ebiryoさんが言われてることです

[ebiryo]

「pairにはidが入っている。idは一意の値」 などは皆さんわかってらっしゃると思いますよ。もし私の言っていることが正しいのであれば、質問を書き直してはいかがでしょうか。

[skitoy4321]

サンプルデータを用意して、ソート前の値と期待するソート後の値を書いた方がわかりやすいと思います。

Answer 1

誕生日が新しい方をキーとして「古い順番にソート」なんでしょうか・・・そうだとすれば

(1)ペアのうち男女両方を残すなら

List<person> list = new List<person>();
list.Add(new person {....});
...
var result = list
  .Where(p => p.pairid != null)
  .OrderBy(p => {
    var o = list.Find(p2 => p2.id == p.pairid);
    return p.birthday > o.birthday ? p.birthday : o.birthday;
  })
  .ToList();

(2)ペアのうち誕生日が古い方を除外(同一なら除外しない)

var result = list
  .Where(p => p.pairid != null)
  .Where(p => {
    var o = list.Find(p2 => p2.id == p.pairid);
    return p.birthday > o.birthday;
  })
  .OrderBy(p => p.birthday)
  .ToList();

ただし、(1)の方は、ソート中にキーを参照する度にFindでペア相手を検索するのは効率が悪そうなのでそれに配慮すると

(3) (1)の改良

var result = list
  .Where(p => p.pairid != null)
  .Select(p => {
    var o = list.Find(p2 => p2.id == p.pairid);
    var k = p.birthday > o.birthday ? p.birthday : o.birthday;
    return new { Key = k, Value = p };
  })
  .OrderBy(kv => kv.Key)
  .Select(kv => kv.Value)
  .ToList();

さらに、順サーチではなくハッシュテーブルによる検索とするなら

(4) Dictionaryを使う

var dictionary = list.ToDictionary(p => p.id);
var result = list
  .Where(p => p.pairid != null)
  .Select(p => {
    var o = dictionary[p.pairid];
    var k = p.birthday > o.birthday ? p.birthday : o.birthday;
    return new { Key = k, Value = p };
  })
  .OrderBy(kv => kv.Key)
  .Select(kv => kv.Value)
  .ToList();

型でvarの使い方がいまいち理解できていません。

varの使い方ではなくて代入文の右辺にある「Linqのメソッドの結果の型がよくわからない」ということではないでしょうか。Linqのメソッドの各々のリファレンスを熟読するなりLinqの解説ページを研究するなりして、各々のメソッドがIEnumerable<T>に対して、何を引数にしてどういう操作を行い結果をどういう形で後段のパイプラインへ渡すのかを覚えてしまいましょう。Linqのメソッドは一見多いように見えますが、オーバーロードによって数が膨らんでいるように見えるだけで、本質的にはそれほど多くのパターンはないように思います。
またVisualStudioで変数名のところへマウスをホバーするとツールチップで型を確認できるので活用してみてください。