2

次のvec_Aインスタンスを格納する がありますclass A。vec_A.push_back(A());

後の段階でベクター内のいくつかの要素を削除したいのですが、2 つの質問がvec_A.erase(iterator) あります。.

b) 条件if(num <5)が num が特定の numberList の中にある場合であると仮定します。これを考えると、以下に示すよりもベクトルの要素を削除するより良い方法はありますか?

#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;
}
4

2 に答える 2

4

a) 要素は次のように削除されます: vec_A.erase(iterator) メモリ リークがないことを確認するために追加する必要がある追加のコードはありますか? .

はい、それだけです。メモリ リークは発生しません。オブジェクトをヒープに割り当てなかったので、vec_A.push_back(A()) を実行すると、新しいオブジェクトがベクターにコピーされます。消去すると、ベクターが要素を削除します。

条件 if(num <5) は、num が特定の numberList 内にある場合であると仮定します。これを考えると、以下に示すよりもベクトルの要素を削除するより良い方法はありますか?

はい、イディオムを削除/消去できます。これは例です:

 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;
}
于 2010-04-30T09:31:01.087 に答える
1

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());}
}
于 2010-04-30T09:31:19.297 に答える