Come applicare una trasformazione a una mappa STL in C++

Question

In C++, sto usando la trasformazione per modificare tutti i valori di una mappa in maiuscolo.

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

Questo mi dà il seguente errore di compilazione:

/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> >&) 

Penso che ci sia qualcosa che non va con il tipo dell'argomento nella mia espressione lambda. Probabilmente è qualcosa di semplice, ma non riesco a capire cosa ci si aspetta.

Answer 1

Manca il const nel primo tipo della coppia.

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

Tuttavia questo non è il vostro problema: non è possibile utilizzare un map come OutputIterator, in quanto non supportano l'assegnazione. È possibile, tuttavia, modificare il secondo argomento utilizzando std::for_each.

buona vecchia map_to_foobar:

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

concettuale roba: Chiamare transform con la stessa gamma di input e output è abbastanza legittime e fa un sacco di senso se tutti i funtori ritornano per valore e non mutano la loro argomenti. Tuttavia, la modifica di qualcosa sul posto può essere più veloce (o almeno sembrare più veloce nel codice, non curare il compilatore di ottimizzazione) e ha molto senso con le funzioni dei membri.