Удалить дубликаты в std :: list [closed]

Question

Удалить дубликаты в std :: list [closed]

0

У меня есть список строк, как я могу удалить дублирующее значение в std :: list, например

std::list<std::string> listName;

listName.push_back("Foo");
listName.push_back("Bar");
listName.push_back("Foo");

Затем я хочу удалить дублирующее имя, которое является "Foo". Остальное - это только имя "Бар".

Благодарю!

domlao 08 окт. 2013, в 02:38

Источник

Теги:

c++

3 ответа

Ещё вопросы

billz · Answer 1 · 2013-10-07T22-56-00.000Z

Если вы хотите только уникальные элементы в listName вы можете сделать:

listName.sort();  
listName.unique();

listName должно оставаться: "Foo", "Bar"

Если вы хотите избавиться от всех элементов, у вас есть дубликат, вы можете использовать std::list::sort для сортировки элементов, а затем использовать std::adjacent_find для поиска повторяющихся элементов.

listName.sort();   

std::list<std::string>::iterator ab = std::adjacent_find(listName.begin(), listName.end());
if (ab != listName.end())  // if duplicate elements are found
{
    std::list<std::string>::iterator ae = std::upper_bound(listName.begin(), listName.end(), *ab);  // try to locate last occurrence 
    if (std::distance(ab, ae) > 1)
    {
        listName.erase(ab, ae); // remove all dup elements
    }
}

listName остается: "Bar"

См. Здесь fore more std::list inerface

ОП указал, что остается только «Бар». Это также будет иметь "Foo".
Я только что применил этот подход, и это было неправильно, так как ОП хочет, And remaining is only "Bar" name.
Люди должны более внимательно прочитать вопрос и перестать рекомендовать list::unique() :)
Виноват. Учитывая производительность std::upper_bound на std::upper_bound итераторе (сравнение логов, но линейный поиск), поиск в прямом направлении может быть проще. Или хотя бы начать поиск с найденного итератора, а не begin ().
Я думаю, что вы можете заменить std::distance(ab, ae) > 1 на std::next(ab) != ae .
@DyP как насчет, если есть несколько дуплов?
@billz Не уверен, что ты имеешь в виду. std::distance(ab, ae) > 1 (обратите внимание на >1 ) должен быть эквивалентен std::next(ab) != ae , так как список отсортирован, а upper_bound возвращает следующий элемент, который больше переданного значения или второй аргумент, если такого нет.

dyp · Answer 2 · 2013-10-07T23-56-00.000Z

Сортируйте список, затем вы можете легко найти дубликаты.

#include <list>
#include <iostream>
#include <algorithm>

int main()
{
    std::list<std::string> listName = {"Foo", "Bar", "Foo", "Foobar"};

    listName.sort();

    for(auto i = listName.begin(); i != listName.end(); )
    {
        auto range = std::equal_range(i, listName.end(), *i);

        if(std::next(range.first) != range.second)
            i = listName.erase(range.first, range.second);
        else ++i;
    }

    for(auto const& e : listName) std::cout << e << ", ";
}

Самая короткая версия, которую я мог бы придумать, но повторяется дважды над каждым элементом, который нужно удалить (один раз в equal_range и один раз во время erase).

Альтернативно, повторяя один раз над каждым элементом:

    listName.sort();

    for(auto i = listName.begin(); i != listName.end(); )
    {
        auto j = std::next(i);
        bool duplicate = false;

        while(j != listName.end() && *j == *i)
        {
            duplicate = true;
            j = listName.erase(j);
        }

        if(duplicate) i = listName.erase(i);

        i = j;
    }

Как упражнение, перемещение duplicate = true из цикла:

    listName.sort();

    for(auto i = listName.begin(); i != listName.end(); )
    {
        auto j = std::next(i);
        bool duplicate = (j != listName.end() && *j == *i);

        if(duplicate)
        {
            do
            {
                j = listName.erase(j);
            }while(j != listName.end() && *j == *i);

            listName.erase(i);
        }

        i = j;
    }

Jonathan Potter · Answer 3 · 2013-10-07T21-59-00.000Z

Один из способов сделать это - использовать std::map вместо list. Карта будет хранить строки и подсчитывать, сколько раз они происходят. Единственное реальное изменение вашего кода - вам нужно использовать ++ вместо push_back чтобы поместить строку на карту;

std::map<std::string, int> listName;
++listName["Foo"];
++listName["Bar"];
++listName["Foo"];

Затем для удаления записей с числом больше 1:

for (auto it = listName.begin(); it != listName.end(); )
{
    if (it->second > 1)
        listName.erase(it++);
    else
        ++it;
}