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

Не удаляет все положительные элементы из вектора - C++

Восстановить пароль Регистрация
 
Тамика
Котовчанин
 Аватар для Тамика
859 / 439 / 129
Регистрация: 16.02.2010
Сообщений: 2,537
Записей в блоге: 27
24.06.2014, 16:33     Не удаляет все положительные элементы из вектора #1
Добрый день, форумчане!
Читала на форуме, как правильно удалять элементы из вектора по индексу... Но почему-то у меня не получается. Прошу вашей подсказки. Может где-то что-то не учла...
В общем, код таков. Есть вектор, он сортируется методом вставки, а потом удаляются все положительные элементы. Это в идеале. Но моя функция
C++
1
void delete_pos(std::vector <int>& m)
пропускает часть положительных элементов... Объясните, пожалуйста, как правильно удалять элементы по индексу?
Всем заранее спасибо!
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
44
45
46
47
48
#include <iostream>
#include <vector>
#include <ctime>
 
void sort_mas(std::vector <int>& m)
{
    for (int i = 1; i < m.size(); ++i)
        for (int j = i; j && m[j] < m[j-1]; --j)
            std::swap(m[j], m[j-1]);
}
 
void show(std::vector <int>& m)
{
    std::cout << std::endl;
    for (int i = 0; i < m.size(); ++i)
        std::cout << m[i] << " ";
}
 
void delete_pos(std::vector <int>& m)
{
    for (int i = 0; i < m.size(); ++i)
    {
        if (m[i] > 0) m.erase(m.begin() + i);
            std::vector<int>(m).swap(m);
    }
}
 
int main() 
{
    srand(time(NULL));
    std::vector <int> data;
    std::cout << "Enter count of elemens - ";
    int n;
    std::cin >> n;
    for (int i = 0; i < n; ++i)
    {
        int c = -20 + rand()%(20 + 20);
        data.push_back(c);
    }
    show(data);
    sort_mas(data);
    std::cout << "\nVector after sorting"; 
    show(data);
    std::cout << "\nVector after deleting positive elements";
    delete_pos(data);
    show(data);
    system("pause>>null");
}
Добавлено через 6 минут
Попробвала без цикла, всё равно не то...
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
44
45
46
47
48
49
50
#include <iostream>
#include <vector>
#include <ctime>
#include <algorithm>
 
void sort_mas(std::vector <int>& m)
{
    for (int i = 1; i < m.size(); ++i)
        for (int j = i; j && m[j] < m[j-1]; --j)
            std::swap(m[j], m[j-1]);
}
 
void show(std::vector <int>& m)
{
    std::cout << std::endl;
    for (int i = 0; i < m.size(); ++i)
        std::cout << m[i] << " ";
}
 
bool if_positive(int i)
{
    return i > 0;
}
 
void delete_pos(std::vector <int>& m)
{
    m.erase(std::remove_if(m.begin(), m.end(), if_positive));
}
 
int main() 
{
    srand(time(NULL));
    std::vector <int> data;
    std::cout << "Enter count of elemens - ";
    int n;
    std::cin >> n;
    for (int i = 0; i < n; ++i)
    {
        int c = -20 + rand()%(20 + 20);
        data.push_back(c);
    }
    show(data);
    sort_mas(data);
    std::cout << "\nVector after sorting"; 
    show(data);
    std::cout << "\nVector after deleting positive elements";
    delete_pos(data);
    show(data);
    system("pause>>null");
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
24.06.2014, 16:33     Не удаляет все положительные элементы из вектора
Посмотрите здесь:

Преобразовать массив, чтобы сначала располагались все положительные элементы, а потом — все отрицательные C++
Заменить положительные элементы вектора А(10) нулями C++
C++ Программа которая удаляет из массива все отрицательные элементы
Дан массив A[1.50], элементы которого отличны от нуля. Расположить их в таком порядке, чтобы первыми были все положительные элементы, а затем - все о C++
C++ Функция удаляет из списка все отрицательные элементы
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Kuzia domovenok
 Аватар для Kuzia domovenok
1882 / 1737 / 116
Регистрация: 25.03.2012
Сообщений: 5,907
Записей в блоге: 1
24.06.2014, 16:34     Не удаляет все положительные элементы из вектора #2
Тамика, ну правильно! Чему равен m[i] после операции m.erase(m.begin() + i);
Тамика
Котовчанин
 Аватар для Тамика
859 / 439 / 129
Регистрация: 16.02.2010
Сообщений: 2,537
Записей в блоге: 27
24.06.2014, 16:38  [ТС]     Не удаляет все положительные элементы из вектора #3
Цитата Сообщение от Kuzia domovenok Посмотреть сообщение
Тамика, ну правильно! Чему равен m[i] после операции m.erase(m.begin() + i);
Блин, точно! Как всегда не мелочах сливаюсь. Спасибо!
А почему второй вариант не работает?..

del
Tulosba
:)
Эксперт C++
4378 / 3221 / 297
Регистрация: 19.02.2013
Сообщений: 9,044
24.06.2014, 16:45     Не удаляет все положительные элементы из вектора #4
Цитата Сообщение от Тамика Посмотреть сообщение
C++
1
m.erase(std::remove_if(m.begin(), m.end(), if_positive));
Что здесь по-Вашему происходит?

Добавлено через 2 минуты
Как-то так надо размер менять:
C++
1
m.resize( std::remove_if(m.begin(), m.end(), if_positive) - m.begin() );
Тамика
Котовчанин
 Аватар для Тамика
859 / 439 / 129
Регистрация: 16.02.2010
Сообщений: 2,537
Записей в блоге: 27
24.06.2014, 16:51  [ТС]     Не удаляет все положительные элементы из вектора #5
std::remove_if - вернет мне итератор преобразованного вектора, в котором все элементы, для которых "предикат" равен фолс, перенесены в конец вектора.
Возвращает нам итератор начала этой части, которую нужно удалить.
Затем с помощью erase удаляем этот кусок.
Чёрт, пока я это писала, то поняла, где зафейлила.

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

C++
1
m.erase(std::remove_if(m.begin(), m.end(), if_positive), m.end());
Вот так работает.
Yandex
Объявления
24.06.2014, 16:51     Не удаляет все положительные элементы из вектора
Ответ Создать тему
Опции темы

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