Wert von privaten Std erhalten :: map

Question

Ich beginne mit C++ und ich versuche, std :: map Klasse über Komposition zu erweitern. Ich möchte diese Klasse CardPackage verwenden, um eine private Karte von IDs und Zeiger zu meinem EntityCards zu haben und drei Methoden AddCard, GetCard und RemoveCard zu implementieren, die auf die private Karte zugreifen werden m.

Das Problem ist, dass ich nicht in der Lage gewesen, mit m Methoden (find und end) zu arbeiten, weil ich die folgenden Fehlermeldung erhalten:

expression must have a pointer type

Ich verstehe, dass m Bedürfnisse ein Zeiger sein Der Zugriff erfolgt durch Pfeilnotation (oder Punkt), aber ich kann den Code nicht anpassen, um meine Anforderungen zu erfüllen.

Rubrik:

#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; 
}; 

Quelle:

#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()) 

Sollte sein:

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

m ist kein Zeiger, so dass Sie . brauchen, anstatt -> . Sie haben den Test auch falsch herum.

Eine etwas bessere Umsetzung wäre:

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

Der Unterschied besteht darin, dass dies tut nur eine Suche in der Karte, während der Code zwei tut.