Question

我从C ++开始，我试图通过合成来扩展std :: map类。我想使用此类CardPackage来获取指向我EntityCards的ID和指针的私有地图，并实现三种方法AddCard，GetCard和RemoveCard访问私人地图m。

问题是我无法使用m方法（find和end），因为我收到以下错误消息：

expression must have a pointer type

我知道m需要是一个指针，可以通过箭头符号（或点）访问，但我无法弄清楚是否需要调整代码以满足我的要求。

部首：

#include <EntityCard.h>

class CardPackage
{
public:

CardPackage();
~CardPackage();

void AddCard(EntityCard* card);
EntityCard* GetCard(int id);
bool RemoveCard(int id);

private:
    map<int, EntityCard*> m;
};

来源：

#include "CardPackage.h"

CardPackage::CardPackage()
{
}


CardPackage::~CardPackage()
{
}

   void CardPackage::AddCard(EntityCard *card)
   {
        m[card->ID] = card;
   }

EntityCard* CardPackage::GetCard(int id)
{
    if (id < 1) { return nullptr; }
    if(m->find(id) == m->end())
    {
        return (m[id]);
    }
    else
    {
        return nullptr;
    }
}

bool CardPackage::RemoveCard(int id)
{
    //TODO
    return false;
}

Answer 1

   if(m->find(id) == m->end())

应该是：

   if(m.find(id) != m.end())

m不是指针，因此您需要.，而不是->。你也有错误的测试方法。

稍微好一点的实现是：

   const auto it = m.find(id);
   if (it != m.end())
       return it->second;
   else
       return nullptr;

不同之处在于，这只会在地图中进行一次查找，而您的代码会执行两次查找。

从私有std :: map获取价值

1 个答案: