Question

所以我在这个错误上画了一个空白。无法比较数组中的两个元素。 Array.Sort（病人）;是错误产生的地方。我有一个IComparable接口，以及一个包含以下代码的类文件：尝试按患者ID号排序

class Patient : IComparable
{
    private int patientID;
    private string patientName;
    private int patientAge;
    private decimal amount;

    public int PatientId { get; set; }

    public string PatientName { get; set; }

    public int PatientAge { get; set; }

    public decimal PatientAmount { get; set; }


    int IComparable.CompareTo(Object o)
    {
        int value;
        Patient temp = (Patient)o;
        if (this.PatientId > temp.PatientId)
            value = 1;
        else if (this.PatientId < temp.PatientId)
            value = -1;
        else
            value = 0;
        return value;
    }
}

这就是我主要方法中的内容。没有添加Display（）因为现在没有添加任何东西，为什么它被注释掉

private static void Main(string[] args)
    {
        int numOfPatients =2 ;

        Patient[] patient = new Patient[numOfPatients];
        for (int x = 0; x < numOfPatients; x++)
        {


            int intvalue;
            decimal dollarValue;
            patient[x] = new Patient();

            Console.Write("Patient {0}: ", (x + 1));
            Console.WriteLine("Enter the Patients ID: ");
            bool isNum = int.TryParse(Console.ReadLine(), out intvalue);
            if (isNum)
            {
                patient[x].PatientId = intvalue;

            }
            else
            {
                Console.WriteLine("Patient ID was invalid. ID needs to be numbers");
                Console.WriteLine("Enter the Patients ID: ");
                int.TryParse(Console.ReadLine(), out intvalue);
            }

            Console.WriteLine("Enter the Patients Name: ");
            patient[x].PatientName = Console.ReadLine();

            Console.WriteLine("Enter the Patients Age: ");
            bool isAge = int.TryParse(Console.ReadLine(), out intvalue);
            if (isAge)
            {
                patient[x].PatientAge = intvalue;

            }
            else
            {
                Console.WriteLine("Patient Age was invalid. Age needs to be numbers");
                Console.WriteLine("Enter the Patients Age: ");
                int.TryParse(Console.ReadLine(), out intvalue);
            }

            Console.WriteLine("Enter the Patients Amount Due: ");
            bool isAmount = Decimal.TryParse(Console.ReadLine(), out dollarValue);
            if (isAmount)
            {
                patient[x].PatientAmount = dollarValue;

            }
            else
            {
                Console.WriteLine("Patient amount Due was invalid. Amount needs to be a numbers");
                Console.WriteLine("Enter the Patients Amount Due: ");
                int.TryParse(Console.ReadLine(), out intvalue);
            }


        }
        Array.Sort(patient);
        Console.WriteLine("Patients in order with Amounts Owed are: ");
        for (int i = 0; i < patient.Length; ++i) ;
        //Display(patient[i], numOfPatients);

Answer 1

我会写

return this.PatientId.CompareTo(temp.PatientId)

在重写CompareTo方法的类中。无需使用等号。这将为您进行int比较并返回正确的值。

我还建议你只使用IList类的一些实现，然后就可以使用LinQ语句了。使用它可以防止“数组”

中出现空值

Answer 2

有些事情会浮现在脑海中：

a）为什么不实施IComparable<Patient>？

b）为什么要重新实施int.CompareTo(int)？您对IComparable的实现只能返回this.PatientID.CompareTo(other.PatientID)。

c）当您对数组进行排序时，确定数组是否已满？我不确定如果它包含null会发生什么。

Answer 3

如果使用类型化数组传递Array.Sort，则会调用Array.Sort<T>(T[])重载。根据{{3}}，此重载使用IComparable<T>接口来比较对象。所以看起来你有两个选择：

您可以实施IComparable<T>而不是IComparable（更好）。
您可以将数组转换为Array以调用使用Array.Sort(Array)界面的IComparable重载（更糟糕）。

使用IComparable

3 个答案: