Cómo aplicar la transformación a un mapa STL en C++

Question

En C++, estoy usando la transformación para cambiar todos los valores de un mapa a mayúsculas.

std::map<std::string, std::string> data = getData(); 

    // make all values uppercase 
    std::transform(data.begin(), data.end(), data.begin(), 
     [](std::pair<std::string, std::string>& p) { 
      boost::to_upper(p.second); 
      return(p); 
     }); 

Esto me da el siguiente error de compilación:

/opt/local/include/gcc46/c++/bits/stl_algo.h:4805:2: error: no match for call to '(main(int, char**)::<lambda(std::pair<std::basic_string<char>, std::basic_string<char> >&)>) (std::pair<const std::basic_string<char>, std::basic_string<char> >&) 

Creo que hay algo mal con el tipo del argumento en mi expresión lambda. Probablemente sea algo simple, pero parece que no puedo entender lo que se espera.

Answer 1

Te estás perdiendo la const en el primer tipo del par.

[](std::pair<const std::string, std::string>& p) { 

Sin embargo esto no es su problema: No se puede utilizar un map como el OutputIterator, ya que no son compatibles con la asignación. Sin embargo, puede mutar el segundo argumento usando std::for_each.

El bueno de map_to_foobar:

std::for_each(data.begin(), data.end(), 
       [](std::pair<const std::string, std::string>& p) { 
       p.second = "foobar"; 
       }); 

conceptual cosas: Llamada transform con el mismo rango que la entrada y salida es muy de fiar y tiene mucho sentido si todos sus funtores regresan por su valor y no mutan su argumentos. Sin embargo, mutar algo en su lugar puede ser más rápido (o al menos parecer más rápido en el código, sin importar el compilador de optimización) y tiene mucho sentido con las funciones de los miembros.