Erreur: pas viable surcharge de l'opérateur[]

C'est un peu mon code:

#include "pugi/pugixml.hpp"

#include <iostream>
#include <string>
#include <map>
int main() {
    pugi::xml_document doca, docb;
    std::map<std::string, pugi::xml_node> mapa, mapb;

    if (!doca.load_file("a.xml") || !docb.load_file("b.xml"))
        return 1;

    for (auto& node: doca.child("site_entries").children("entry")) {
        const char* id = node.child_value("id");
        mapa[new std::string(id, strlen(id))] = node;
    }

    for (auto& node: docb.child("site_entries").children("entry"))
        const char* idcs = node.child_value("id");
        std::string id = new std::string(idcs, strlen(idcs));
        if (!mapa.erase(id)) {
            mapb[id] = node;
        }
    }

Lors de la compilation, j'obtiens cette erreur: 

src/main.cpp:16:13: error: no viable overloaded operator[] for type 'std::map<std::string, pugi::xml_node>'
        mapa[new std::string(id, strlen(id))] = node;
  • new std::string... --> std::string
  • Êtes-vous java homme? Parce que je pense que le langage Java ici: std::string id = new std::string(idcs, strlen(idcs)); . Vous n'avez pas new locales les variables en C++.
  • De nos jours, vous n'avez pas new rien en C++.
InformationsquelleAutor Jimmy | 2015-04-17
  1. 7

    Vous avez une incompatibilité de type. mapa est de type:

    std::map<std::string, pugi::xml_node> mapa,
         ^^^^^^^^^^^^

    Mais vous êtes en train de faire:

    mapa[new std::string(id, strlen(id))] = node;
     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
         string*

    std::map a deux surcharges de operator[]:

    T& operator[](const Key& );
T& operator[](Key&& );

    Dans votre cas, Key est std::string. Mais vous êtes en essayant de passer dans std::string*, pour qui il n'y a pas de conversion à std::string - d'où vous obtenez une erreur "non viable surchargé operator[]".

    Ce que vous vouliez faire, c'était:

    mapa[id] = node;

    Même commentaire pour cette ligne:

    std::string id = new std::string(idcs, strlen(idcs));

    C++, Java n'est pas, vous venez de le faire:

    std::string id(idcs, strlen(idcs));

    ou tout simplement:

    std::string id = idcs;
    • Je vous remercie beaucoup. Ce qui semble signaler ces erreurs: src/main.cpp:21:24: error: use of undeclared identifier 'idcs' std::string id(idcs, strlen(idcs)); ^ src/main.cpp:23:24: error: use of undeclared identifier 'node' mapb[id] = node; Est-ce lié au nouveau code ou est-ce une question distincte?
    • Vous le déclarer sur la ligne avant (const char* idcs = node.child_value("id");)?
    InformationsquelleAutor Barry