重载模板运算符调用独立类运算符

Question

我有一个模板类包含其他类的优先级队列，我需要使用优先级重载器调用各个类重载器，根据各个类的偏好进行比较（在这种情况下是年龄，在另一个类中可能是价格。

我绝对相信我已经实现了不正确的运算符重载，因此非常感谢您的建议。

举个例子

#include <iostream>
#include <queue>
#include <string>

using namespace std;    

class Animal {
    public:
        Animal();
        Animal(string t, int a);
        int get_age()const;
        bool operator< ( Animal& b) const;
        void display()const;
    private:
        string type;
        double age;
};

void Animal::display() const
{
    cout << "Type: " << type << "    Age: " << age;
}
int Animal::get_age() const
{
    return age;
}

Animal::Animal(){}

Animal::Animal(string t, int a)
{
    type = t;
    age = a;
}

bool Animal::operator< ( Animal& b) const
{
    return b.get_age();
}

template<typename T>
class Collection {
    public:
        Collection();
        Collection(string n, string d);
        void add_item(const T& c); 
    private:
        priority_queue <T> pets;
        string name; // Name of the collection
        string description; // Descriptions of the collection
};

template<typename T>
Collection<T>::Collection(){}

template<typename T>
Collection<T>::Collection(string n, string d)
{
    name = n;
    description = d;
}

template<typename T>
bool operator<(const T& one, const T& two) 
{
     return one.operator<(two);
}

template<typename T>
void Collection<T>::add_item(const T& c)
{
    pets.push(c);
}

int main(){
    Animal p1("Dog", 10);
    Animal p2("Cat", 5);
    Animal p3("Turtle", 24);
    Collection<Animal> P("Pets", "My Pets");
    P.add_item(p1);
    P.add_item(p2);
    P.add_item(p3);
    cout << endl;

    return 0;
}

我得到这个错误，我不知道我需要做什么来修复它。我必须将类重载保持为单个变量（Animal）

任务cpp:在“布尔运算符”的实例化中

Answer 1

bool Animal::operator< ( Animal& b) const

Animal也应该是const。您还需要比较这两个参数，否则（例如您的priority\u队列）将需要

您不使用来自动物的任何非公共内容，因此我建议您将其更改为

bool operator< (const Animal & lhs, const Animal & rhs)
{ return lhs.get_age() < rhs.get_age(); } 

这样做的好处是对双方一视同仁，而不是含蓄地对待一方。

template<typename T>
bool operator<(const T& one, const T& two) 
{
     return one.operator<(two);
}

此模板匹配所有类型，完全是多余的<代码>a

Answer 2

你的比较

bool Animal::operator< ( Animal& b) const
{
    return b.get_age();      // returns true always unless age == 0 
}

不进行比较，它应该采用const参数。你应该吃点类似的东西

bool Animal::operator< (const Animal& b) const 
                       // ^----------------------- const !
{
    return get_age() < b.get_age();
}

顺便说一句，您不需要使用成员运算符