Как применить преобразование к карте STL в C ++ - PullRequest
20 голосов
/ 24 октября 2011

В C ++ я использую transform, чтобы изменить все значения карты в верхний регистр.

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

Это дает мне следующую ошибку компиляции:

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

Я думаючто-то не так с типом аргумента в моем лямбда-выражении.Возможно, это что-то простое, но я не могу понять, что ожидается.

1 Ответ

22 голосов
/ 24 октября 2011

Вам не хватает const в первом типе пары.

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

Однако это не ваша проблема: вы не можете использовать map в качестве OutputIterator, поскольку они не поддерживают присваивание.Однако вы можете изменить второй аргумент, используя std::for_each.

Старый добрый map_to_foobar:

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

Концептуальные вещи: вызов transform с тем же диапазоном, что и для ввода и вывода,вполне законно и имеет смысл, если все ваши функторы возвращаются по значению и не изменяют свои аргументы.Однако изменение чего-либо на месте может быть быстрее (или, по крайней мере, выглядеть быстрее в коде, не говоря уже о оптимизирующем компиляторе) и имеет большой смысл с функциями-членами.

...