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

@Тамика · Регистрация: 16.02.2010

Добрый день, форумчане!
Читала на форуме, как правильно удалять элементы из вектора по индексу... Но почему-то у меня не получается. Прошу вашей подсказки. Может где-то что-то не учла...
В общем, код таков. Есть вектор, он сортируется методом вставки, а потом удаляются все положительные элементы. Это в идеале. Но моя функция

C++

1	void delete_pos(std::vector <int>& m)

пропускает часть положительных элементов... Объясните, пожалуйста, как правильно удалять элементы по индексу?
Всем заранее спасибо!

C++

#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++

#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");
}

@Kuzia domovenok · 24.06.2014, 16:34

Тамика, ну правильно! Чему равен m[i] после операции m.erase(m.begin() + i);

@Тамика · 24.06.2014, 16:38 **[ТС]**

Сообщение от Kuzia domovenok

Тамика, ну правильно! Чему равен m[i] после операции m.erase(m.begin() + i);

Блин, точно! Как всегда не мелочах сливаюсь. Спасибо!

А почему второй вариант не работает?..

del

Tulosba · 24.06.2014, 16:45

Сообщение от Тамика

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() );

@Тамика · 24.06.2014, 16:51 **[ТС]**

std::remove_if - вернет мне итератор преобразованного вектора, в котором все элементы, для которых "предикат" равен фолс, перенесены в конец вектора.
Возвращает нам итератор начала этой части, которую нужно удалить.
Затем с помощью erase удаляем этот кусок.
Чёрт, пока я это писала, то поняла, где зафейлила.

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

C++

1	m.erase(std::remove_if(m.begin(), m.end(), if_positive), m.end());

Вот так работает.

@Kuzia domovenok 1892 / 1747 / 119 Регистрация: 25.03.2012 Сообщений: 5,936 Записей в блоге: 1
	24.06.2014, 16:34	#2
	Тамика, ну правильно! Чему равен m[i] после операции m.erase(m.begin() + i); 1

@Тамика Котовчанин 912 / 455 / 145 Регистрация: 16.02.2010 Сообщений: 3,144 Записей в блоге: 27
	24.06.2014, 16:38 [ТС]	#3
	Сообщение от Kuzia domovenok Тамика, ну правильно! Чему равен m[i] после операции m.erase(m.begin() + i); Блин, точно! Как всегда не мелочах сливаюсь. Спасибо! А почему второй вариант не работает?.. del 0

	24.06.2014, 16:51

Опции темы

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

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