Pergunta

Eu quero criar uma função que pode assumir diferentes tipos de iteradores que armazenam o mesmo tipo de objeto:
O primeiro é um contendo std::map shared_ptr<Foo> (typedef-ed como FooMap) e o outro é um std::list que também contém shared_ptr<Foo> (FooList).

Eu realmente gosto os MSalters solução sugerida por uma questão semelhante e tentaram implementar iteradores boost::variant , qual a função terá como parâmetros para fazer uma iteração do primeiro para o segundo.

minha função é assim (simplificado um pouco):

set<Foo> CMyClass::GetUniqueFoos(FooIterator itBegin, FooIterator itEnd)
{
    set<Foo> uniques;
    for(/**/;
        apply_visitor(do_compare(), itBegin, itEnd);  // equals "itBegin != itEnd"
        apply_visitor(do_increment(), itBegin))       // equals "++itBegin"
    {
        // Exact mechanism for determining if unique is omitted for clarity
        uniques.insert( do_dereference< shared_ptr<Foo> >(), itBegin) );
    }

    return uniques;
}

O FooIterator e os visitantes são definidos da seguinte forma:

typedef
    boost::variant<
        FooMap::const_iterator,
        FooList::const_iterator>
    FooIterator;

struct do_compare : boost::static_visitor<bool>
{
    bool operator() (
        const FooMap::const_iterator & a,
        const FooMap::const_iterator & b) const
    { return a != b; }

    bool operator() (
        const FooList::const_iterator & a,
        const FooList::const_iterator & b) const
    { return a != b; }
};

struct do_increment: boost::static_visitor<void>
{
    template<typename T>
    void operator()( T& t ) const
    { ++t; }
};

template< typename Reference >
struct do_dereference: boost::static_visitor<Reference>
{
    template<typename T>
    Reference operator()( const T& t ) const
    { return *t; }
};

Eu tenho a maioria dos acima do destacamento de este e-mail . Essa solução também usa adaptadores e políticas, o que parece ser um pouco demais, de acordo com a resposta de MSalters, então eu não quero simplesmente copiar esse código. Especialmente porque eu só compreender parte dela.

Com o código acima eu recebo o seguinte erro compilador VS2008 (este é apenas as primeiras linhas de 160 no total, que eu acho que é um pouco demais para postar aqui, no entanto eu vou ser feliz para adicioná-los Se alguém quer ver tudo):

1>c:\boost\boost\variant\detail\apply_visitor_binary.hpp(63) :
 error C2664: 'bool CMyClass::do_compare::operator ()(
 const std::list<_Ty>::_Const_iterator<_Secure_validation> &,
 const std::list<_Ty>::_Const_iterator<_Secure_validation> &) const' :
 cannot convert parameter 1 from 'T0' to
 'const std::list<_Ty>::_Const_iterator<_Secure_validation> &'
1>        with
1>        [
1>            _Ty=shared_ptr<Foo>,
1>            _Secure_validation=true
1>        ]
1>        Reason: cannot convert from 'T0' to 'const std::list<_Ty>::_Const_iterator<_Secure_validation>'
1>        with
1>        [
1>            _Ty=shared_ptr<Foo>,
1>            _Secure_validation=true
1>        ]
1>        No user-defined-conversion operator available that can perform this conversion, or the operator cannot be called
1>        c:\boost\boost\variant\variant.hpp(806) : see reference to function template instantiation 'bool boost::detail::variant::apply_visitor_binary_invoke<Visitor,Value1>::operator ()<T>(Value2 &)' being compiled
1>        with
1>        [
1>            Visitor=const CMyClass::do_compare,
1>            Value1=T0,
1>            T=T1,
1>            Value2=T1
1>        ]
[...]

O que estou fazendo de errado?

Foi útil?

Solução

Eu suspeito que você não encontrou casos em seu static_visitor do_compare. Lembre-se, as variantes pode ter qualquer coisa, então você precisa de todas as combinações possíveis, como comparar um FooList :: const_iterator a um FooMap :: const_iterator. Ele está reclamando porque o compilador está tentando encontrar algum jogo para esse caso, e não pode converter um FooMap :: const_iterator a um FooList :: const_iterator.

martelando-lo:

struct do_compare : boost::static_visitor<bool>
{
    bool operator() (
        const FooMap::const_iterator & a,
        const FooMap::const_iterator & b) const
    { return a != b; }

    bool operator() (
        const FooList::const_iterator & a,
        const FooList::const_iterator & b) const
    { return a != b; }

    bool operator() (
        const FooMap::const_iterator & a,
        const FooList::const_iterator & b) const
    { return false; }

    bool operator() (
        const FooList::const_iterator & a,
        const FooMap::const_iterator & b) const
    { return false; }
};

Aqui está uma versão com modelos:

template <typename A, typename B>
bool operator() (
    const A & a,
    const B & b) const
{ return false; }

template <typename A>
bool operator() (
    const A & a,
    const A & b) const
{ return a != b; }

É compilar em comeau, mas eu não estou 100% ele vai trabalhar, então alguns testes é necessária. Outros que mais limpo, mais versátil de código, ele não deve ter qualquer efeito, enquanto ele funciona.

Licenciado em: CC-BY-SA com atribuição
Não afiliado a StackOverflow
scroll top