Форум программистов, компьютерный форум CyberForum.ru

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
 
Рейтинг: Рейтинг темы: голосов - 12, средняя оценка - 4.75
booker
23 / 23 / 2
Регистрация: 21.11.2012
Сообщений: 106
#1

vector::erase()? - C++

30.12.2012, 12:53. Просмотров 1476. Ответов 30
Метки нет (Все метки)

есть вектор структур

C++
1
2
3
4
5
struct person{
    string name;
    int age;
 
vector <person> repw;
задача отсортировать вектор так, чтобы каждый последующий контейнер при условии совпадений поля name был удалён.

пытаюсь так

C++
1
2
3
4
int it1,it2;
    for (it1 = 0;it1!=repw.size();it1++)
        for (it2 = it1+1;it2!=repw.size();it2++)
            if(repw[it1].name==repw[it2].name) repw.erase(repw.begin()+it2);
не выходит. С итераторами колдовать?
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Croessmah
Модератор
Эксперт CЭксперт С++
12702 / 7176 / 801
Регистрация: 27.09.2012
Сообщений: 17,702
Записей в блоге: 2
Завершенные тесты: 1
30.12.2012, 12:56     vector::erase()? #2
Цитата Сообщение от booker Посмотреть сообщение
С итераторами колдовать?
А почему бы и нет?
remove_copy_if же есть в STL
booker
23 / 23 / 2
Регистрация: 21.11.2012
Сообщений: 106
30.12.2012, 13:34  [ТС]     vector::erase()? #3
хехе еслиб силён я был в STL))

допустим определяю я итератор по правилам

vector<person> iterator it1;

но в структуре person поле name является типом string, т.е. размер всё равно не предопределён.

поэтому непонятно как быть.
go
Эксперт C++
3585 / 1365 / 128
Регистрация: 16.04.2009
Сообщений: 4,528
30.12.2012, 13:39     vector::erase()? #4
booker, я не понимаю задание. Или есть конкретные вопросы?
Avazart
7044 / 5221 / 259
Регистрация: 10.12.2010
Сообщений: 22,945
Записей в блоге: 17
30.12.2012, 14:23     vector::erase()? #5
Походу нужно обеспечить уникальность элементов в контейнере.
Vourhey
Почетный модератор
6473 / 2248 / 123
Регистрация: 29.07.2006
Сообщений: 12,635
30.12.2012, 14:26     vector::erase()? #6
Цитата Сообщение от Avazart Посмотреть сообщение
Походу нужно обеспечить уникальность элементов в контейнере.
Тогда есть алгоритм unique, которому можно передать предикат, где реализовать сравнение имет структур.

Не по теме:

Это ответ автору ) просто, задание вроде такое, поэтому процитировал тебя

Avazart
7044 / 5221 / 259
Регистрация: 10.12.2010
Сообщений: 22,945
Записей в блоге: 17
30.12.2012, 14:32     vector::erase()? #7
Если так то сначала std::sort() потом std::unique().
Но если допустимо можно сразу использовать std::set (или std::map) вместо std::vector

Добавлено через 5 минут
Цитата Сообщение от Croessmah Посмотреть сообщение
remove_copy_if же есть в STL
Но в итоге все ровно нужно будет воспользоваться erase() так как remove только перемещает элементы в конец а не удаляет их.
Vourhey
Почетный модератор
6473 / 2248 / 123
Регистрация: 29.07.2006
Сообщений: 12,635
30.12.2012, 14:34     vector::erase()? #8
Цитата Сообщение от Avazart Посмотреть сообщение
Но в итоге все ровно нужно будет воспользоваться erase() так как remove только перемещает элементы в конец а не удаляет их.
Нет. Все элементы уйдут в output iterator без лишних. Потому что remove_copy
Avazart
7044 / 5221 / 259
Регистрация: 10.12.2010
Сообщений: 22,945
Записей в блоге: 17
30.12.2012, 14:35     vector::erase()? #9
Цитата Сообщение от Vourhey Посмотреть сообщение
Нет. Все элементы уйдут в output iterator без лишних. Потому что remove_copy
Ну да не углядел...
Но только это лишнее копирование тогда...
Vourhey
Почетный модератор
6473 / 2248 / 123
Регистрация: 29.07.2006
Сообщений: 12,635
30.12.2012, 14:47     vector::erase()? #10
Цитата Сообщение от Avazart Посмотреть сообщение
Но только это лишнее копирование тогда...
В случае unique тоже копирование. Просто в одном контейнере
DiffEreD
1427 / 764 / 95
Регистрация: 21.06.2011
Сообщений: 1,740
Записей в блоге: 2
30.12.2012, 14:53     vector::erase()? #11
Как по мне, unique самое то. Вот как бы я решил:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
#include <iostream>
#include <string>
#include <vector>
#include <algorithm>
 
struct person{
    std::string name;
    int age;
 
    person(const std::string& s, const int val): name(s), age(val) {}
};
 
int main()
{
    auto print = [](const std::vector<person>& per)
    {
        for (auto& val : per) std::cout<<val.name<<"\n";
    };
 
    std::vector<person> repw;
    repw.push_back(person("Igor", 25));
    repw.push_back(person("Slava", 25));
    repw.push_back(person("Igor", 30));
    repw.push_back(person("Svetick", 25));
    repw.push_back(person("Igor", 18));
 
    print(repw);
 
    std::sort(repw.begin(), repw.end(), [](const person& per1, const person& per2){return per1.name<per2.name;});
    repw.erase(std::unique(repw.begin(), repw.end(), [](const person& per1, const person& per2){return per1.name==per2.name;}), repw.end());
 
    std::cout<<"\n********************************\n";
    print(repw);
 
    std::cout<<"\n";
    system("pause");
    return 0;
}
Avazart
7044 / 5221 / 259
Регистрация: 10.12.2010
Сообщений: 22,945
Записей в блоге: 17
30.12.2012, 14:59     vector::erase()? #12
C++
1
std::map<std::string,int> personS;
-=ЮрА=-
Заблокирован
Автор FAQ
30.12.2012, 15:00     vector::erase()? #13
booker, вот так это делается
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
#include <ctime>
#include <vector>
#include <string>
#include <iostream>
using namespace std;
 
struct person
{
    string name;
    int age;
};
 
int main()
{
    int i;
    person p1;
    person p2;
    person px;
    p1.name = "name1";p1.age = 10;
    p2.name = "name2";p2.age = 15;
    vector <person> pList;
    //Рандомное заполение
    srand(time(0));
    cout<<"Input : "<<endl;
    for(i = 0; i < 20; i++)
    {
        if(rand() % 2 == 1)
            px = p1;
        else
            px = p2;
        cout<<"struct # "<<i + 1<<" :\t"<<px.age<<"\t"<<px.name<<endl;
        pList.push_back(px);
    }
    for(i = 1; i < pList.size(); i++)
    {
        if(pList[i].name == pList[i - 1].name)
            pList.erase(pList.begin() + i--);
    }
    cout<<"Output : "<<endl;
    for(i = 0; i < pList.size(); i++)
        cout<<"struct # "<<i + 1<<" :\t"<<pList[i].age<<"\t"<<pList[i].name<<endl;
    return 0;
}
PS:Основной твой недочёт что при erase позиция i соответсвует следующему после удаляемого элемента и ты её на итерации проскакиваешь полагая что на i-м месте всё также будет удалённый элемент. Обрати внимание на i-- в моём коде...
От себя - итераторы, тяжеловесные констукции - зачем это всё? std::vector допускает простую адресацию по индексу, как к элементу массива
Миниатюры
vector::erase()?  
Avazart
7044 / 5221 / 259
Регистрация: 10.12.2010
Сообщений: 22,945
Записей в блоге: 17
30.12.2012, 15:08     vector::erase()? #14
Цитата Сообщение от -=ЮрА=- Посмотреть сообщение
От себя - итераторы, тяжеловесные констукции - зачем это всё? std::vector допускает простую адресацию по индексу, как к элементу
Как раз через индексацию - медленнее...

Добавлено через 3 минуты
Цитата Сообщение от Vourhey Посмотреть сообщение
В случае unique тоже копирование. Просто в одном контейнере
Ну в принципе да, и тут как повезет (где меньше копирований будет)
go
Эксперт C++
3585 / 1365 / 128
Регистрация: 16.04.2009
Сообщений: 4,528
30.12.2012, 15:20     vector::erase()? #15
Цитата Сообщение от -=ЮрА=- Посмотреть сообщение
От себя - итераторы, тяжеловесные констукции - зачем это всё? std::vector допускает простую адресацию по индексу, как к элементу массива
Не согласен!
При полной оптимизации итератор превращается в обычный указатель, и доступ к нему - обычная операция разыменования. В случае индексации на каждой итерации происходит прибавление к базе значения (i * sizeof( type )).
booker
23 / 23 / 2
Регистрация: 21.11.2012
Сообщений: 106
30.12.2012, 15:40  [ТС]     vector::erase()? #16
Основной твой недочёт что при erase позиция i соответсвует следующему после удаляемого элемента и ты её на итерации проскакиваешь полагая что на i-м месте всё также будет удалённый элемент. Обрати внимание на i-- в моём коде...
точно не учёл что индексы сдвигаются, спасибо исправил всё заработало.

Добавлено через 11 минут
go, просто нет еще достаточного понимания контейнеров, поэтому с индексами будет попонятней пока.
Avazart
7044 / 5221 / 259
Регистрация: 10.12.2010
Сообщений: 22,945
Записей в блоге: 17
30.12.2012, 15:41     vector::erase()? #17
Цитата Сообщение от booker Посмотреть сообщение
go, просто нет еще достаточного понимания контейнеров, поэтому с индексами будет попонятней пока.
Ну собственно зачем тогда вообще использовать STL ?
booker
23 / 23 / 2
Регистрация: 21.11.2012
Сообщений: 106
30.12.2012, 15:51  [ТС]     vector::erase()? #18
нужен был вектор, обычный динамический массив был бы менее эффективен в моём случае, свой вектор писать не хотелось))
-=ЮрА=-
30.12.2012, 16:11
  #19

Не по теме:

booker, многие на этом форуме мнят себя супер экспертами, не понимая, что на поверку не смогут даже что-то маломальски сложное написать, так что не бери в голову некоторые пустые замечания. От себя добавлю, что лично сам использую индексную адресацию и каких либо тормозов я не замечал, так что используй на здоровье если нравиться, захочешь потом перейдёшь на итераторы это всегда можно успеть сделать

MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
30.12.2012, 16:13     vector::erase()?
Еще ссылки по теме:

C++ Ошибка “vector<bool> erase iterator outside range” при работе алгоритма Хаффмана
C++ Struct / Vector / Expression: vector subscript out of range
C++ При include<vector> не включает и не видит vector
Ошибка [Linker error] undefined reference to `Vector::Vector(int)' C++
C++ Ошибка при использовании erase для std::vector

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
Avazart
7044 / 5221 / 259
Регистрация: 10.12.2010
Сообщений: 22,945
Записей в блоге: 17
30.12.2012, 16:13     vector::erase()? #20
Всегда можно успеть пользоваться индексами, а с итераторов лучше начинать сразу...
Ибо это основы STL понимание которых дает возможности...
Yandex
Объявления
30.12.2012, 16:13     vector::erase()?
Ответ Создать тему
Опции темы

Текущее время: 22:45. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru