Question

我还是新手来重载运营商。在我遇到这个问题之前，我以为自己做得很好。在！=运算符上抛出NullReferenceException。我假设它在CompareTo方法中使用它，但我不完全确定。如果有人能指出我正确的方向，我将非常感激。

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;

namespace WindowsFormsApplication2
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
            List<Task> tasks = new List<Task>();
            tasks.Add(new Task( "first",  DateTime.Now.AddHours(2)));
            tasks.Add(new Task( "second", DateTime.Now.AddHours(4)));
            tasks.TrimExcess();
            tasks.Sort();
        }
    }
    public class Task : IComparable
    {
        public Task()
        {
        }
        public Task(string nameIn, DateTime dueIn)
        {
            nameOfTask = nameIn;
            dateDue = dueIn;
        }
        DateTime dateDue;
        string nameOfTask;

        public static bool operator <(Task t1, Task t2)
        {
            return (t1.dateDue < t2.dateDue);
        }
        public static bool operator >(Task t1, Task t2)
        {
            return (t1.dateDue > t2.dateDue);
        }
        public static bool operator ==(Task t1, Task t2)
        {
            return (t1.dateDue == t2.dateDue);
        }
        public static bool operator !=(Task t1, Task t2)
        {
                return (t1.dateDue != t2.dateDue);

        }
        public override int GetHashCode()
        {
            return Int32.Parse(this.dateDue.ToString("yyyymmddhhmmss"));
        }
        public override bool Equals(System.Object obj)
        {
            if (obj == null) return false;    

            Task t = obj as Task;
            if ((System.Object)t == null) return false;
            return (this.dateDue == t.dateDue);
        }
        int IComparable.CompareTo(object obj)
        {
            if (obj == null) return 1;

            Task t = obj as Task;
            if (t != null)
            {
                return this.dateDue.CompareTo(t.dateDue);
            }
            else
                throw new ArgumentException("Object is not a Task");
        }
    }
}

当我评论binaory操作符时，程序按预期运行。我的问题是如何保护我的二元运算符免受空引用的影响，以便我可以将它们保存为手动比较？谢谢你的时间。

Answer 1

到目前为止给出的答案都是错误的。接受的答案是错误的，因为它不小心递归。另一个答案是错误的，因为它表示null不等于null。

你的运营商的实施都是错的; 他们需要正确处理空输入。

你的GetHashCode实现已经深深打破;您尝试将十四位数字放入可接受九位数的格式中。只需在日期调用GetHashCode;没有必要经历这种把它变成一个字符串然后把它变成一个数字的繁琐！

编写代码的正确方法是使用object.ReferenceEquals进行参考比较，而不是使用==和!=运算符;意外递归太容易了。

典型模式如下：

public static bool operator ==(Task t1, Task t2)         
{
    if (object.ReferenceEquals(t1, t2)) return true;
    // All right. We know that they are (1) not the same object, and
    // (2) not both null. Maybe one of them is null.
    if (object.ReferenceEquals(t1, null)) return false;
    if (object.ReferenceEquals(t2, null)) return false;
    // They are not the same object and both are not null.
    return t1.dateDue == t2.dateDue;
}

public static bool operator !=(Task t1, Task t2)         
{
    // Simply call the == operator and invert it.
    return !(t1 == t2);
}

public override bool Equals(object t)
{
    return (t as Task) == this;   
}

public override int GetHashCode()
{
    return this.dateDue.GetHashCode();
}

其他比较运算符留作练习。

Answer 2

您将与Task进行比较的!=个对象之一设置为null。内置运算符!=比较引用并且不会中断，但是运算符会尝试取消引用任务，并且会中断。

public static bool operator !=(Task t1, Task t2) {
    if (ReferenceEquals(t1, null)) {
        return !ReferenceEquals(t2, null); // return true only if t2 is *not* null
    }
    if (ReferenceEquals(t2, null)) {
        return true; // we know that t1 is not null
    }
    return (t1.dateDue != t2.dateDue);
}

当两个任务都是false时，此实现返回null。您应该在==运算符中实现对称空值检查。

Answer 3

 public static bool operator !=(Task t1, Task t2) 
        { 
              if (null == t1 || null == t2) { return false;}
                return (t1.dateDue != t2.dateDue); 

        }

如何正确地覆盖平等？

3 个答案: