同じオブジェクトの１件目と２件目以降の各プロパティの値が同じかどうかを比較をしたい。

Question

お世話になります。
標記のとおり、同じオブジェクト（_ZokuseiDto ）の１件目と２件目以降の各プロパティの値が同じかどうかを比較をしたいのですが、これを実行するとオブジェクト（_ZokuseiDto ）の中で定義している別のIList型のオブジェクトのプロパティのCount？のところで値が相違してしまいます。
どう対処すればよいのか見当がつきません。
どなたかご教授おねがいします。

//クラスです。長すぎるので一部抜粋です。

    [Serializable]

    class EigyoGyoumuDto
    {
        /// <summary>
        /// 共通ロジック
        /// </summary>
        public CommonLogic cmnLc;

        /// <summary>
        /// 担当者マスタ辞書
        /// </summary>
        public IDictionary<int?, MTantouDto> _MTantouDic { set; get; }
        
        /// <summary>
        /// 得意先マスタ辞書
        /// </summary>
        public IDictionary<String, MTokuisakiDto> _MTokuisakiDic { set; get; }

        /// <summary>
        /// 見積りヘッダ
        /// </summary>
        public TMitsumoriHeadDto _TMitsumoriHeadDto { set; get; }

        /// <summary>
        /// 見積りヘッダ属性
        /// </summary>
        public TMitsumoriHeadZokuseiDto _TMitsumoriHeadZokuseiDto { set; get; }

        /// <summary>
        /// 見積り明細
        /// </summary>
        public TMitsumoriDetailDto _TMitsumoriDetailDto { set; get; }

        /// <summary>
        /// 見積り明細属性
        /// </summary>
        public TMitsumoriDetailZokuseiDto _TMitsumoriDetailZokuseiDto { set; get; }

        /// <summary>
        /// 受注属性
        /// </summary>
        public TJuchuZokuseiDto _TJuchuZokuseiDto { set; get; }

        /// <summary>
        /// 変換前ファイル名
        /// </summary>
        public String BeforeConversionFileName { set; get; }

        /// <summary>
        /// 変換前ファイルパス名
        /// </summary>
        public String BeforeConversionFilePath { set; get; }



        /// <summary>
        /// 変換後ファイル名
        /// </summary>
        public String AfterConversionFileName
        {
            get
            {
                if (String.IsNullOrEmpty(BeforeConversionFileName) && _TMitsumoriDetailZokuseiDto == null)
                {
                    return String.Empty;
                }

                String MitFileNm = String.Empty;
                //コンボボックスを選択している場合
                if (!String.IsNullOrEmpty(BeforeConversionFileName))
                {
                    //変換前ファイル名の拡張子を取得
                    String Extension = Path.GetExtension(BeforeConversionFileName);
                    MitFileNm = MitCode + "-" + GenericUtil.ZeroPadding(GenericUtil.StringTryparse2(_TMitsumoriDetailDto.MitSeq), 2) + Extension;

                    //既存のファイル名取得
                    if (_TMitsumoriDetailZokuseiDto == null)
                    {
                        TMitsumoriDetailZokuseiDtoNewInstance();

                    }
                    _TMitsumoriDetailZokuseiDto.MitFileNm = MitFileNm;

                }


                return _TMitsumoriDetailZokuseiDto.MitFileNm;

            }

        }

        /// <summary>
        /// 計算書発行日（属性情報）
        /// </summary>
        public DateTime? MitDate
        {
            set
            {
                if (_TMitsumoriHeadZokuseiDto == null)
                {
                    TMitsumoriHeadZokuseiDtoNewInstance();
                }

                _TMitsumoriHeadZokuseiDto.MitDate = value;

            }
            get
            {
                if (_TMitsumoriHeadZokuseiDto == null)
                {
                    return null;
                }
                return _TMitsumoriHeadZokuseiDto.MitDate;
            }

        }

        /// <summary>
        /// 御見積書発行日（属性情報）
        /// </summary>
        public DateTime? MitMitsumorishoDate
        {
            set
            {
                if (_TMitsumoriHeadZokuseiDto == null)
                {
                    TMitsumoriHeadZokuseiDtoNewInstance();
                }

                _TMitsumoriHeadZokuseiDto.MitMitsumorishoDate = value;

            }
            get
            {
                if (_TMitsumoriHeadZokuseiDto == null)
                {
                    return null;
                }
                return _TMitsumoriHeadZokuseiDto.MitMitsumorishoDate;
            }

        }
        

        /// <summary>
        /// 備考３
        /// </summary>
        public String MitEigyoBikou3
        {

            set
            {
                TMitsumoriDetailZokuseiDtoNewInstance();

                _TMitsumoriDetailZokuseiDto.MitEigyoBikou3 = value;
            }

            get
            {
                if (_TMitsumoriDetailZokuseiDto == null)
                {
                    return String.Empty;
                }

                return _TMitsumoriDetailZokuseiDto.MitEigyoBikou3;


            }

        }
    }        







                    //２件目以降のデータが１件目と同じかチェックする場合
                    if (ChkDataSameCheck.Checked)
                    {

                        IList<EigyoGyoumuDto> _EigyoGyoumuDtoList = new List<EigyoGyoumuDto>();
                        _EigyoGyoumuDtoList = DataList;
						//比較用に１件目を退避しておく
                        ZokuseiDto First_ZokuseiDto = _EigyoGyoumuDtoList[0]._ZokuseiDto;
                        PropertyInfo[] infoArrayOfFirst = First_ZokuseiDto.GetType().GetProperties();

                        foreach (EigyoGyoumuDto dto in _EigyoGyoumuDtoList)
                        {

                            if (dto._ZokuseiDto != null)
                            {

                                PropertyInfo[] infoArray = dto._ZokuseiDto.GetType().GetProperties();
                                int idx = 0;
                                // プロパティ情報出力をループで回す
                                foreach (PropertyInfo infoFirst in infoArrayOfFirst)
                                {

                                    MethodInfo getAccessor = infoFirst.GetMethod;

                                    Object first = infoFirst.GetValue(First_ZokuseiDto);
                                    Object data = infoArray[idx].GetValue(dto._ZokuseiDto);
                                    if (first == null || data == null)
                                    {

                                        if ((first != null && data == null) || (first == null && data != null))
                                        {
                                            Message_Warn("２件目以降のデータが１件目と違います。");
                                            return;
                                        }

                                    }
                                    else
                                    {
                                        if (!first.Equals(data))
                                        {
                                            Message_Warn("２件目以降のデータが１件目と違います。");
                                            return;
                                        }
                                    }

                                    idx++;
                                }

                            }
                        }
                    }

Comments

[papinianus]

EigyoGyoumuDtoのクラス構造は未知というか、自作ではないという前提なのですか？(もしかしてクラス構造が変わっても対応できるようにしようとしている？)

[cutedog]

EigyoGyoumuDtoのクラス構造は自作です。画面の一覧のDatasourceに食わせるためのクラスなので、内容としては、ＤＢから取得したIList型のオブジェクトが数個と、それらから切り出した項目のGetやSetが含まれます。将来的にプロパティが増える可能性はあります。比較したいのは、切り出した項目のプロパティです。

Answer 1

別のIList型のオブジェクトのプロパティのCount？のところで値が相違してしまいます。

これはおそらく参照型(値型との対比)であるために、ReferenceEqualで等値判断がなされて一致しないことになるのだと思われます。
なので例えばGetValueしたfirstなどをIList<T>にキャストするなどして、nullでなかったらSequenceEqualで判定する、とか、型がIListを実装しているか判定するというような処理を書けば判断はできると思いますが、やめたほうがいいです。

YAmaGNZさんと似た話になるのですが、IEquatableを実装して(つまりEquals/GetHashcodeを実装して)EigyoGyoumuDtoクラス自身を"等値比較可能"な型にするほうがいいです。
それはEigyoGyoumuDtoクラスの責務です。EigyoGyoumuDtoが同じかどうかの判断をそのクラス外でするのはおかしいと思います。
また、TMitsumoriDetailDtoのようなものも内部に含んでおり、これらもプロパティをとって単に==で比較することはできません。つまり、IListであるかの判定以外にも特定のクラスであるかの判定を繰り返さないといけなくなります。Dtoクラスを拡張して、参照型以外のプロパティが増えればifの分岐がどんどん増えることになり、到底メンテできないと思います(等値判断が将来的にここだけでしかやらないというのでしょうか？)

また、IEquatableを実装しておけば、Distinct()が使えるので、Count()とDistinct().Count()を比べるだけで、重複があるかを判定できることになります。

今一度設計から見直すことをおすすめします。

蛇足ですが、objectで比較するコードが出てくるのでは、何のために静的型言語を使っているのかわからなくなります。objectが出てきた時点でまずいコードを書いていると考えたほうがいいと思います。

Comments

[cutedog]

ありがうございます。
下記のようなサンプル見つけたのですが、こういうふうに書いた方がいいということですかね？
しかし、IEquatableの実装内容の意味はわかるのですが、
GetHashCode()の実装を何故しないといけないのかがわからない・・・
でも下記のサンプルのようにすればいいのですかね？
public class Person : IEquatable<Person>
{
private string name;
private DateTime birthday;

public string Name
{
get { return name; }
}

public DateTime Birthday
{
get { return birthday; }
}

public Person(string name, DateTime birthday)
{
this.name = name;
this.birthday = birthday;
}

// 実装するのを忘れやすいので注意
public override int GetHashCode()
{
return name.GetHashCode() ^ birthday.GetHashCode();
}

#region IEquatable<Person> メンバ

bool IEquatable<Person>.Equals(Person other)
{
if (other == null)
{
return false;
}

return name == other.name && birthday == other.birthday;
}

#endregion
}

[papinianus]

そうですね、こういう感じだと思います。クラス自身が同じかの決め方をルール付けしたほうがいいので。
今回のだと、見積とかもIEquatableを実装しとかないと上手く比較できないかもしれないです。
Hash値は名前のとおり、HashSetで同一判定したり、あるいはDictionaryのキーになったときに使われたりします(のはずです)。簡易、高速に同一かを判定するときに使います。なので、違うとみなすものは、違うHash値になるようにしてください。

[YAmaGNZ]

GetHashCodeですが、Distinct()などで使っているので、Equalsと同様の結果となるように実装しないと正しい結果となりません。

また、IEquatableだけではなく、Equals(object)のオーバーロードも必要かと思います。
List<Person>のContainsを使用する場合などはEquatable<Person>.Equalsが使用されますが、
Person.Equalsの場合はEquals(object)が使用されるので、結果が違ってきます。
また、==、!=演算子との結果も異なってきますので、こちらも実装したほうがよいかと思います。

どのような使用のしかたをするかによって、最低限どのメソッドを実装するかは変わってくると思いますので考えてみてください。

[cutedog]

了解です。
元が独学の老年コボラーなので理解に時間が掛かりそうですが
試行錯誤してみます。
ありがとうございました。。

Answer 2

状況を正確に把握できていませんが、
公開しているプロパティ全てを比較したいのではなく、一部のプロパティのみを比較したいというのであれば
==、!=演算子（Equals、GetHashCodeも）をオーバーロードするのがいいのではないかと思います。

そうでないなら、読み飛ばしてください。

Comments

[cutedog]

参考になりました。ありがとうございます。