Частое использование dynamic_cast в конкретных целях. Правильно ли?

void Add(Drawable* drawable)
{
...
    if (Parallelogram* pg = dynamic_cast<Parallelogram*>(drawable))
    {
        ...
    }
...
}

void Polygon::Add(Drawable *pDrawable)
{
    pDrawable->CalculateBestPosition(this);
}

class Depictable : public SomeLib::Drawable 
{
    ...
};

class Visitor;
 
class Depictable : public Lib::Drawable
{
public:
    ...
    void SetPosition(vector2f const& coord);
    virtual void accept(class Visitor*) = 0;
    ...
}
 
class Parallelogram : public Depictable
{
public:
    ...
    void accept(Visitor*);
    ...
}
 
void Parallelogram::accept(Visitor *v)
{
    v->visit(this);
}
 
class Visitor
{
    ...
  public:
    virtual void visit(Parallelogram*) = 0;
    ...
};
 
class CalcPositionVisitor : public Visitor
{
    ...
public:
    CalcPositionVisitor(Polygon* polygon) : m_polygon(polygon) {}
    virtual void visit(Parallelogram*);
private:
    Polygon* m_polygon;
    ...
};
 
class Polygon
{
    ...
public:
    void Add(Depictable* depictable);
private:
    std::list<Depicatble*> m_depicatbles;
    ...
};
 
void Polygon::Add(Depictable* depictable)
{
    CalcPositionVisitor calcPosition(this);
    depictable->accept(&calcPosition);
    depictable->SetPosition(calcPosition.getPosition());
    
    m_depicatbles.push_back(m_depictable);
}