Question

我写了以下代码

 class Program
{
    static void Main(string[] args)
    {
        Circle c1 = new Circle(5);
        Circle c2 = new Circle(10);
        Console.WriteLine(c1.Area().ToString());
        if (c1>c2)
        {
        }
    }
}
public class Circle:System.IComparable<Circle>,IComparable
{
    public int radius { get;private set; }
    public double Area()
    {
        return Math.PI * radius * radius;
    }
    public Circle(int radius)
    {
        this.radius = radius;
    }
    public int CompareTo(Circle c)
    {
        if (c.Area() == this.Area())
            return 0;
        if (c.Area() > this.Area())
            return 1;
        return -1;
    }
    public int CompareTo(Object c)
    {
        if (((Circle)c).Area() == this.Area())
            return 0;
        if (((Circle)c).Area() > this.Area())
            return 1;
        return -1;
    }
}

然而它的错误错误1运算符'＆gt;'不能应用于'ConsoleApplication1.Circle'和'ConsoleApplication1.Circle'类型的操作数

我已经实现了这两种方法，但无法确定错误

Answer 1

你需要重载＆gt;操作员做你正在尝试的事情：

  public static bool operator > (Circle c1, Circle c2)
  {
     return (c1.CompareTo(C2) > 0);
  }

Answer 2

实施IComparable或IComparable<T>不会自动启用比较运算符。您必须专门提供他们的实现。大多数其他答案（在本答复时）提供了不正确或不完整的运算符实现。

您需要先修复CompareTo实施。第一个问题是确保处理null个案例。对象的总体比例大于null每个规格（see docs）：

public int CompareTo(Circle c)
{
    return c == null ? 1 : CompareAreas(this.Area(), c.Area());
}
public int CompareAreas(double a, double b)
{
    return a > b ? 1 : a == b ? 0 : -1;
}

CompareTo的非通用版本还需要处理与null的比较，并确保将其与Circle对象进行比较：

public int CompareTo(Object obj)
{
    if (obj == null) return 1;
    var c = obj as Circle;
    if (c == null) throw new ArgumentException(null, "obj");
    return CompareTo(c); // delegate to CompareTo(Circle)
}

最后，>和<运算符实现需要考虑一方或双方null：

public static bool operator >(Circle a, Circle b)
{
    return Compare(a, b) > 0;
}
public static bool operator <(Circle a, Circle b)
{
    return Compare(a, b) < 0;
}
public static int Compare(Circle a, Circle b)
{
    return a == null && b == null ? 0 : a == null ? 1 : a.CompareTo(b);
}

Answer 3

public static bool operator < (Circle c1, Circle c2)
{
    return compare(c1, c2) < 0;
}

public static bool operator > (Circle c1, Circle c2)
{
    return compare(c1, c2) > 0;
}

public static bool operator == (Circle c1, Circle c2)
{
    return compare(c1, c2) == 0;
}

public static int compare(Circle c1, Circle c2)
{
    return c1.radius.CompareTo(c2.radius);
}

Answer 4

实现IComparable本身不会为该类创建＆gt;，＆gt; =，＆lt;，＆lt; =运算符。如果您希望这些运算符可用于您的类，则必须实现它们：

public static bool operator > (Circle x, Circle y) {
   return x.CompareTo(y) > 0;
}

// and so on for the other operators

如果您决定创建这些运算符，您可能还需要重载.Equals方法和==运算符，以便您的Circle对象在比较操作中实际显示值类型行为。

此外，由于区域与半径成正比，您可以很好地比较半径（？拼写）而不是区域。

Answer 5

另一种解决方案也可能是IComparable类型的Circle。在这种情况下，您可以这样做，例如：

.....

if (c.Area().CompareTo(this.Area()) == 0)
     return 0;

.....

Answer 6

代码中的问题：
（1）您应该覆盖运算符>以将其与2个Circle对象一起使用。
（2）在比较方法中，您应该返回一个值，表示圆形对象之间的距离，而不仅仅是1或-1或0。
（3）您应该调用Area（）方法一次并保存其值，然后在需要时使用此值，而不是每次需要时使用此方法。

public class Circle : System.IComparable<Circle>
{
    public int radius { get; private set; }
    public double Area()
    {
        return Math.PI * radius * radius;
    }
    public Circle(int radius)
    {
        this.radius = radius;
    }
    public int CompareTo(Circle c)
    {
        //you should not just return 1 or -1, you should return a value indicating their "distance"
        return (int)(this.Area() - c.Area());
    }
    public static bool operator >(Circle a, Circle b)
    {
        return a.CompareTo(b) > 0;
    }
}

IComparable实施

6 个答案: