Nadużywanie dynamic_cast

Question

Cześć, czy jest dobrym stylem programowania robić coś takiego?

Nie chcę dla każdego typu wskaźnika pisać odzielnej funkcji.

void Truss::addElements(Element * obj)
{
    if ( dynamic_cast<CommonRafter *> (obj) ) {
        commonRaftersPtr = obj;
        return;
    }

    if ( dynamic_cast<HipRafter *> (obj) )  {
        hipRaftersPtr = obj;
        return;
    }

    if ( dynamic_cast<ValleyRafter *> (obj) )  {
        valleyRaftersPtr = obj;
        return;
    }

    if ( dynamic_cast<Purlin *> (obj) )  {
        purlinsPtr = obj;
        return;
    }

    if ( dynamic_cast<WallPlate *> (obj) )  {
        wallPlatesPtr = obj;
        return;
    }

    if ( dynamic_cast<CollarBeam *> (obj) )  {
        collarBeamsPtr = obj;
        return;
    }
}

 

Comments

Bez szerszego kontekstu jest mi ciężko jednoznacznie odpowiedzieć na to pytanie.
Pisanie osobnych funkcji dla każdego typu zajęłoby mniej miejsca. Jednak, jeżeli wszystkie elementy są przechowywane w jednym kontenerze o typie Element* to takie osobne funkcje i tak nie byłyby wykonywane.

Przygotowałem małe porównanie wydajności obu sposobów https://onlinegdb.com/HygoPVuYD
Użycie dynamic_cast okazało się 20x wolniejsze.

---

Dzięki za porównanie.

Właśnie chciałem wywoływać tę funkcję dla wskaźników z kontenera typu Element*, no ale może spróbuję tak jak koledzy niżej podpowiadają.

---

No to może być problem z pozbyciem się dynamic_cast (chyba że zaimplementujesz jakiś prosty mechanizm rozpoznawania typu klasy pochodnej).

Generalnie coś nie tak jest w projekcie (jeśli dobrze kojarzę, masz tu naruszenie zasady podstawienia Liskov).

---

Dodałem proste rozpoznawanie klasy (bazujące na typie wyliczeniowym i funkcji wirtualnej) https://onlinegdb.com/HyJzfudtw

---

Super, dzięki bardzo!

Answer 1

To z całą pewnością w 90% przypadków zły styl. Od takich rzeczy masz dynamiczny polimorfizm albo niektóre wzorce projektowe a by uniknąć jawnej detekcji typu. Masz oczywiście jeszcze std::variant https://en.cppreference.com/w/cpp/utility/variant .

Answer 2

Użycie polimorfizmu ma za zadanie między innymi zmniejszenie objętości kodu, a takie użycie dynamic_cast<> w tym krótkim fragmencie kodu jest jego zaprzeczeniem. Podejrzewam jakiś problem XY, ale bez całości kodu trudno powiedzieć coś więcej.

 

 

Comments

Też mi wyglądało to na zaprzeczanie polimorfizmu ale nie znam jeszcze innego sposobu na sprawdzenie typu. Spróbuję z std::variant. Dzięki.