Zitat Zitat
Code: 
                if( id > 0 && id <= RPG::map->events.count() )
                    return RPG::map->events[id]->getName();
                break;
Keine gute Idee. Angenommen eine Map hat die Events 1, 2, 5 und 9. RPG::map->events.count() ist dann 4. \e[5] und \e[9] werden also fehlschlagen. Besser: if(RPG::map->events[id]) bzw. RPG::Event *event = RPG::map->events[id] und dann if(event) return event->getName()

Technisch betrachtet trifft das auch auf Actors und andere NamedCatalogPtr-Typen zu (count() sollte da nur für das Iterieren über alle, mit ->list[i], verwendet werden), aber bei denen sind eigentlich immer die IDs mit 1 beginnend ohne Lücken vorhanden.