Shared_ptr i dynamic_pointer_cast, zwraca empty choć nie powinien

Question

Cześć forumowicze!

Już nie jeden raz pomogliście mi uporać się z programistycznymi problemami i tym razem również liczę na Waszą pomoc. 

 

Posiadam w swoim programie następujące klasy:

accessories

enemies : accessories

chicken : enemies

box : accessories

chest : accessories

 

Posiadam również taką funkcję:

shared_ptr<accessories> map_level::objectsFactory(int type)
{
	shared_ptr<accessories> temp = nullptr;
	switch (TexNames(type))
	{
	case CHICKEN:
		temp = make_shared<accessories>(chicken());
		break;
	case CHEST:
		temp = make_shared<accessories>(chest());
		break;
	case COIN:
		temp = make_shared<accessories>(coin());
		break;
	}
	return temp;
}

Posiadam również 2 vectory:

	vector <shared_ptr <accessories>> objectsVector;
	vector <shared_ptr <enemies>> enemiesVector;

W zależności od obiektu, który zwraca objectsFactory() chciałbym umieścić dany obiekt do odpowiedniego wektora.

Próbowałem to zrobić tak:

shared_ptr<accessories> temp = objectFactory(type);

// rzutowanie w dół
shared_ptr<enemies> isEnemie = dynamic_pointer_cast<enemies, accessories>(temp);

if (isEnemie)
{
	enemiesVector.push_back(isEnemie);
else
{
	objectsVector.push_back(temp);
}

Problem w tym, że dla obiektu który jest chicken powyższe rzutowanie zwraca wskaźnik isEnemie jako stan empty, a więc wszystko wrzuca mi do tego drugiego wektora.

 

Będę wdzięczny za każdą pomoc i wskazówkę.  

 

Comments

Problem w tym, że dla obiektu który jest chicken powyższe rzutowanie zwraca isEmpty jako dosłownie empty

O co chodzi z isEmpty i dosłownie empty? 

---

Pomyłka, isEmpty to isEnemie, natomiast empty to wartość tego shared_ptr.

Już poprawiam :) 

Answer 1

temp = make_shared<accessories>(chicken());

Te linijki tworzą zawsze obiekt typu accessories. make_shared robi obiekt typu T z przekazanych argumentów. Więc w tym przypadku tworzysz accessories(chicken()). Konstruktor kopiujący obiektu accessories od obiektu chicken. To przechodzi bo klasa bazową chicken jest wlasnie accessories. I koniec końców fabryka zawsze zwraca Ci obiekt typu accessories.

Zmień te linijki na coś takiego to zadziała:

temp = shared_ptr<chicken>(new chicken());

ewentualnie jak lubisz make_shared to:

temp = make_shared<chicken>();

A tak na marginesie - opieranie przebiegu programu na sprawdzaniu typu dynamicznego przy pomocy RTTI to bardzo kiepskie rozwiązanie projektowe. Jeśli musisz użyć czegoś takiego to znaczy, że źle rozplanowałeś dziedziczenie.

Comments

Dziękuję za pomoc oraz wyjaśnienie. Wszystko oczywiście już działa, a ja znów się czegoś nauczyłem. 

Poszedłem też za radą i zmieniłem delikatnie strukturę programu, tak, że teraz zapisuje każdy obiekt w pliku mniej więcej tak:

int type; float x, y; bool isEnemie

A swoją fabrykę obiektów rozdzieliłem dodając fabrykę potworów. Łatwym kosztem pozbyłem się więc rzutowania. 

Jeszcze raz dzięki!