Сортировать уникальные значения с карты

Question

У меня есть пользовательская карта в моем заголовочном файле

class Tramway
{
private:
    using stations = std::vector <std::string>;
    using Tramlines = std::map <std::string, stations>;
...../

Я пытаюсь отсортировать уникальные значения, но пока мой подход дает мне ошибки компиляции. Вот мой код.

void Tramway::print_stations(const Tramway::Tramlines &tramlines)
{

    for(auto map_iter = tramlines.cbegin(); map_iter != tramlines.cend(); ++map_iter)
    {
        std::unique(map_iter->second.cbegin(), map_iter->second.cend());

        std::sort(map_iter->second.begin(), map_iter->second.end());

       for( auto vec_iter = map_iter->second.cbegin() ; vec_iter != map_iter->second.cend() ; ++vec_iter )
           std::cout << *vec_iter << std::endl;
    }

}

Есть ли способ получить уникальные значения и отсортировать их одновременно? Я пробовал std::sort(std::unique(..,..)), но std::sort() нужны два аргумента для работы.

Ниже приведены ошибки, которые я продолжаю получать

    x86_64-w64-mingw32\7.3.0\include\c++\bits\stl_algobase.h:148: error: no matching function for call to 'swap(const std::__cxx11::basic_string<char>&, const std::__cxx11::basic_string<char>&)'
           swap(*__a, *__b);
           ~~~~^~~~~~~~~~~~

x86_64-w64-mingw32\7.3.0\include\c++\bits\stl_algo.h:975: error: passing 'const std::__cxx11::basic_string<char>' as 'this' argument discards qualifiers [-fpermissive]
    *++__dest = _GLIBCXX_MOVE(*__first);
              ^

x86_64-w64-mingw32\7.3.0\include\c++\bits\stl_algo.h:1852: error: passing 'const std::__cxx11::basic_string<char>' as 'this' argument discards qualifiers [-fpermissive]
        *__first = _GLIBCXX_MOVE(__val);
                 ^

x86_64-w64-mingw32\7.3.0\include\c++\bits\stl_heap.h:252: error: passing 'const std::__cxx11::basic_string<char>' as 'this' argument discards qualifiers [-fpermissive]
       *__result = _GLIBCXX_MOVE(*__first);
                 ^

x86_64-w64-mingw32\7.3.0\include\c++\bits\move.h:187: error: no type named 'type' in 'struct std::enable_if<false, void>'

86_64-w64-mingw32\7.3.0\include\c++\bits\stl_heap.h:225: error: passing 'const std::__cxx11::basic_string<char>' as 'this' argument discards qualifiers [-fpermissive]
    *(__first + __holeIndex) = _GLIBCXX_MOVE(*(__first + __secondChild));
                             ^

86_64-w64-mingw32\7.3.0\include\c++\bits\stl_heap.h:231: error: passing 'const std::__cxx11::basic_string<char>' as 'this' argument discards qualifiers [-fpermissive]
    *(__first + __holeIndex) = _GLIBCXX_MOVE(*(__first
                             ^
x86_64-w64-mingw32\7.3.0\include\c++\bits\stl_algobase.h:548: error: passing 'const std::__cxx11::basic_string<char>' as 'this' argument discards qualifiers [-fpermissive]
      *--__result = std::move(*--__last);
      ~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~

Answer 1

Три задачи:

1. std::unique требует модифицируемых итераторов. Вы не должны передавать им cbegin/cend. Функция также должна принимать изменяемую ссылку.

2. std::sort сначала, std::unique после. std::unique удаляет только смежные эквивалентные элементы.

3. std::unique не имеет разрешения на изменение размера вектора. Он просто помещает полученную последовательность в начало вектора. Используйте идиому remove-erase для удаления удаленных элементов.

Фиксированный код:

void Tramway::print_stations(Tramway::Tramlines& tramlines)
{
    for (auto map_iter = tramlines.cbegin(); map_iter != tramlines.cend(); ++map_iter)
    {
        std::sort(map_iter->second.begin(), map_iter->second.end());
        map_iter->second.erase(std::unique(map_iter->second.begin(), map_iter->second.end()), map_iter->second.end());

       for (auto vec_iter = map_iter->second.cbegin(); vec_iter != map_iter->second.cend(); ++vec_iter)
           std::cout << *vec_iter << '\n';
    }    
}

Также рассмотрите возможность использования дляЦиклы и ссылки для упрощения кода:

void Tramway::print_stations(Tramway::Tramlines& tramlines)
{
    for (auto& tramline : tramlines) {
        auto& station = tramline.second;
        std::sort(station.begin(), station.end());
        station.erase(std::unique(station.begin(), station.end()), station.end());

        std::copy(station.begin(), station.end(),
                  std::ostream_iterator<stations>{std::cout, '\n'});
    }    
}