读改善c#代码157个建议：建议10~12

目录：

• 建议10：创建对象时需要考虑是否实现比较器
• 建议11：区别对待==与Equals
• 建议12：重写Equals时也要重写GetHashCode

一、建议10：创建对象时需要考虑是否实现比较器

比较一下基本工资：

 class Salary : IComparable
    {
        public string Name { get; set; }
        public decimal BaseSalary { get; set; } 
        public decimal Bonus { get; set; }
        public int CompareTo(object obj)
        {
            Salary comparer = obj as Salary;            

            if (BaseSalary > comparer.BaseSalary)
            {
                return ;
            }
            else if (BaseSalary == comparer.BaseSalary)
            {
                return ;
            }
            else
            {
                return -;
            }
        }
    }

客户端调用：

 List<Salary> salaries = new List<Salary>();
            salaries.Add(new Salary() { Name = "Sun", BaseSalary =  });
            salaries.Add(new Salary() { Name = "Yuan", BaseSalary =  });
            salaries.Add(new Salary() { Name = "Kun", BaseSalary =  });
            salaries.Add(new Salary() { Name = "Qun", BaseSalary =  });
            salaries.Add(new Salary() { Name = "Sun", BaseSalary =  });

            salaries.Sort();

            foreach (var s in salaries)
            {
                Console.WriteLine("【Name】：{0},【BaseSalary】：￥{1}{2}", s.Name, s.BaseSalary,System.Environment.NewLine);
            }

            Console.ReadKey();

运行：

如果不想用基本工资BaseSalary进行排序，而是以奖金Bonus进行排序，使用IComparer实现自定义比较器：

class BonusComparer : IComparer<Salary>
    {
        public int Compare(Salary x, Sarlary y)
        {           return left.Bonus.CompareTo(right.Bonus);
        }
    }

客户端提供我们上面创建的比较器：

            List<Salary> salaries = new List<Salary>();
            salaries.Add(new Salary() { Name = "Sun", BaseSalary = ,Bonus= });
            salaries.Add(new Salary() { Name = "Yuan", BaseSalary = , Bonus =  });
            salaries.Add(new Salary() { Name = "Kun", BaseSalary = , Bonus =  });
            salaries.Add(new Salary() { Name = "Qun", BaseSalary = ,Bonus= });
            salaries.Add(new Salary() { Name = "Dun", BaseSalary = ,Bonus= });

            salaries.Sort(new BonusComparer());

            foreach (var s in salaries)
            {
                Console.WriteLine("Name：【{0}】,BaseSalary：￥{1}，Bonus:{2}{3}", s.Name, s.BaseSalary,s.Bonus, System.Environment.NewLine);
            }

            Console.ReadKey();

输出：

二、建议11：区别对待==与Equals

两者都是指相等性，即：值相等性和引用相等性。

值类型：如果值类型相等，返回True。

引用类型：如果指向同一个引用，返回True。

很好理解，举个例子：

1、值类型：==与Equls()

            int x = ;

            int y = ;

            Console.WriteLine("int x=1;{0}int y=1;{0}", System.Environment.NewLine,System.Environment.NewLine);

            Console.WriteLine("x==y:{0}",x == y);

            Console.WriteLine("x.Equals(y):{0}{1}",x.Equals(y),System.Environment.NewLine);

            x = y;

            Console.WriteLine("x=y;{0}", System.Environment.NewLine);

            Console.WriteLine("x==y:{0}", x == y);

            Console.WriteLine("x.Equals(y):{0}", x.Equals(y));

            Console.ReadKey();

运行：

2、引用类型

class People
    {
        public String Name { get; set; }
    }

客户端：

            People p1 = new People() { Name = "Sun" };

            People p2 = new People() { Name = "Yuan" };

            Console.WriteLine("People p1 = new People();{0}People p2 = new People();{1}", System.Environment.NewLine, System.Environment.NewLine);

            Console.WriteLine("p1==p2:{0}", p1 == p2);

            Console.WriteLine("p1.Equals(p2):{0}{1}", p1.Equals(p2), System.Environment.NewLine);

            Console.WriteLine("------------------------------------");

            p1 = p2;

            p1.Name = "Moon";

            Console.WriteLine("p1=p2;{0}", System.Environment.NewLine);

            Console.WriteLine("p1==p2:{0}", p1 == p2);

            Console.WriteLine("p1.Equals(p2):{0}", p1.Equals(p2));

运行：

后面我们修改了p1里Name="Moon"的值，但是，p2的Name值也变成了Moon。所以，==与Equal()在比较引用类型时，引用地址一样，返回True

3、引用类型重载Equals()达到值类型比较效果

还有一点，有时我们需要我们的类型看上去和string类型类似，有值类型的感觉。所以说，我们的这个引用类型，需要重载==或者Equals()。

这里建议只重载Equals()来达到像值类型一样的比较效果。保留==，保留引用比较。例如：生活中我们认为身份证号码一样的是同一个人。

 class People
    {
        public String Name { get; set; }
        public string IDCode { get; set; }

        public override bool Equals(object obj)
        {

            People p = obj as People;

            return p.IDCode == IDCode;
        }
    }

客户端：

 People p1 = new People() { IDCode="No1" };

            People p2 = new People() { IDCode = "No1" };

            Console.WriteLine("People p1 = new People();{0}People p2 = new People();{1}", System.Environment.NewLine, System.Environment.NewLine);

            Console.WriteLine("p1.IDCode={0}", p1.IDCode);
            Console.WriteLine("p2.IDCode={0}", p2.IDCode);
            Console.WriteLine();

            Console.WriteLine("p1==p2:{0}【保留引用地址的对比】", p1 == p2);

            Console.WriteLine("p1.Equals(p2):{0}【重载比较IDCode,值类型比较效果】{1}", p1.Equals(p2), System.Environment.NewLine);

            Console.WriteLine("----------------------------------");

            p1 = p2;

            p1.IDCode = "No2";

            Console.ForegroundColor = ConsoleColor.Red;

            Console.WriteLine("p1.IDCode={0}", p1.IDCode);
            Console.WriteLine("p2.IDCode={0}", p2.IDCode);
            Console.WriteLine();

            Console.ForegroundColor = ConsoleColor.White;

            Console.WriteLine("p1=p2;{0}", System.Environment.NewLine);

            Console.WriteLine("p1==p2:{0}", p1 == p2);

            Console.WriteLine("p1.Equals(p2):{0}", p1.Equals(p2));

            Console.ReadKey();

运行：

还有，Object.ReferenceEquals方法比较实例是否相同。验证引用的相等性。

三、建议12：重写Equals时也要重写GetHashCode

当我们重写Equals时，编译器会提示一条警告：

为什么会有这样的提示？

因为在 System.Collections.Hashtable类型和System.Collections .Generic.Dictionary类型以及一些其他的集合类，要求两个对象相等，必须具有相同的哈希码。

所以在重写Equals时，也应该重写GetashCode,确保相等性的算法和对象哈希码算法一致。

添加：添加一个新的键值对时，首先会获取对象的哈希码，这个哈希码指出键值对应该存在哪一个哈希桶中。

查询：查询集合的一个键时，也获取指定键对象的哈希码，这个哈希码指定了我们查找键值对所存在哪一个哈希桶中。所以我们就去哈希桶中搜索与当前指定的键对象的哈希码。

看一下下面这个实例：

class Program
    {
        static Dictionary<People, string> ppdic = new Dictionary<People, string>();

        static void AddPP()
        {
            People p = new People(){Name="Sun"};

            ppdic.Add(p, "Sun");

　　　　　　　 //Console.WriteLine(p.GetHashCode());　　　　　　　　　　
            Console.WriteLine(ppdic.ContainsKey(p));

        }

        static void Main(string[] args)
        {
            AddPP();

            People pp = new People() { Name = "Sun" }; 

　　　　　　　 //Console.WriteLine(pp.GetHashCode());           

            Console.WriteLine(ppdic.ContainsKey(pp));

            Console.ReadKey();
        }
    }

class People
    {
        public String Name { get; set; }
        public string IDCode { get; set; }

        public override bool Equals(object obj)
        {

            People p = obj as People;

            return p.IDCode == IDCode;
        }
    }

这里，我们重写了People类的Equals方法，客户端中，首先调用了AddPP()方法，添加一个Name="Sun"的People对象.

紧跟着，我们也定义了同样一个Name="Sun"的People对象。因该说两次People对象一样，所以两次输出都应该为True.

(这里我们不重写Equals效果也是一样的，但这里的重点是说明:GetHashCode)

为什么相同的对象返回的结果不一样？其实上面说过了，CLR会为每个对象创建唯一的哈希码（在生存周期内），因为当前类没有重写GetHashCode方法，所以会调用Object的GetHashCode。

解开上面的两句注释，运行：我们看到两个实例对象（p、pp）的哈希码是不一样的。

如果我们定义的自定义类型会被用作字典等类型的Key值，那我们可能需要重写Equals的同时还要重写GetHashCode。以来正确地实现我们的需求。

 class People
    {
        public String Name { get; set; }
        public string IDCode { get; set; }

        public override bool Equals(object obj)
        {

            People p = obj as People;

            return p.IDCode == IDCode;
        }

        public override int GetHashCode()
        {
            if (IDCode != null)
                return this.IDCode.GetHashCode();
            return base.GetHashCode();
        }
    }

为了产生更好的哈希值的随机分布：

public override int GetHashCode()
{
            if (IDCode != null)
                return (System.Reflection.MethodBase.GetCurrentMethod().DeclaringType.FullName + "$" + this.IDCode).GetHashCode();
            return base.GetHashCode();
}

当然最后我们也最好实现继承类型安全接口：IEquatable<People>

我们的身份证IDCode设计为只读属性，实例化时跟随一个身份证。

最终版本：

 class People:IEquatable<People>
    {

        public People(string idCode)
        {
            this._idCode = idCode;
        }
        public String Name { get; set; }

        private string _idCode;
        public string IDCode { get; private set; }

        public override bool Equals(object obj)
        {

            People p = obj as People;

            return p.IDCode == IDCode;
        }

        public override int GetHashCode()
        {
            if (IDCode != null)
                return (System.Reflection.MethodBase.GetCurrentMethod().DeclaringType.FullName + "$" + this.IDCode).GetHashCode();
            return base.GetHashCode();
        }

        public bool Equals(People other)
        {
            return IDCode == other.IDCode;
        }
    }