Question

我正在尝试按名称排序指向对象的数组，但我不知道我是否采用了正确的方法。到目前为止，这是我的代码......

主

Person *personArray[3]; //pointers to person objects

personArray[0] = new Employee("Bill", 1200);
personArray[1] = new Customer("Steve");
personArray[2] = new Employee("Bill", 1200);

Person *tempArray[3];
string temp1;

for(int i=3-1;i>0;i--) 
{
    int min = 0;
    for(int j=1;j<i;j++)
    {
        if(*tempArray[j] < *personArray[min])
        {
            min = j;
        }
    }
  temp1 = tempArray[min]->printname();
  tempArray[min] = tempArray[i];
  tempArray[i] = temp1;
}

class Person
    {
    public:
      Person(string); 
      virtual void printname() = 0;
      bool operator <(const Person& name1) const;
      bool operator ==(const Person& name1) const;

    protected:
      string name;
    };

    bool Person::operator <(const Person& name1) const
    {
       return (this->name < name1.name);
    }
    bool Person::operator ==(const Person& name1) const
    {
       return (this->name == name1.name);
    }

    void Person::printname()
    {
      cout << "Name: " << name << endl;
    }

Answer 1

我认为这一行是问题所在：

if(*tempArray[j] < *personArray[min])

应该是：

if(*personArray[j] < *personArray[min])

此时尚未初始化tempArray。

更重要的是，临时对象应该足够了，而不是整个数组。

这些线......

temp1 = tempArray[min]->printname();
tempArray[min] = tempArray[i];
tempArray[i] = temp1;

太。 tempArray什么都没有，应该是这样的：

temp1 = personArray[min]->printname();
personArray[min] = personArray[i];
personArray[i] = temp1;

BTW，temp1应与您的对象类型相同（不是字符串）。

将指针数组排序到对象

1 个答案: