"Klasse std::map verwendet, die ohne template-Parameter" - Fehler

Ich würde sagen, ich bin kein Experte in der Verwendung der STL. Hier ist mein problem, ich habe eine Klasse Namens LdapClientManager, die behauptet, eine Reihe von LDAP-clients, die verwaltet werden von ID. Der Behälter mit der LdapClients ist deklariert als eine member-variable, d.h.

typedef std::map<int, LdapClient *> LdapClientMap;
LdapClientMap _ldapClientMap;

Folgende Funktion schlägt fehl, um die Kompilierung mit der Fehlermeldung:

LdapClient * LdapClientManager::getLdapClient(unsigned int templateID)
{
    //Do we have an LdapClient
    LdapClientMap::const_iterator it = _ldapClientMap.find(templateID);
    if (it == std::map::end) {
        //no existing client, lets create it
        LdapClient * ldapClient = new LdapClient();
        if (ldapClient == NULL) {
            //TODO: handle out of memory condition
        }

        _ldapClientMap[templateID] = ldapClient;
        return ldapClient;
    }

    return it->second;
}

Leider bekomme ich die folgende Fehlermeldung zur compile-Zeit, was bedeutet es. Ich habe noch keine Lösung gefunden in google, wie noch.

LdapClientManager.cc: In member function LdapClient*
LdapClientManager::getLdapClient(unsigned int)':
LdapClientManager.cc:33:template-Klasse std::map' verwendet werden, ohne die template-Parameter

  • Nur ein Hinweis, neu nicht NULL zurückgeben, in der out-of-memory Bedingungen, (es sei denn, Sie sind mit alten Compilern wie VC6). Es wirft std::bad_alloc exception.
InformationsquelleAutor Matt | 2009-07-22
  1. 11

    Ersetzen std::map::end mit _ldapClientMap.end().
    Auch new nie 0 zurück, wirft es eine Ausnahme, wenn die Allokation fehlschlägt.

    Beachten Sie, dass das Programm gemacht werden kann, viel kürzer.

    LdapClient * LdapClientManager::getLdapClient(unsigned int templateID)
{
    LdapClient *& value = _ldapClientMap[templateID];
    if (value == 0)
        value = new LdapClient();
    return value;
}
    • typename vor LdapClientMap? Warum?!
    • Hoppla, entfernt. Es wäre notwendig, wenn getLdapClient gab eine Vorlage. Danke, EFraim.
    • In std::map-operator[] ist ein mutierend Betrieb. Dies kann Schränken die Nutzung (kann nicht verwendet werden, die innerhalb einer Konstanten Methode), oder es kann auf das system auswirken (einem späteren Aufruf von find() wird der Ertrag ein iterator in das array, wird die Karte wachsen im Speicher für jeden test fehlgeschlagen...)
    • Auch zu berücksichtigen, dass auch "pods" wie der Zeiger, der nicht initialisiert zu 0, so dass Ihr prüfen beim erstellen neuer client-Objekt falsch sein könnte
    • driberas, das problem wird nur auftreten, wenn die Allokation fehlschlägt (das passiert selten und wird in den meisten Fällen, dass ein Programm das Herunterfahren sowieso).
    • CsTamas, map's operator[] null-initialisiert der Wert. Der code funktioniert Prima.
    • Doh, scheint nun offensichtlich... typisch, ich habe es kopiert aus einem Beispiel, das war falsch. Danke, wusste nicht, new wirft eine exception, wenn die Zuteilung fials.
    • Dank für, die. Ich habe sich auf die Speicherung der Objekte selbst eher als Zeiger und Rücksendung _ldapClientMap[templateID]; für getLdapClient... noch einfacher!

    InformationsquelleAutor avakar
  2. 3

    Bedeutet es genau das, was es sagt, es bedeutet. std::map ist ein Klasse Vorlage. Es ist nicht eine Klasse, in der und von sich selbst. Es muss-template-Parameter, wie Sie verwendet werden, wenn Sie definiert die LdapClientMap geben. Später, sagen Sie std::map::end, und der compiler sagt, dass muss Parameter zu.

    Aber Sie meinte wohl _ldapClientMap.end(). Jede Karte hat seine eigenen Ende; end ist nicht eine statische Funktion, so müssen Sie rufen Sie es über eine Instanz. Wenn es waren statisch, würden Sie benötigt, um einen template-Parameter, genau wie bei der Definition der Art: std::map<int, LdapClient*>::end.

    InformationsquelleAutor Rob Kennedy
  3. 1

    std::map::end() ist eine Memberfunktion der container-Instanz und nicht ein universeller Wert, so müssen Sie überprüfen das Ergebnis der std::map::find() gegen _ldapClientMap.end().

    Noch ein paar Vorschläge, den code zu verbessern:

    • Standard-C++ Container-value-Semantik (Sie speichern möchten, die das eigentliche Objekt und nicht einen Zeiger auf das Objekt). Wenn Sie wirklich brauchen, um zu speichern Zeiger auf LdapClients anstelle des LdapClient-Objekte selbst, würde ich dringend empfehlen das einwickeln von Ihnen in einer entsprechenden smart-pointer wie boost::shared_ptr (nicht std::auto_ptr, die nicht funktionieren). Diese Möglichkeit, die automatische Speicherverwaltung der std::map wird immer noch funktionieren und zu zerstören die Objekte, wie Sie soll. Wenn Sie nicht möchten, verwenden Sie ein smart-pointer, oder legen Sie die aktuelle LdapClient-Objekt in den container, Sie müssen manuell die Objekte verwalten' Leben und rufen Sie löschen, wenn angemessen, um Speicherverluste zu verhindern. Meine Präferenz wäre, ändern Sie den Typ der map std::map, es sei denn, das LdapClient-Objekte sind polymorph.
    • Es sei denn, Sie verwenden einen sehr veralteten compiler, das Ergebnis überprüfen von regulären new() gegen 0 oder NULL wird nicht nachgeben keine neuen Erkenntnisse als neue wirft eine std::bad_alloc diesen Tagen, wenn es nicht zugewiesenen Speicher aus welchem Grund auch immer.
    • Anstatt _ldapClientMap[x] = y; fügen Sie ein neues element, das ich verwenden würde, _ldapClientMap.insert(LdapClientMap::value_type(x,y)), da letztere nicht überschreibt einen bestehenden Wert für den Schlüssel x (die frühere tun wird) und return 'false' falls die Schlüssel bereits in der Zuordnung vorhanden ist. Das ist natürlich, wenn das ist Ihre Absicht.
    • Vielen Dank für die Hinweise, es gab einige gute Kommentare drin.
    InformationsquelleAutor Timo Geusch
  4. -1
    LdapClientMap _ldapClientMap;

    Sollten Sie es vermeiden, die Verwendung von Bezeichnungen mit einem führenden Unterstrich. Technisch ist es Undefiniertes Verhalten, auch wenn der compiler es zulässt, weil durch Sie in Konflikt mit aktuellen oder zukünftigen reservierten Namen.

    • Das ist es nicht. Einzelnen führenden Unterstrich vor Kleinbuchstaben ist nur in der globalen Umfang; Klasse-Mitglieder und lokale Variablen sind in Ordnung, und er erklärt, eine member-variable. Einzelnen führenden Unterstrich vor einem Großbuchstaben und doppelten Unterstrich überall in Bezeichner sind reserviert, in allen Bereichen (daher der Grund, warum alle STL-Implementierungen verwenden _Foo für argument-und member-Namen - als standard, der explizit sagt, dass Benutzer-code neu definieren diese Bezeichner über Makros U. B.).
    • Ich benutze immer ein einzelner Unterstrich für member-Variablen zu unterscheiden.
    InformationsquelleAutor navigator
Schreibe einen Kommentar