STL Karte fügt kein Paar nach dem Entfernen der ersten Paare

Question

In diesem Stück Code füge ich ein Paar auf einer Karte und alles ist in Ordnung, aber wenn ich ein Paar, das nicht das letzte ist, die Karte nicht löscht Fügen Sie weitere Paare hinzu. Was mache ich falsch ??

SomeClass::add(Object object) 
if (!object.empty()) 
{ 
    ObjectList::iterator result = find(object.name()); 
    if (result == ObjectList.end()) 
    { 
     object.order(size() + 1); 
     ObjectList.insert(orderedObject(object.order(), object)); 
    } 
    else 
    { 
     ObjectList[result->first] = object; 
    } 
} 

Object und OrderedObject erklärt sich wie folgt:

typedef std::pair<int, Object> orderedObject; 
typedef std::map<int, Object> ObjectList; 

Dies ist der Löschcode:

SomeClass::eraseNamed(std::string aName) 
{ 
    if (!isEmpty()) 
    { 
     ObjectList::iterator result; 
     result = find(aName); 
     if (result != ObjectList.end()) 
     { 
      ObjectList.erase(result); 
      reorgObjectList(); 
      return true; 
     } 

    } 
    return false; 
} 

Für find-Methode:

ObjectList::iterator SomeClass::find(std::string aName) 
{ 
    ObjectList::iterator result = ObjectList.begin(); 
    while (result != ObjectList.end()) 
    { 
     if (aName == result->second.name()) 
      return result; 
     result++; 
    } 
    return result; 
} 

und für reorgObjectList:

bool SomeClass::reorgObjectList() 
{ 
    ObjectList::iterator i=ObjectList.begin(); 
    int j=1; 
    for (i = ObjectList.begin(); i != ObjectList.end(); ++i) 
    { 
     if(j!=i->second.order()) 
      i->second.order(j); 
     j++; 
    } 
    return true; 
} 

Irgendwelche Vorschläge ???

Answer 1

Nun, Sie sind auf die Größe der Karte Keying, scheint dies möglicherweise Ihre Probleme verursachen.

Also, wenn Sie 3 Dinge in der Karte haben, werden Sie

1 => Obj1 
    2 => Obj2 
    3 => Obj3 

haben, wenn Sie eines dieser Elemente entfernen, sagen auf 1, werden Sie

2 => Obj2 
    3 => Obj3 

haben dann später Sie gehen zu einfügen, und setzen Sie den Schlüssel auf "size() + 1", Größe wird 2 zurückgeben, und Sie werden versuchen, bei Schlüssel 2 + 1 == 3 einfügen. 3 ist bereits vergeben. Es wird also entweder überschrieben oder fehlgeschlagen (nicht sicher, wie Ihr Fund oben funktioniert).

Anstatt in der Größe + 1 einzufügen, würde ich den letzten Schlüssel prüfen und um 1 erhöhen, wenn Sie so den Schlüssel verwalten möchten.