Question

我正在尝试将多个键值对添加到https://www.w3schools.com/bootstrap4/bootstrap_get_started.asp中。由于某些原因，仅将我添加的第一个键值对添加到地图中，而找不到第二个键值对。

#include<string>
#include<iostream>
#include<map>
using namespace std;

struct RJNodeAddress
{
            string m_ip;
            string m_port;
            bool operator<(const RJNodeAddress &l)const
            {
                    int l_size=l.m_ip.size();
                    int r_size=l.m_port.size();                                          
                    return (l_size < r_size);
            }
};

struct RJNodeDetails
{
        string m_NodeType;
        int        m_appId;
};

class RJWebsocket
{
        public:
            static map<RJNodeAddress,RJNodeDetails> m_Nodes;
};

map<RJNodeAddress,RJNodeDetails> RJWebsocket::m_Nodes;

int main()
{
        RJNodeAddress l_node1,l_node2;
        RJNodeDetails l_add1,l_add2;

        l_node1.m_ip="172.16.129.68";
        l_node1.m_port="8766";
        l_node2.m_ip="172.16.128.130";
        l_node2.m_port="8768";

        l_add1.m_appId=0;
        l_add1.m_NodeType="CM";

        l_add1.m_appId=1;
        l_add1.m_NodeType="CM";

        RJWebsocket::m_Nodes.insert({l_node1,l_add1});
        RJWebsocket::m_Nodes.insert({l_node2,l_add2});

    for(const auto& j:RJWebsocket::m_Nodes)
    {
    cout<<endl<<"Iterating through RJWebsocket::m_Nodes"<<endl;
    cout<<endl<<"port: "<<j.first.m_port<<"  "<<"IP: "<<j.first.m_ip<<endl;
    }

}

上面的代码给出以下输出：

Iterating through RJWebsocket::m_Nodes

port: 8766  IP: 172.16.129.68

Answer 1

您的比较运算符做得很奇怪，我建议像这样std::tuple来依靠它：

#include <tuple>

bool operator<(const RJNodeAddress &l)const
{
    return std::tie(m_ip, m_port) < std::tie(l.m_ip, l.m_port);
}

这样，就可以正确执行std::map::insert中的等效性检查，并且您在main函数中的代码段确实构造了两个条目。然后我机器上的输出是

Iterating through RJWebsocket::m_Nodes

port: 8768  IP: 172.16.128.130

Iterating through RJWebsocket::m_Nodes

port: 8766  IP: 172.16.129.68

请注意，正如@DanielLangr在评论中指出的那样，RJNodeAddress也可以直接用作std::pair<std::string, std::string>或std::tuple<std::string, std::string>的类型别名，该别名提供了字典上的operator <框，然后退回到std::string::operator <。这免除了您手动指定比较的麻烦，但有两个缺点：它不提供命名struct及其数据成员的表达能力，并且按字典顺序进行的比较可能并非您的初衷。 / p>

Answer 2

您的运营商不尊重compare requirement

bool operator<(const RJNodeAddress &l)const
{
        int l_size=l.m_ip.size();
        int r_size=l.m_port.size();                                          
        return (l_size < r_size); // Wrong
}

您可能想要类似的东西：

bool operator<(const RJNodeAddress &rhs)const
{
    auto as_tuple = [](const RJNodeAddress & a){std::tie(a.m_ip, a.m_port); };
    return as_tuple(*this) < as_tuple(rhs);
}

Answer 3

您的operator<应该在两个不同的RJNodeAddress之间进行比较，您似乎只看操作数（l）。尝试这样的事情：

        bool operator<(const RJNodeAddress &l)const
        {
            if (m_ip == l.m_ip)
                return m_port < l.m_port;
            return m_ip < l.m_ip;
        }

无法添加多个键以将结构作为键的映射

3 个答案: