Problem z deque

Witam,
mam problem z deque, jak podejrzewam, pierwszy raz używam "gotowej" kolejki. Program się uruchamia, ale wywala błąd gdy dochodzi do listy. "Wymuszono zakonczenie..." W pewnym momencie qDebug() <<this->Mapa->obiekty.size() << " pierwszy"; wywalało siże jakąś dużą liczbę a nie 1,2...

void CGame::create()
{


    CTank *czolg = new CTank;
    this->Mapa->obiekty.push_front(*czolg);
    qDebug() <<this->Mapa->obiekty.size() << " pierwszy";
     
    CTank *czolg2 =new CTank;
    this->Mapa->obiekty.push_front(*czolg2);
    qDebug() <<this->Mapa->obiekty.size() << " drugi";

}

class CMap
{
int szerokosc;
int wysokosc;
public:
    CMap();
    std::deque<CObject> obiekty;
};

CMap *Mapa było :P
ale już Mapa=new CMap nie :P

Czesc
Niestety ale rozwiazalo to tylko część mojego problemu. Nadal nie działa mi taka kolejka.
Ogólnie chce stworzyć kolejkę zawierającą różnej elementy różnych klas. Wszystkie dziedziczą po CObject. Uzywam virtual void. i chce przeleciec wszystkie elementy petla for by wykonala sie funkcja akcja. Ale akcja nie klasy bazowej. A akcja wszystkich klas pochodnych. Ale nie dziala...

    CTank *czolg2 =new CTank;
    this->Mapa->obiekty.push_back(*czolg2);
    czolg2->hp;
    this->Mapa->obiekty[1].x;
    qDebug() <<this->Mapa->obiekty.size() << " drugi";

Problem jest z polimorfizmem albo coś z tą kolejką. np. jak dodam do std::deque<CObject> obiekty; mój element czolg2. to dodaje ten element. Ale jest to element CObject i nie mam dostepu do atrybutów czolg2.

np. nie moge wykonać this->Mapa->obiekty[1].hp (hp jest w CTank, a nie w CObiekty).

Ktoś ma jakiś pomysł?

supermario napisał(a):

Ogólnie chce stworzyć kolejkę zawierającą różnej elementy różnych klas. Wszystkie dziedziczą po CObject. Uzywam virtual void. i chce przeleciec wszystkie elementy petla for by wykonala sie funkcja akcja. Ale akcja nie klasy bazowej. A akcja wszystkich klas pochodnych.
...
Ktoś ma jakiś pomysł?

Ja mam pomysł, że to jest złe użycie polimorfizmu -- jeśli dobrze Cię rozmumiem. :)

Jak chcesz wywołać "akcję nie klasy bazowej tylko klas pochodnych"? Chodzi o funkcję wirtualną, czy nie? Jeśli wirtualną, to nie powinno być problemu:

for(auto & x : zestaw) {
    x->akcja();
}

Ale jeśli nie, to jak inaczej chcesz osiągnąć to co chcesz? Wtedy jest to właśnie złe użycie polimorfizmu.

Gdzie w Twoim kodzie jest wspomniana funkcja akcja()? Gdzie jest jej wykorzystanie?
Coś piszesz, coś wklejasz, coś strzelasz i zgadujesz. Żadnych konkretów by można było Ci pomóc.

Po drugie: Co rzeczywiście jest problemem? Raz piszesz, że:

by wykonala sie funkcja akcja. Ale akcja nie klasy bazowej.

A za chwilę, że:

i nie mam dostepu do atrybutów czolg2.

A na koniec (i na początek) rzucasz tylko tyle, że:

Ale nie dziala...

Co się dokładnie dzieje? Jaki błąd otrzymujesz? Czego oczekujesz - co dostajesz?

noom może trochę chaotycznie napisałem.
Ale problem rozwiązany. Dla mających ten sam problem w przyszłości odpowiedz : metody virtualne działają tylko dla wskaznikow. A wiec w deklaracji deque powinna być dodana *

std::deque<CObject*> obiekty;

supermario napisał(a):

noom może trochę chaotycznie napisałem.
Ale problem rozwiązany. Dla mających ten sam problem w przyszłości odpowiedz : metody virtualne działają tylko dla wskaznikow.

No co Ty??? :)

A poważnie -- dla referencji też. :)

A wiec w deklaracji deque powinna być dodana *

std::deque<CObject*> obiekty;

Lepiej std::deque<std::unique_ptr<CObject>> obiekty; albo, jeśli to nie wystarczy: std::deque<std::shared_ptr<CObject>> obiekty;.