如何在pimpl实现中修复预期的primary-expression COMPILE ERROR?

时间:2013-12-20 18:59:02

标签: c++ templates c++11 compiler-errors pimpl-idiom

背景

我有两个coord_t

的实现
  1. simp_t只存储x,y
  2. dep_t,它接受​​一个从属父coord_t并为其添加一个偏移量
  3. 这些是较低级别的实现类。在用户级别,用法应如下所示:

    coordinate_t<>  ts( 3, 5 );
    coordinate_t<>  ts_derived( ts, 9 );  // ts + { 9, 9 }
    

    问题

    如果我使用std::unique_ptr<>实现基础,我有工作代码。但是,当我尝试将实现转换为pimpl<>时,我在g++ (Ubuntu/Linaro 4.7.3-1ubuntu1) 4.7.3上遇到以下编译错误:

    junk.cpp: In constructor ‘coordinate_t<T>::coordinate_t(T, T)’:
    junk.cpp:54:47: error: expected primary-expression before ‘>’ token
    junk.cpp: In constructor ‘coordinate_t<T>::coordinate_t(const coordinate_t<T>&, const T&)’:
    junk.cpp:58:46: error: expected primary-expression before ‘>’ token
    

    我做错了什么,如何解决这个问题才能正确构建?

    CODE

    代码按原样无法编译,但如果您注释掉#define USE_PIMPL直接使用std::unique_ptr<>,则编译正常。

    #include <memory>
    
    template<typename T>
    class pimpl
    {
            std::unique_ptr<T> m_up;
    
    public:
            pimpl() { }
            template<typename ...Args> pimpl( Args&& ...args )
                    : m_up{ new T{ std::forward<Args>(args)... } } { }
    
            template<typename D,typename ...Args>
            static pimpl<T> Derived( Args&& ...args )
            {
                    pimpl<T> x;
                    x.m.reset( new D{ std::forward<Args>(args)... } );
                    return x;
            }
    
            ~pimpl() { }
    };
    
    template<typename T>
    struct coord_t { };
    
    template<typename T>
    struct simp_t : public coord_t<T>
    {
            T  m_x, m_y;
            simp_t( T x, T y ) : m_x( x ), m_y( y ) { }
    };
    
    template<typename T>
    struct dep_t : public coord_t<T>
    {
            using parent_t = coord_t<T>;
            parent_t const&  m_parent;
            T                m_offset;
            dep_t( parent_t const& p, T offset ) : m_parent( p ), m_offset( offset )
            { }
    };
    
    #define USE_PIMPL    // if we comment this out and USE_UNIQUE_PTR, it works ok
    
    #ifdef USE_PIMPL
    template<typename T=int>
    class coordinate_t
    {
            pimpl<coord_t<T>>  m_impl;
    
            public:
                    coordinate_t( T x, T y ) :
                            m_impl( pimpl<coord_t<T>>::Derived<simp_t<T>>( x, y )) // ERROR HERE
                    {
                    }
                    coordinate_t( coordinate_t<T> const& parent, T const& offset ) :
                            m_impl( pimpl<coord_t<T>>::Derived<dep_t<T>>( parent, offset )) // ERROR HERE
                    {
                    }
                    ~coordinate_t() { }
    };
    #else
    template<typename T=int>
    class coordinate_t
    {
            std::unique_ptr<coord_t<T>>  m_impl;
    
            public:
                    coordinate_t( T x, T y ) :
                            m_impl{ new simp_t<T>( x, y ) }
                    {
                    }
                    coordinate_t( coordinate_t<T> const& parent, T const& offset ) :
                            m_impl{ new dep_t<T>( *parent.m_impl, offset ) }
                    {
                    }
                    ~coordinate_t() { }
    };
    #endif
    
    int main()
    {
            coordinate_t<>  ts( 3, 5 );
            coordinate_t<>  ts_derived( ts, 9 );  // ts + { 9, 9 }
    }
    

1 个答案:

答案 0 :(得分:2)

<强>解

juanchopanza 是对的:问题是,在调用类模板中的函数模板Derived之前,我需要一个模板关键字。

<强> FIXES / MODIFICATIONS

  1. 添加模板关键字以帮助编译器
  2. 因为类模板pimpl<>包含std::unique_ptr<>成员,所以需要移动构造函数,否则您将无法从函数返回实例
  3. 我在pimpl<>添加了便利运算符,因此从用户的角度看它更像std::unique_ptr<>
  4. 修正了m
  5. m_uppimpl<>的成员错误名称中的愚蠢重构错误
  6. 为了清晰起见,删除了非pimpl代码
  7. 工作代码

    template<typename T>
    class pimpl
    {
            std::unique_ptr<T> m_up;
    
    public:
            pimpl() { }
            pimpl( pimpl&& rhs ) : m_up( std::move( rhs.m_up )) { }
    
            template<typename ...Args>
            pimpl( Args&& ...args )
                    : m_up{ new T{ std::forward<Args>(args)... } } { }
    
            template<typename D,typename ...Args>
            static pimpl<T> Derived( Args&& ...args )
            {
                    pimpl<T> x;
                    x.m_up.reset( new D{ std::forward<Args>(args)... } );
                    return x;
            }
    
            ~pimpl() { }
    
            T* operator->() { return m_up.get(); }
            T& operator*() { return *m_up.get(); }
            T const* operator->() const { return m_up.get(); }
            T const& operator*() const { return *m_up.get(); }
    };
    
    template<typename T>
    struct coord_t
    {
    };
    
    template<typename T>
    struct simp_t : public coord_t<T>
    {
            T  m_x, m_y;
            simp_t( T x, T y ) : m_x( x ), m_y( y ) { }
    };
    
    template<typename T>
    struct dep_t : public coord_t<T>
    {
            using parent_t = coord_t<T>;
            parent_t const&  m_parent;
            T                m_offset;
            dep_t( parent_t const& p, T offset ) : m_parent( p ), m_offset( offset )
            { }
    };
    
    template<typename T=int>
    class coordinate_t
    {
            pimpl<coord_t<T>>  m_impl;
    
            using my_simp_t  = simp_t<T>;
            using my_dep_t   = dep_t<T>;
            using my_pimpl_t = pimpl<coord_t<T>>;
    
            public:
                    coordinate_t( T x, T y ) :
                            m_impl( my_pimpl_t::template Derived<my_simp_t>( x, y ))
                    {
                    }
                    coordinate_t( coordinate_t<T> const& parent, T const& offset ) :
                            m_impl( my_pimpl_t::template Derived<my_dep_t>( *parent.m_impl, offset ))
                    {
                    }
                    ~coordinate_t() { }
    };
    
    int main()
    {
            coordinate_t<>  ts( 3, 5 );
            coordinate_t<>  ts_derived( ts, 9 );  // ts + { 9, 9 }
    }