удаление объектов-элементов вектора std с помощью стирания: а) обработка памяти и б) лучший способ? - PullRequest
Новая
       10

удаление объектов-элементов вектора std с помощью стирания: а) обработка памяти и б) лучший способ?

2 голосов
/ 30 апреля 2010

У меня есть vec_A, который хранит экземпляры class A как: vec_A.push_back(A());

Я хочу удалить некоторые элементы в векторе на более поздней стадии и задать два вопроса: a) Элемент удален как: vec_A.erase(iterator) Есть ли дополнительный код, который мне нужно добавить, чтобы убедиться в отсутствии утечки памяти ? ,

b) Предположим, что условие if(num <5) состоит в том, что num относится к определенному списку номеров. Учитывая это, есть ли лучший способ удалить элементы вектора, чем то, что я иллюстрирую ниже? 

#include<vector>
#include<stdio.h>
#include<iostream>

class A {
      public:

             int getNumber();            
             A(int val);
             ~A(){};
      private:
              int num;
};

A::A(int val){
         num = val;
         };

int A::getNumber(){
    return num;
};

int main(){


    int i  =0;
    int num;
    std::vector<A> vec_A;
    std::vector<A>::iterator iter;

          for ( i = 0; i < 10; i++){
              vec_A.push_back(A(i));
          }
          iter = vec_A.begin();

          while(iter != vec_A.end()){
              std::cout <<  "\n --------------------------";
              std::cout <<  "\n Size before erase =" << vec_A.size();
              num = iter->getNumber() ;
              std::cout <<  "\n num = "<<num;
              if (num < 5){
                      vec_A.erase(iter);
                      }
              else{
                   iter++;
                   }

              std::cout <<  "\n size after erase =" << vec_A.size();
          }        


    std::cout << "\nPress RETURN to continue...";
    std::cin.get();

    return 0;
}

Ответы [ 2 ]

4 голосов
/ 30 апреля 2010

a) Элемент удаляется как: vec_A.erase (итератор) Есть ли дополнительный код мне нужно добавить, чтобы сделать уверены, что нет утечки памяти? .

Да, это все, что вам нужно сделать. Там не будет утечки памяти. Поскольку вы не размещали свой объект в куче, когда вы сделали vec_A.push_back (A ()), новый объект скопировал в вектор. Когда вы удалите, vector позаботится об удалении элементов.

Предположим, что условие if (num <5) имеет значение if num входит в число определенных списков номеров. Учитывая это, есть ли лучший способ удалить элементы вектора, чем что я иллюстрирую ниже? </p>

Да, вы можете удалить / стереть идиома. Это пример: 

 class A
{
public:
    A(int n) : m_n(n)
    {
    }

    int get() const
    {
        return m_n;
    }
private:
    int m_n;
};

bool lessThan9(const A& a)
{
    return a.get() < 9;
}

//Or if you want for a generic number
struct Remover : public std::binary_function<A,int,bool>
{
public:
    bool operator()(const A& a, int n)const
    {
        return a.get() < n;
    }
};

int main()
{
    std::vector<A> a;
    a.push_back(A(10));
    a.push_back(A(8));
    a.push_back(A(11));
    a.push_back(A(3));

    a.erase(std::remove_if(a.begin(), a.end(), lessThan9), a.end());

    //Using the user-defined functor
    a.erase(std::remove_if(a.begin(), a.end(), std::bind2nd(Remover(), 9)), a.end());

    return 0;
}
1 голос
/ 30 апреля 2010

1) Обработка ресурсов осуществляется самим классом. Деструктор класса отвечает за отсутствие утечки памяти.

2) Удаление элементов из вектора лучше всего делать задом наперед: 

for (std::vector<A>::reverse_iterator it = vec_A.rend(); it != vec_A.rbegin(); --it)
{
  if (it->getNumber() < 5) {vec_A.erase(it.base());}
}
...