プライベートstd :: mapから値を取得

Question

私はC++から始まり、std :: mapクラスをコンポジションで展開しようとしています。 私はEntityCardsにIDとポインタのプライベートマップを持っているために、このクラスCardPackageを使用してプライベートマップmにアクセスする三つの方法AddCard、GetCardとRemoveCardを実装したいです。

expression must have a pointer type

私はmニーズはへのポインタであることがあることを理解する：

問題は、私は、次のエラーメッセージが表示されますので、m方法（findとend）で動作することができていないということです矢印表記（またはドット）でアクセスできますが、要件を尊重するようにコードを適応させることはできません。

ヘッダー：

#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は、ポインタではありませんので、あなたは.、というよりも->を必要とします。あなたはまた、間違った方法でテストを持っています。

Aわずかに良い実装は次のようになります。

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

違いはあなたのコードは、2つを行うのに対し、これはのみ、マップで1つのルックアップを行うことです。