特定のクラスに一致する場合、ベースクラスオブジェクトのベクトルからオブジェクトを削除します

Question

私はいくつかの異なる動物の基本クラスである単純なクラスのAnimalを作成しました。次に、動物型オブジェクトをベクターに格納し、これらのオブジェクトで動作するメソッドをほとんど保持しない、Herdという別のクラスを作成しました。

class Herd { 
public: 

Herd() {} 
~Herd() {} 

Herd operator+(Animal arg) { 
    vec.push_back(arg); 
    return *this; 
} 

void operator+=(Animal arg) { 
    vec.push_back(arg); 
} 

Herd operator-(Animal arg) { 
    std::vector<Animal>::iterator position = std::find(vec.begin(), vec.end(), arg); 
    if (position != vec.end()) // == myVector.end() means the element was not found 
     vec.erase(position); 
    return *this; 

} 


void make_noise() { 
    vector<Animal>::iterator v = vec.begin(); 
    while (v != vec.end()) { 
     v->give_sound(); 
     v++; 
    } 
    cout << endl; 
} 

private: 
vector<Animal> vec; 
}; 

問題は、Vectorから特定のオブジェクトを削除することです。私は演算子をオーバーロードしました - そして、私はいくつかのAnimalから派生したクラスをパラメータとして取り込み、ベクトルからこのクラスの最初の出現を取り除きたいと思いました：ベクトルがDog、Cat、Dogを含んでいるとしましょう - 、 犬。私は私のベクトルを反復処理し、argオブジェクトに一致する要素を見つけることでこれを行います。

残念なことに、Catを引数として渡した後、私のクラスはそれを動物型のように扱いますので、関数に渡すものは常に最初の要素が削除されます。

私は、オブジェクトではなくポインタのベクトルを作ることでこれを達成しようとしましたが、私の全体のプログラムは、この全部の動作を修正する方法を知らないエラーでカバーします。

Answer 1

この場合、object slicingを体験します。それを動作させるには、ポインタを使用する必要があります。しかし、この場合はメモリを解放する必要がないので、std::shared_ptrまたはstd::unique_ptrを生ポインタの代わりにベクトルに格納することをお勧めします。

最後に、正確なタイプを確認するには、ポインタdynamic_castを試してみるか、typeidをチェックしてください。あるいは、ポリモーフィックのtype()関数を作成し、それを各派生クラスでオーバーライドして正確な型を返すこともできますが、この場合は恐らく過剰です。

vector<shared_ptr<Animal>> vec; 

for(auto animal : vec) 
{ 
    if(dynamic_cast<Cat*>(animal.get())) 
    { 
     //cat 
    } 
    if(typeid(*animal) == typeid(Cat)) 
    { 
     //also a cat 
    } 
}