オブジェクトへのポインタの配列を管理するには？

私はオブジェクトの動的ベクトルを生成し、別のクラスでそれを操作するためにそれを返す必要があるオブジェクトへのポインタの配列:(..オブジェクトへのポインタの配列を管理するには？

に問題がある。以下のコードでありますそれを継承しをインスタンス化することができる抽象的でCarArrivalある Eventクラス

生成し、私はこの機能を持っている配列を埋めるクラスインサイド：。

Event** EventGenerator::getEvents() { 

Event* cars[EVENTS]; 

for (int i=0; i<EVENTS; i++) { 
    cars[i] = new CarArrival(generator->getNextNumber(8,(float)sqrt(0.4))); 
} 

sort(cars, cars+(EVENTS), Event::cmp); 

return cars;

}

私は、このようにontherクラスでこの関数を呼び出します。私は、「セグメンテーション違反」を取得する最初の要素の印刷後

Event** cars = generator->getEvents(); 

for(int i=0; i<EVENTS; i++) { 
    cout << i <<":" << (*cars)[i]->getScheduleTime() << endl; 
}

。

私はいくつかのことをオンラインで読んできましたが、（* cars）はという配列の最初の要素へのポインタを評価しているので間違いがあることを理解しています。 2番目のクラスの配列のすべての要素にアクセスする方法を理解します。

どうすればこの問題に直面することができますか？すべての

おかげで、

アルベルト

出典

2011-12-06 kappaalby

は、「私は動のを生成する必要が**ベクトル**オブジェクトの」...うーん、唯一あった場合、 'のstd :: vector' ...あなたが本当にしなければならない場合、 –

はポインタのみを使用します。可能な限りSTLを使用してください。十分にテストされており、多くの問題を解決します。 – hochl

私はあなたの代わりにstd::vector<Event*>を使用することをお勧めしたいです。あなたはこのように多くの苦痛を救うでしょう。それは、バックグラウンドですべての厄介なメモリ管理を処理し、あなたはそれに任意の数の項目を簡単に押すことができます。あなたの場合の最良の部分は、通常の配列では安全でないreturnvectorです。

Event* cars[EVENTS];もあなたのローカルで宣言されています。完了したら、それは存在しなくなります。かもしれないあなたのSegfaultを引き起こします。アレイをnewで動的に割り当てる必要がありますが、それでもstd::vectorで試してみてください（documentation hereを参照）。

EDIT：使用例：

std::vector<Event*> EventGenerator::getEvents() { 
    std::vector<Event*> cars; 
    for (int i=0; i<EVENTS; i++) { 
     cars.push_back(new CarArrival(generator->getNextNumber(8,(float)sqrt(0.4)))); 
    } 

    sort(cars.begin(), cars.end(), Event::cmp); 

    return cars; 
}

std::vector<Event*> cars = generator->getEvents(); 

for(int i=0; i<cars.size(); i++) { 
    cout << i <<":" << (*cars)[i]->getScheduleTime() << endl; 
}

出典

2011-12-06 11:17:55 Constantinius

ありがとうございました！ – kappaalby