Посетитель и двойная отправка без переопределения метода accept в С++

Любое элегантное решение по этому поводу?

Не совсем. Вот что делает шаблон посетителя немного болезненным. Хотя вы можете немного уменьшить дублирование с помощью шаблона:

class BaseComposit 
{
private:  

    std::vector<BaseComposit*> nodes;

protected:

    template<class T>
    void accept_impl( Visitor* v, T* this_ )
    {
        v->visit( this_ );

        for(int i = 0; i < nodes.size(); i++)
            nodes[i]->accept(v);
    }

public:

    virtual void accept( Visitor* v ) { accept_impl( v, this ); }
};

Теперь дублирование, которое должен выполнить accept, меньше.

Кроме того, как указал @Oliv, в вашем примере действительно должна быть accept виртуальная функция. В противном случае все это не будет работать.

Если вы чувствуете себя действительно предприимчивым, вы можете ввести макрос, чтобы упростить «введение» accept в каждый класс. Вот так:

#define INJECT_ACCEPT() void accept( Visitor* v ) override { accept_impl( v, this ); }

Но вам все равно нужно использовать его в каждом производном классе, чтобы определение появилось.

class DerivedComposit : public BaseComposit
{
public:
    INJECT_ACCEPT();
};

Точка с запятой разрешена любопытной особенностью грамматики C++. Поэтому я полагаю, что можно утверждать, что вышеизложенное выглядит «естественно».

Если вы хотите, чтобы какой-то код всегда выполнялся, используйте шаблон невиртуального интерфейса (NVI):

class BaseComposit 
{
private:  
    std::vector<BaseComposit*> nodes;

    virtual void call_visit(Visitor* v) { v->visit(this); }

public:

    void accept(Visitor* v)
    {
        call_visit(v);

        for(int i = 0; i < nodes.size(); i++)
            nodes[i]->accept(v);
    }
};

class DerivedComposit : public BaseComposit
{
    void call_visit(Visitor* v) override { v->visit(this); }
public:
};