从地图复制到指针列表

Question

我有一个有趣的任务，我有一个 std::map 的 CTurist（上一课）和 unsigned 变量。代码如下：

class CTurist
{
protected:
    string tName;
    int age;

public:
    CTurist() {};

    CTurist(string name, int age2)
    {
        tName = name;
        age = age2;
    }


    bool operator<(const CTurist& e) const
    {
        return age < e.age;
    }

    friend ostream& operator<<(ostream& os, const CTurist&& e);
    friend ifstream& operator>>(ifstream& is, CTurist&& e);
};

class CHotel:public CTurist
{
protected:

    string hName;
    int stars;
    int beds;
    map<CTurist, unsigned> Turisti;

public:
    unsigned Sum = 0;

    CHotel(){};

    CHotel(string hName2, int zvezdi, int legla)
    {
        hName = hName;
        stars = zvezdi;
        beds = legla;
    }



    int Compare()
    {
        list<CTurist*> list;

        int br=0;
        CTurist vyzrast;
        map<CTurist, unsigned>::iterator it = Turisti.begin();
        while (it != Turisti.end())
        {
            if (it->first < vyzrast)
            {
                br++;
            }
            else
            {
                list.push_back(std::move(&it->first));
            }
        }
    }
};

我知道这很长，但我认为最好把所有信息都告诉你。

现在，底部的int Compare() 函数是给我带来问题的函数。

我必须检查游客的年龄是否高于或低于我在这里称为vyzrast 的参数。我们正在比较age。如果它在下面，那就很简单了。

如果它在上面，我必须将这些游客添加到list<CTurist*>，又名。指向指针列表。如果我从对象而不是指针创建列表。运气不好，因此我在这里寻找如何解决它的建议。

Answer 1

您为什么对operator 重载使用右值引用？您应该使用const CTurist & 来表示operator<< 和CTurist& 来表示operator>>。你应该使用std::istream 代替operator>>：

friend ostream& operator<<(ostream& os, const CTurist &e);
friend istream& operator>>(istream& is, CTurist &e);

除此之外，Compare() 在填充std::list 时根本没有理由使用std::move()，因为它是添加指针，而不是移动实际对象。

Compare() 不正确有几个原因：

1. it->first 是一个实际的const CTurist 对象，但您的std::list 期待CTurist* 指针。 std::map 键是 const，所以 &it->first 是 CTurist const * 指针（指向 const 对象的非 const 指针），而不是像您期望的 CTurist* 指针（指向非 const 对象的非 const 指针） .

2. 您的 vyzrast 对象未初始化。您根本没有为其age（和tName）成员分配任何值，因此您的比较结果是不确定的。

3. 您不会在每次循环迭代时递增您的 it 迭代器，因此如果 std::map 不为空，您将陷入无限循环。

试试类似的方法：

int Compare()
{
    std::list<const CTurist *> clist;

    int br = 0;
    CTurist vyzrast("", SomeAgeValueHere);

    std::map<CTurist, unsigned>::iterator it = Turisti.begin();
    while (it != Turisti.end())
    {
        if (it->first < vyzrast)
        {
            br++;
        }
        else
        {
            clist.push_back(&it->first);
        }
        ++it;
    }

    // use clist as needed...

    return SomeValueHere;
}

Live Demo