c# .net Equals()函数重载， 像下面这样写好吗？

Question

帮忙评审一下下面这个写法

 class Product

    {

        public string Name;

        public DateTime ExpiryDate;

        public decimal Price;

        public string[] Sizes;

        public override bool Equals(object obj)

        {

            Product p2 = (Product)obj;

            if (Name == p2.Name && ExpiryDate == p2.ExpiryDate && Price == p2.Price)

            {

                for (int i = 0; i < Sizes.Length; i++)

                {

                    if (!Sizes[i].Equals(p2.Sizes[i]))

                        return false;

                }

                return true;

            }

            else

                return true;

        }

    }

Answer 1

先判断obj is Product，以避免obj不是Product的情况。另外，可以把两个Product比较的逻辑放在双等运算符重载里面，Equals中调用。具体逻辑没有什么统一的标准，比如你可以比较所有字段，也可以在有id的情况只比较id，因业务而异。