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

《.NET本质论－第1卷:公共语言运行库》，Don Box, Chris Sells著，张晓坤译，中国电力出版社，2004年4月第1版
“第5章 实例”，“相等与同一”小节，pp.129-130:
-----------------------------------------------------------------------------
    示例5.7 实现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语句可靠。 :)