Вам не хватает 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
с тем же диапазоном, что и для ввода и вывода,вполне законно и имеет смысл, если все ваши функторы возвращаются по значению и не изменяют свои аргументы.Однако изменение чего-либо на месте может быть быстрее (или, по крайней мере, выглядеть быстрее в коде, не говоря уже о оптимизирующем компиляторе) и имеет большой смысл с функциями-членами.