Ошибка: не жизнеспособны перегруженный оператор []

Question

Это немного моего кода:

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

При компиляции я получаю эту ошибку:

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; 

Answer 1

У вас есть несоответствие типов. mapa имеет тип:

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

Но вы делаете:

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

std::map имеет два перегруженных operator[]:

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

В вашем случае, Key является std::string. Но вы пытаетесь перейти в std::string*, для которого нет конвертации в std::string - следовательно, вы получите сообщение об ошибке «нет жизнеспособных перегруженных operator[]».

Что ты хотел сделать, это:

mapa[id] = node; 

же комментарий к этой линии:

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

C++ не является Java, вы просто делаете:

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

или просто:

std::string id = idcs;