独自クラスを持つリスト同士のプロパティを比較し一致した値を代入する処理

Question

独自クラスを持つリストが２つあり、2つのリストにあるCDプロパティを比較して
CDに一致するNameプロパティを上書きする処理をエレガントに書く方法を教えて下さい。

以下のプログラムではuser3のCDとowner3のCDが初めに一致するためnameプロパティがowner3に書き換えられます。
※owner99も一致しますが、CDはユニークな想定なため初めに一致したCDを返す想定です。

    class Person {
        public int CD;
        public string Name;
    }

    List<Person> userPersonList = new List<Person>(){
                new Person(){ CD = 1,Name = "user1"},
                new Person(){ CD = 2,Name = "user2"},
                new Person(){ CD = 3,Name = "user3"}
    };

    List<Person> ownerPersonList = new List<Person>(){
                new Person(){ CD = 4,Name = "owner1"},
                new Person(){ CD = 5,Name = "owner2"},
                new Person(){ CD = 6,Name = "owner3"},
                new Person(){ CD = 3,Name = "owner3"}
                new Person(){ CD = 3,Name = "owner99"}
    };


    foreach(var userPerson in userPersonList){
      foreach(var ownerPerson in ownerPersonList){
        if(userPerson.CD == ownerPerson.CD){
           userPerson.Name = ownerPerson.Name;
           break;
        }
      }
    }

以下は試してみた代入部分のLINQ構文です。

foreach(var result in userPersonList.Where(x => ownerPersonList.Select(y => y.CD).Contains(x.CD))){
  result.Name = ownerPersonList.Where(x => x.CD == result.CD).Select(x => x.Name).FirstOrDefault();
}

新規に拡張メソッド作成する方法は使わないようにお願いします。
C#のバージョンは6.0です。

Answer 1

もうベストアンサーもついてるので意味はないかもしれませんがあくまで書き換え処理に絞って考えると下記のようにDictionaryを使用する方法なんかもありではないでしょうか？

var ownerPersons = ownerPersonList
    .GroupBy(x => x.CD)
    .Select(x => x.First())
    .ToDictionary(x => x.CD);

foreach (var userPerson in userPersonList) {

    if (ownerPersons.TryGetValue(userPerson.CD, out var ownerPerson)) {
        userPerson.Name = ownerPerson.Name;
    }

}

Comments

[HAL-]

ご回答ありがとうございます。
Dictionaryへ変換しtryGetで比較しそのまま代入する方法は思いつきませんでした。

私の書いた方法よりかなりスマートだと思います！
質問させて頂いた処理はプログラムを書いていくとわりとぶつかる処理だと思うので拡張メソッドを使用しない書き換え処理を行う場合に、また悩んだ際はこちらのコードを参考にさせていただきたいと思います。

Answer 2

Person に IEqutable<Person> を実装して

public class Person : IEquatable<Person>
{
    public int CD;
    public string Name;

    public bool Equals(Person other)
    {
        if (other is null)
            return false;

        return CD == other.CD && Name == other.Name;
    }

    public override bool Equals(object obj) => Equals(obj as Person);
    public override int GetHashCode() => (CD, Name).GetHashCode();
}

Enumerable.Intersect メソッドで userPersonList の積集合を取得することで下記のように記述できます。エレガントかはわかりません。

var intersect = userPersonList.Intersect(ownerPersonList);
foreach(var user in intersect){
    var owner = ownerPersonList.FirstOrDefault(x => x.CD == user.CD);
    user.Name = owner?.Name;
}

IEqutable<TSource> を実装したくない場合は IEqualityComparer<TSource> を使っても実現できます。

IEqualityComparer を使う場合

public class Person
{
    public int CD;
    public string Name;
}

public class PersonComparer : IEqualityComparer<Person>
{
    public bool Equals(Person x, Person y)
    {
        if (Object.ReferenceEquals(x, y)) return true;

        if (Object.ReferenceEquals(x, null) || Object.ReferenceEquals(y, null))
            return false;

        return x.CD == y.CD && x.Name == y.Name;
    }

    public int GetHashCode(Person person)
    {
        if (Object.ReferenceEquals(person, null)) return 0;

        int hashPersonName = person.Name == null ? 0 : person.Name.GetHashCode();
        int hashPersonCode = person.CD.GetHashCode();

        return hashPersonName ^ hashPersonCode ;
    }
}

var intersect = userPersonList.Intersect(ownerPersonList, new PersonComparer());
foreach(var user in intersect){
    var owner = ownerPersonList.FirstOrDefault(x => x.CD == user.CD);
    user.Name = owner?.Name;
}

Comments

[HAL-]

早速のご回答ありがとうございます。
IEqutable<Person> の実装は理想的なのですが共通クラスへの実装を想定しているため、overrideするのは難しそうです。
IEqualityComparer でも実装できるということで存じ上げなかったので調べて挑戦してみます！
難しい場合はご提案頂いたunionを用いた実装しようと思います。

[BluOxy]

Union メソッドを用いることと IEqutable<Person> を実装することは、別の手法ではありません。
Union メソッドで使用する比較条件を指定する手段の1つが IEqutable<Person> を実装することです。

[BluOxy]

もしくは IEqualityComparer<Person> を実装したクラスを定義しても Union メソッドで使用する比較条件を指定することもできます。代わりに Union メソッドの引数には IEqualityComparer<Person> のインスタンスを渡す必要があります。

リファレンスのサンプルを真似しただけですが、IEqualityComparer<Person> を使用したコードも追記しました。

[HAL-]

ご指摘の通りUnionメソッドを用いた手法は勘違いしていました・・・。
リファレンスを参照し上記内容を参考に無事実装することができました。
ご丁寧に指導していただきありがとうございました。