实现System.IComparable接口的一个问题

 public sealed class Person : System.IComparable { 
      internal int age; 
      public int CompareTo(object rhs) { 
        if (this == rhs) return 0; // 相同 
        Person other = (Person)rhs; 
        if (other.age > this.age) return -1; 
        else if (other.age < this.age) return 1; 
        else return 0; 
      } 
    } 

    你可以在CompareTo方法中，对复合的if-else语句进行修改，如下所示： 
    return this.age - other.age; 
    如果this.age大于other.age，该方法将返回一个正数；如果两个值相等，该方法返回零；否则，该方法返回一个负数。 
----------------------------------------------------------------------------- 
    我们考虑以下测试程序： 

using System;

namespace Benben.Test.EssentialNet.Chapter5 
{ 
  public sealed class Person : IComparable 
  { 
    internal int age; 
    public int CompareTo(object rhs) 
    { 
      if (this == rhs) return 0; // 相同 
      Person other = (Person)rhs; 
      return this.age - other.age; 
    } 
  } 
  
  sealed class Test 
  { 
    static void Main() 
    { 
      Person Lippman = new Person(); 
      Person Richter = new Person(); 
      Lippman.age = int.MaxValue; 
      Richter.age = -1; 
      int cmp = Lippman.CompareTo(Richter); 
      string str = (cmp > 0) ? ">" : ((cmp < 0) ? "<" : "="); 
      Console.WriteLine("Lippman.age: {0:N0}", Lippman.age); 
      Console.WriteLine("Richter.age: {0:N0}", Richter.age); 
      Console.WriteLine("ComperTo return: {0:N0}", cmp); 
      Console.WriteLine("Comper result: Lippman {0} Richter", str); 
    } 
  } 
} 

    运行结果如下：

Lippman.age: 2,147,483,647 
Richter.age: -1 
ComperTo return: -2,147,483,648 
Comper result: Lippman < Richter

    该测试程序得到了错误的比较结果。可见，大师也有考虑不周到的地方。 :) 
    如果使用  /checked+ 参数重新编译该程序，再运行时将抛出一个System.OverflowException异常，也就是说算术运算导致溢出。这就是导致错误的比较结果的原因。 
    当然，人的年龄既不可能是负数，也不可能太大。实际上，把age字段的类型改为short或者ushort就可以避免这个问题。 
    如果真的需要比较两个int或者long类型的数，还是老老实实地使用复合的if-else语句可靠。 :)