Wie bewerben Sie sich transformieren, um eine STL-map in C++

In C++, ich bin mit der Transformation zu ändern, die Werte einer map in Großbuchstaben um.

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

Dieser gibt mir folgende Kompilierungsfehler:

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

Ich glaube, es gibt etwas falsch mit der Art der Argumentation in meinem lambda-Ausdruck. Es ist wahrscheinlich etwas einfach, aber ich kann nicht scheinen, um herauszufinden, was erwartet wird.

  • Anstatt davon ausgehen, ein container speichert eine bestimmte Art. Sie können den Zugriff auf die Typ-Informationen über value_type. std::map<std::string, std::string>::value_type
  • danke, ich Lerne noch c++ - Idiome...
InformationsquelleAutor daj | 2011-10-24
  1. 22

    Ihnen fehlt das const bei der ersten Art von paar.

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

    Dies ist jedoch nicht Ihr problem: Sie können nicht mit einem map als OutputIterator, als Sie keine Unterstützung für die Zuordnung. Sie können jedoch mutieren das zweite argument mit std::for_each.

    Guten alten map_to_foobar:

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

    Konzeptionelle Sachen: Aufruf transform mit der gleichen Palette als ein-und Ausgang ist durchaus legitim und macht eine Menge Sinn, wenn alle funktoren Rückkehr von Wert und mutieren nicht Ihre Argumente. Allerdings mutierend etwas kann ein schneller (oder zumindest schneller Blick in den code, nevermind der optimierende compiler) und macht sehr viel Sinn, mit member-Funktionen.

    • Hinzufügen eines const-gibt mir eine andere Fehlermeldung /opt/local/include/gcc46/c++/bits/stl_pair.h:156:2: error: passing 'const std::basic_string<char>' as 'this' argument of 'std::basic_string<_CharT, _Traits, _Alloc>& std::basic_string<_CharT, _Traits, _Alloc>::operator=(const std::basic_string<_CharT, _Traits, _Alloc>&) [with _CharT = char, _Traits = std::char_traits<char>, _Alloc = std::allocator<char>, std::basic_string<_CharT, _Traits, _Alloc> = std::basic_string<char>]' discards qualifiers [-fpermissive]
    • Sorry, ich erkannte zu spät. Das Bearbeiten erklärt.
    • Alternativ können Sie std::transform mit einer transform-iterator. Zum Beispiel, mein key_iterator, posted in eine Antwort auf eine andere Frage, werden trivial in einem umgebauten value_iterator. Dann würden Sie am Ende mit std::transform(begin_values(data), end_values(data), begin_values(data), [](std::string s) { boost::to_upper(s); return s; });. In diesem Fall würde ich vorschlagen, dass std::for_each, obwohl, weil es vermeidet unnötiges kopieren.
    • Boost bietet die Palette Adapter boost::adaptors::map_values was funktioniert der trick. Darunter auch die unnötige Kopie natürlich.
    InformationsquelleAutor pmr
Schreibe einen Kommentar